Skip to content

Conversation

@ryanlee911
Copy link

@ryanlee911 ryanlee911 commented May 5, 2025

1.무엇을?

React Router로 Todo 페이지와
로그인 / 회원가입 페이지 나눠주기

2.상세설명

루트 페이지에 로그인/회원가입 버튼
회원 가입 할 때 이름/아이디/비밀번호를 입력
로그인 화면에서는 회원가입 페이지로 이동할 수 있는 버튼 구현
스크린샷 2025-05-05 145008
스크린샷 2025-05-05 145017
스크린샷 2025-05-05 145040

@ryanlee911 ryanlee911 changed the title [5주차]이정윤 - 로그인/회원가입 UI구현 [5주차]이정윤 / 로그인,회원가입 UI구현 May 5, 2025
Copy link
Member

@yezzan9 yezzan9 left a comment

Choose a reason for hiding this comment

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

5주차 과제도 수고하셨습니다!

이전주차부터 꾸준히 언급 해드리고 있는데, 투두 페이지의 디자인이 요구사항과 다르게 구현되어있습니다. 해당 부분을 주어진 과제에 맞게 수정하지 않을 경우 패널티 사유가 될 수 있으니 꼭! 수정부탁드립니다😊


base 브랜치가 Leets-official:이정윤/main 이 되도록 수정해주세요!
스크린샷 2025-05-08 오전 12 09 03


정윤님이 구현해주신 부분과 과제를비교해보면, 정렬이 다르게 구현되어있어요😂
과제와 동일하게 정렬 수정 부탁드립니다 !

스크린샷 2025-05-08 오전 12 06 44 스크린샷 2025-05-08 오전 12 06 55

package.json Outdated
Comment on lines 13 to 22
<<<<<<< HEAD
"@types/nanoid": "^2.1.0",
"nanoid": "^5.1.5",
"react": "^19.0.0",
"react-dom": "^19.0.0",
"react-router-dom": "^7.5.3"
=======
"react": "^19.0.0",
"react-dom": "^19.0.0"
>>>>>>> a6d231f497726e90ded202a3730c47258c1d5a19
Copy link
Member

Choose a reason for hiding this comment

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

충돌을 해결하신 후에는 <<<<<<< HEAD 이런식으로 나타나는 부분을 모두 삭제해주셔야합니다 !
이렇게 충돌 내용이 남아있을 경우 에러가 나타날 수 있어요😧

Copy link
Member

Choose a reason for hiding this comment

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

App 파일이 두개 있는 이유가 뭘까용..?! 해당 파일은 사용되지 않는 것 같으니 삭제해주세요!

@ryanlee911 ryanlee911 changed the base branch from main to 이정윤/main May 8, 2025 05:44
@ryanlee911
Copy link
Author

넵 UI 사진과 동일하게 수정 후 재커밋했습니다. app파일은 지난주차들 과제 내용이라 혹시 몰라 냅뒀습니다. 피드백 감사합니다

@yezzan9
Copy link
Member

yezzan9 commented May 8, 2025

넵 UI 사진과 동일하게 수정 후 재커밋했습니다. app파일은 지난주차들 과제 내용이라 혹시 몰라 냅뒀습니다. 피드백 감사합니다

수정해주신 부분 확인했습니다 !
Todo 페이지 스타일도 과제와 동일하게 수정해주세요☺️
@ryanlee911

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.

2 participants