+11,565
−0
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BbangSe01
reviewed
Oct 31, 2024
There was a problem hiding this comment.
제이님! 현재 실습이나 미션을 실행하실 때 실습/미션별로 폴더를 만들고 폴더별로 영화사이트 내용을 통째로 붙여서 작업하고 계신 것 같아요! 그런데 이렇게 올리시면 하나의 commit에 모든 파일이 다 변경사항으로 올라가서 진짜로 확인해야 하는 변경사항을 파악하기가 어려워져요!
현재 저는 영화사이트 폴더를 복사 붙여넣기 하지 않고 폴더명만 3-5주차 이런식으로 바꿔서 이어서 작업하고 있어요. 어차피 commit을 하면 제이님이 어떤 부분에 어떻게 코드를 작성했는지 기록이 되기 때문에 일일이 폴더별로 통째로 복붙을 해서 작업하실 필요는 없으실 것 같아요..! 앞으로는 이어서 작업하고 commit을 상세히 쪼개서 올려주시면 너무 좋을 것 같습니다!
| <StyledTxt style={{marginBottom: '40px', textAlign: 'center'}}>회원가입</StyledTxt> | ||
| <StyledInput type={'email'} placeholder='이메일을 입력해주세요!' {...register("email")}/> | ||
| {/* register 한 이름에 맞게 연결해주세요! */} | ||
| <ErrorTxt>{errors.email?.message}</ErrorTxt> |
There was a problem hiding this comment.
useState와 onBlur 함수를 이용해서 사용자가 입력하지도 않았는데 오류메세지가 뜨는 상황을 방지하는 게 좋을 것 같아요!
|
옙!! 그렇게 하면 무슨 파일을 수정한건지 훨씬 수월하게 파악할 수 있겠네요!! 기능을 구현할 때마다 중간중간 commit을 상세히 남기는 습관을 들여야 겠어요. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Merge pull request는 모든 리뷰가 완료된 후에 눌러주세요!
1. Add a title
2. Reviewers
3. Assignees
4. Labels