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

퀘스트 API 추가 #39

Merged
merged 9 commits into from
Oct 1, 2024
Merged

퀘스트 API 추가 #39

merged 9 commits into from
Oct 1, 2024

Conversation

david-parkk
Copy link
Member

✏️ 작업 개요

⛳ 작업 분류

  • 퀘스트 API 추가
  • 테스트 작성

🔨 작업 상세 내용

  1. 퀘스트 API 2개를 개발하였습니다.
  2. test 환경은 h2의존성을 유지했습니다 (CI 때문에)

💡 생각해볼 문제

  • LocalDataTime은 분초 모두 저장한다는 점에서 query에 옵션을 추가해야합니다. 일일 퀘스트 갱신을 위한 created_at 필드를 LocalTime으로 변경하였습니다.
  • POST /quests에 body에 email을 넣을 수 있었지만 최종적으로 jwt가 되야한다는 관점에서 body와 분리하여 QueryParameter로 변경하였습니다.

@david-parkk david-parkk added the 기능구현 Good for newcomers label Oct 1, 2024
@david-parkk david-parkk self-assigned this Oct 1, 2024
@david-parkk david-parkk changed the title Feat/quest 퀘스트 API 추가 Oct 1, 2024
Copy link

github-actions bot commented Oct 1, 2024

Test Results

22 tests   22 ✅  2s ⏱️
 5 suites   0 💤
 5 files     0 ❌

Results for commit 4445895.

Copy link
Member

@kamothi kamothi left a comment

Choose a reason for hiding this comment

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

현재의 경우 퀘스트가 아직 정해져 있지 않기에 이정도면 충분한 것 같습니다. 추후 퀘스트 종류에 따라 엔티티를 수정해야할 것 같습니다. 예를 들어, 건덕 리듬게임 3판 승리

@kamothi kamothi merged commit 05bfa98 into main Oct 1, 2024
3 checks passed
@kamothi kamothi deleted the feat/quest branch October 1, 2024 08:41
@kamothi
Copy link
Member

kamothi commented Oct 1, 2024

/쿠타버스 배포

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능구현 Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants