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

🎨 App폴더 하위 디렉토리 구조 변경 #139

Merged
merged 4 commits into from
Dec 26, 2023

Conversation

woobottle
Copy link
Collaborator

🤔 해결하려는 문제가 무엇인가요?

closed #85

🎉 변경 사항

  • app 폴더 하위 디렉토리의 구조를 변경하였습니다
  • url의 이름또한 변경했습니다
    certification -> record
    complete -> success
    select -> new
  • src/constants/router.ts에 전체 router가 관리되어질 수 있도록 하였습니다
image

🙏 여기는 꼭 봐주세요!

사용 방법

🌄 스크린샷

📚 참고

https://nextjs.org/docs/app/building-your-application/routing/dynamic-routes

@woobottle woobottle added the enhancement New feature or request label Dec 25, 2023
@woobottle woobottle self-assigned this Dec 25, 2023
Copy link

vercel bot commented Dec 25, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
10mm-client-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 26, 2023 6:16am

Copy link
Contributor

Copy link
Collaborator

@wade3420 wade3420 left a comment

Choose a reason for hiding this comment

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

이전 ROUTER 객체를 다른 곳에서 이미 쓰고 있었던 것 같아요 (빌드 깨짐 에러)

스크린샷 2023-12-25 오후 6 36 52

@woobottle
Copy link
Collaborator Author

이전 ROUTER 객체를 다른 곳에서 이미 쓰고 있었던 것 같아요 (빌드 깨짐 에러)

스크린샷 2023-12-25 오후 6 36 52

수정해서 푸시 완료~~

Copy link
Collaborator

@wade3420 wade3420 left a comment

Choose a reason for hiding this comment

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

아... 이거 머지되고 제거 컨플릭 푸는게 나았으려나요 ㅜㅜ 제꺼 먼저 머지해버렸네요 죄송합니다

Copy link
Member

@sumi-0011 sumi-0011 left a comment

Choose a reason for hiding this comment

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

미션 완료 페이지를 mission > [id] > success 경로로 한 이유가 있을까용? 🤔🤔
미션 완료 페이지는 id와 관련없이 같은 페이지가 보여지는것이라고 생각했는데, 그렇다면 mission > success 경로여도 되지 않을까 생각했어요!

Comment on lines 3 to 8
MISSION: {
NEW: '/mission/new',
RECORD: (id: string) => `/mission/${id}/stopwatch`,
STOP_WATCH: (id: string) => `/mission/${id}/stopwatch`,
SUCCESS: (id: string) => `/mission/${id}/success`,
},
Copy link
Member

Choose a reason for hiding this comment

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

오 이런 방식이 있군요 👍👍👍👍

@woobottle
Copy link
Collaborator Author

미션 완료 페이지를 mission > [id] > success 경로로 한 이유가 있을까용? 🤔🤔
미션 완료 페이지는 id와 관련없이 같은 페이지가 보여지는것이라고 생각했는데, 그렇다면 mission > success 경로여도 되지 않을까 생각했어요!

아래와 같이 생각했었어요

  1. mission/[id]/record 에서 mission/[id]/success로 가는게 자연스럽지 않을까 생각했던 것 같아요
    1-1. 근데 미션 인증의 성공이라 몬가몬가 한것 같아요
  2. 흠 이거 수미님 의견대로 mission/success로 바꿔서 가고
  3. tutorial쪽에서는 type으로 쿼리스트링 추가해서 분기처리하는게 좋을것 같다는 생각이 드네용

@sumi-0011
Copy link
Member

미션 완료 페이지를 mission > [id] > success 경로로 한 이유가 있을까용? 🤔🤔
미션 완료 페이지는 id와 관련없이 같은 페이지가 보여지는것이라고 생각했는데, 그렇다면 mission > success 경로여도 되지 않을까 생각했어요!

아래와 같이 생각했었어요

  1. mission/[id]/record 에서 mission/[id]/success로 가는게 자연스럽지 않을까 생각했던 것 같아요
    1-1. 근데 미션 인증의 성공이라 몬가몬가 한것 같아요
  2. 흠 이거 수미님 의견대로 mission/success로 바꿔서 가고
  3. tutorial쪽에서는 type으로 쿼리스트링 추가해서 분기처리하는게 좋을것 같다는 생각이 드네용

의견 반영 감사합니다!
저도 1-1 정도의 생각이였어서, 반영 전/후 크게 상관없긴 하지만 그렇게 하신 이유가 궁금했었어요!

@woobottle woobottle merged commit 059eeea into develop Dec 26, 2023
3 checks passed
@woobottle woobottle deleted the task/editAppFolderStructure branch December 26, 2023 06:24
Copy link
Contributor

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

Successfully merging this pull request may close these issues.

[Task] app 하위 폴더 구조 변경
3 participants