Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KDT0_KimSoJeong 직원 관리 서비스 #67

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

KSJT
Copy link

@KSJT KSJT commented Aug 19, 2023

구현 사항

  • 직원 관리 리스트 페이지
스크린샷 2023-08-19 오후 6 16 40
  • 직원 등록하기: 버튼 클릭 시 직원 등록 폼으로 이동

  • 직원 정보 삭제: 버튼 클릭 시 삭제

  • 직원 정보 수정*: 버튼 클릭 시 직원 정보 수정 폼으로 이동

  • 직원 프로필 페이지

스크린샷 2023-08-19 오후 6 20 16

유저 플로우

스크린샷 2023-08-19 오후 6 37 34
  • 아쉬운 점
    state 관리/ firebase 이용에 미숙해서 직원 정보 수정 기능 구현을 제대로 못 함
    모바일 반응형 구현 안 함
    애니메이션 구현 안 함
    firebase db와 storage, store 컴포넌트 사용해서 처음부터 다시 제작 구현해보겠음

@ewinkite
Copy link

혹시 저만 배포 페이지가 안보일까요 ㅜ 업데이트 되면 사이트와 코드 함께 다시 확인해보겠습니다 ㅎㅎ 힘든 상황속에서도 고생많으셨습니다!!

@YongYong21
Copy link

현재까지 구현한 부분은 잘 작성하신 것 같습니다.

아직 구현을 못한 부분이 많기 때문에 나중에 꼭 구현을 해보시는 것을 추천 드립니다.

@dabin-Hailey
Copy link

배포링크가 없어서 코드만 보는 것은 어려웠지만.. (이건 온전히 저의 부족입니다 ㅎㅎ)
코드 보면서 느낀 점은 async, await 패턴 사용해서 비동기 처리를 하려고 노력하신 것같네요!
저는 과제할 당시에 비동기처리에 대한 학습이 부족해서 적용하지 못했는데 다음 리팩토링 때 개선해봐야겠습니다~

그 외에는 코드 작성하실 때 주석을 작성해보시면 어떨까요?
다른 개발자가 소정님의 코드를 읽을 때도 도움이 되지만, 소정님 본인이 코드를 작성하실 때도 분명 도움이 될거에요! 지난 번에 저희 멘토링 시간에 코드 작성하다 잠시 집중이 흐려지만 어느 부분을 작성하고 있었는지조차 헷갈린다 했는데 주석을 작성하는 것으로 그런 점을 해결할 수 있더라구요! 참고해보세요 :)

Copy link

@LEEJAEHYUB LEEJAEHYUB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전반적으로 레이아웃을 구성하고, javascript로 html파일 간 라우팅을 구현하시고
그 다음에 기능을 붙혀보세요! 일단 기능을 구현할 때, 클라우드 기능을 사용하지 않는다고 생각하시고 구현한 다음 구현이 된다 싶으면 그 다음에 클라우드 기능을 적용해보세요!
javascript를 작성하는 방식이나 순서가 잘못되지는 않았습니다.
한번에 모든걸 구현하려고 하는 것보다는 천천히 한 스텝씩 구현해보세요!
클론코딩처럼 HTML 마크업을 먼저하고
CSS로 간단하게 스타일을 입히고
Javascript로 기능들을 구현해보고
다 되었다 싶으면 클라우드를 연동해보시면 좋을 것 같습니다 :)

async function addStaff(e) {
e.preventDefault();

var file = document.querySelector('.image').files[0];

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

변수 선언할 때 let, const에 대해 학습하고 사용해보세요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants