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

[FIX] 정의하기 결과 화면 QA 사항 반영 및 카카오 공유 기능 추가 #41

Merged
merged 12 commits into from
May 18, 2024

Conversation

AAminha
Copy link
Member

@AAminha AAminha commented May 17, 2024

1️⃣ 요약

2️⃣ 변경사항

기존 코드에 영향을 미치는 변경사항

  • 전체적인 디자인 코드가 수정되었습니다.
  • 카카오 공유 기능으로 env가 업데이트 됩니다. 추후 노션에 반영하겠습니다.
  • 로딩 화면 동작 로직을 수정했습니다.
  • router를 수정했습니다.
    • /test/define 뒤에 다른 path가 없거나 유효하지 않은 path는 전부 /test/define/1로 가도록 수정했습니다.

기존 코드에 영향을 미치지 않는 변경사항

버그 수정을 위한 변경사항

3️⃣ 참고사항

4️⃣ 추후 작업

5️⃣ 리뷰 요구사항

@AAminha AAminha added the 🐞 Bug 버그 이슈 label May 17, 2024
@AAminha AAminha self-assigned this May 17, 2024
@AAminha AAminha marked this pull request as ready for review May 17, 2024 17:45
@AAminha AAminha requested a review from mungjin01 as a code owner May 17, 2024 17:45
@AAminha AAminha merged commit 730c40a into develop May 18, 2024
@AAminha AAminha deleted the fix/#38/define-test branch May 18, 2024 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 Bug 버그 이슈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 정의하기 결과 화면 QA 사항 반영 및 카카오 공유 기능 추가
2 participants