-
Notifications
You must be signed in to change notification settings - Fork 0
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
7월 3일 flow 수정 작업 #209
Merged
Merged
7월 3일 flow 수정 작업 #209
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
coummunityId -> id
kimyouknow
added a commit
that referenced
this pull request
Jul 3, 2023
* Feat: 에러바운더리 디자인 수정 (#183) * 마이페이지 관련 수정 (버튼 동작 및 ui 수정) (#185) * feat: 마이페이지에서 행성 만들기 버튼 동작 수정 * refactor: icon에 fill 제거 * refactor: 주민증없는 경우 ui 수정 * Fix/comment 3 (#192) * Fix: response type 수정 * Fix: comment error 수정 #159 * 404 페이지 (#194) * feat: error-ufo * feat: not-found page * Community route (#195) * feat: 행성 만들기 routing #170 * feat: #172 행성 id 기본값, communityTitle 제거 * refactor: switchCommunity 타입변경 반영 * style: cspell 추가 * feat: 마이페이지 행성 분기 로직 * feat: 행성 목록 user id * fix: coverImageUrl type null 처리 * feat: 행성 로고 기본 이미지 적용 * feat: 행성 기본 이미지 * feat: 행성 꾸미기 페이지 * feat: community -> planet 경로 수정 * feat: community -> planet 경로 수정 * feat: 행성 꾸미기 후 라우팅 * feat: 행성 꾸미기 설명 추가 * Add routing logic (#196) * Feat: Add routing logic when planet is created * Fix: Fix dev command * Feat: 행성 홈에서 주민증을 만드는 버튼 routing 로직 추가 * Feat: 주민증 생성 완료한 후 routing 로직 추가 * Feat: 행성 관리에서 수정 버튼 routing 로직 추가 --------- Co-authored-by: hyehyeon-moon <[email protected]> * Feat: 서비스 전체 에러페이지 (#197) * Feat/husky pre push (#201) * refactor: CommentInput 스토리 Props에러 * feat: pre-push build * Issue#189 (#206) * feat: gear icon * feat: 행성 관리 페이지로 이동 * refactor: AdminCommunityEditPage 서버 컴포넌트로 변경 * refactor: 행성 꾸미기 기본값 누락 * 7월 3일 flow 수정 작업 (#209) * refactor: getInvitationCodeIsValid(public -> private) * refactor: TemplateWrapper h-screen 높이 제거 * refactor: InvitationCodeValidationModel 스키마 수정 coummunityId -> id * refactor: middleware private api 관련 이슈 주석 * feat: root 경로 route로 처리 * refactor : UserInfoResponse 타입 수정 (dto) * refactor: 주민증 수정 mock profileImageUrl제거 * feat: mypage home root로 라우팅 * refactor: my-page:id -> my-page:communityId * feat: root url 변경 * refactor: CharacterCompleteStep에 useGetUserInfo타입 수정 --------- Co-authored-by: darae07 <[email protected]> --------- Co-authored-by: 강지영 <[email protected]> Co-authored-by: darae lee <[email protected]> Co-authored-by: 문혜현 <[email protected]> Co-authored-by: hyehyeon-moon <[email protected]> Co-authored-by: darae07 <[email protected]>
This was referenced Jul 4, 2023
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.
middleware 관련 중요 이슈
: middleware에서
next/headers
사용하면 에러 발생 -> NextRequest로 변경 ([Next 13.4] Middleware acts like a client component vercel/next.js#49246): public api 사용하면
axios http adapter
관련 에러 발생(Axios doesn't work on edge runtime, Adapter 'http' is not available in the build axios/axios#5523)임시 방편으로 next13의 route 사용
flow 정리
[Client] 초대코드로 접근 → 로그인 → 초대코드 페이지로 이동 #215
[Design] 초대 페이지 UI 작업 #217
이미 가입된 행성일 때 에러처리 메시지 보여주기
이미 가입된 행성의 경우, BE에서 4XX 대의 에러코드로 넘겨주시지 않고 다른 코드를 정의해서 넘겨주시기로 하셨어요!
그 이유는 "공유된 링크"를 이용해 접속하는 사용자가 많을 것으로 예상했기 때문이에요. 그래서 에러 메시지를 보여주기 보다는 flow를 타서 메인으로 유도하는 것을 생각했습니다.