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

feat: select 페이지 생성 #11

Merged
merged 29 commits into from
Sep 6, 2024
Merged

feat: select 페이지 생성 #11

merged 29 commits into from
Sep 6, 2024

Conversation

WithJo
Copy link
Collaborator

@WithJo WithJo commented Sep 4, 2024

구현한 기능

논의하고 싶은 내용

기타

src/Router.tsx Outdated Show resolved Hide resolved
src/constants/Question.tsx Outdated Show resolved Hide resolved
src/pages/SelectPage.tsx Outdated Show resolved Hide resolved
@junyeokk
Copy link

junyeokk commented Sep 5, 2024

구현하시느라 고생 많으셨습니다. 전체적으로 커밋들을 세분화하신 부분은 잘 진행하신 것 같습니다.

다만 코드들이 조금 정돈되면 좋겠다는 느낌을 많이 받은 것 같아요. 아무래도 일관성을 지킨 코드는 나중에 유지보수도 쉽고 협업하기에도 더 수월할 것이기 때문에 그 부분만 조금만 조심해서 코드 작성하면 좋을 것 같습니다!

src/pages/SelectPage.tsx Outdated Show resolved Hide resolved
Copy link
Member

@toothlessdev toothlessdev left a comment

Choose a reason for hiding this comment

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

LGTM!

@toothlessdev toothlessdev merged commit b7df9b8 into develop Sep 6, 2024
1 of 2 checks passed
@toothlessdev toothlessdev deleted the feat/select branch September 6, 2024 14:55
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.

3 participants