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

케이크를 생성하고 수정한다. #41

Merged
merged 54 commits into from
Jul 12, 2024
Merged

케이크를 생성하고 수정한다. #41

merged 54 commits into from
Jul 12, 2024

Conversation

ieun32
Copy link
Contributor

@ieun32 ieun32 commented Jul 12, 2024

PR 유형 선택

  • ✨ 기능 추가
  • 🐛 버그 수정
  • 🖼️ UI 변경
  • 🫧 코드 리팩토링
  • ✅ 테스트 코드 추가
  • 📦 기타 (문서, CI/CD, 되돌리기, 포맷팅 …)

작업 내용

  • 로그인 페이지 (루트 경로) 접속시 토큰 유효성을 검증하고 유효하다면 케이크 페이지로 이동합니다.
  • 본인 케이크 페이지 접속 후 시트, 크림 정보가 존재하지 않는다면 자동으로 케이크 만들기 페이지로 이동합니다.
  • 케이크 만들기 페이지에서 케이크의 시트, 크림 정보를 선택할 수 있고, 데이터 베이스에 정보를 저장합니다.
  • 타인 케이크 페이지 접속 후, 케이크 꾸며주기 버튼을 누르면 편지 작성 페이지로 이동합니다.
  • 타인 케이크 페이지 접속 후, 내 케이크로 이동하기 버튼을 누르면 토큰 유효성 검증 후 유효하다면 이동, 그렇지 않다면 로그인이 필요하다는 alert 를 띄웁니다.

관련 문서

리뷰 요구사항

  • 불필요하거나 컨벤션에 맞지 않은 코드가 있다면 말씀해주세요!
  • 로직에 문제가 있다면 말씀해주세요!

ieun32 added 30 commits July 10, 2024 10:43
@ieun32 ieun32 requested a review from kyr4601 July 12, 2024 11:57
@ieun32 ieun32 self-assigned this Jul 12, 2024
@ieun32 ieun32 added 🔧 Refator 코드 리팩토링 ✨ Feature 기능 추가 labels Jul 12, 2024
Copy link
Contributor

@kyr4601 kyr4601 left a comment

Choose a reason for hiding this comment

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

수고하셨숩니당!!!
어떤 부분은 axios 사용 시 then이 쓰이고 어떤 부분은 try catch가 쓰여서 이 부분 같이 얘기해서 통일하면 더 좋을 것 같습니당 👍👍

@kyr4601 kyr4601 merged commit 5076edc into develop Jul 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 추가 🔧 Refator 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants