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] 10/11 요구사항 반영 #52

Merged
merged 6 commits into from
Oct 12, 2024
Merged

[Fix] 10/11 요구사항 반영 #52

merged 6 commits into from
Oct 12, 2024

Conversation

david-parkk
Copy link
Member

@david-parkk david-parkk commented Oct 12, 2024

✏️ 작업 개요

  • 10/11 주간 회의의 요구사항을 반영합니다.

⛳ 작업 분류

  • 일부 API request ,response 변경
  • 데이터 Integer -> Long
  • quest entity 추가

🔨 작업 상세 내용

  1. 케스트 인덱스 17에서 06
  2. POST /quests, GET /quests progress 필드 추가
  3. GET /character 치장정보 추가
  4. 기존 quest -> questUser (유저가 소유한 퀘스트 정보) 로 변경되고 quest(퀘스트 전역 정보) 엔티티가 추가되었습니다.

💡 생각해볼 문제

  • Enum으로 퀘스트를 관리하고자 했지만 상점아이템과 같은 이유로 db로 분리하였습니다 ([Feat] 상점 엔티티 설계 #47 )
  • 현재 캐릭터 도메인에서 인벤토리 엔티티를 사용한다는 점에서 도메인분리가 되지 않기 시작합니다
  • mysql table convention은 단수를 사용한다고 하네요 (하하 왜 복수로 알고 있었지)

+) 밤에 클라와 이야기하기로 했는데 여기서 더 변경될 수도 있습니다

@david-parkk david-parkk requested a review from kamothi October 12, 2024 05:21
@david-parkk david-parkk self-assigned this Oct 12, 2024
Copy link

Test Results

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

Results for commit d9e13db.

@david-parkk david-parkk added the bug Something isn't working label Oct 12, 2024
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.

전체적으로 큰틀이 바뀐 것이 아닌 요구사항에 맞게 조건들이 바뀐거 같습니다. 수고하셨습니다.

현재 캐릭터 도메인에서 인벤토리 엔티티를 사용한다는 점에서 도메인분리가 되지 않기 시작합니다

이게 어떤 의미인가요

@david-parkk
Copy link
Member Author

전체적으로 큰틀이 바뀐 것이 아닌 요구사항에 맞게 조건들이 바뀐거 같습니다. 수고하셨습니다.
이게 어떤 의미인가요
캐릭터에서 인벤토리를 호출하는 상황을 말한거 였어요
감수해야할 부분 같네요

@david-parkk david-parkk merged commit 1629546 into main Oct 12, 2024
3 checks passed
@david-parkk
Copy link
Member Author

/쿠타버스 배포

@kamothi
Copy link
Member

kamothi commented Oct 12, 2024

🌎 배포하였습니다.

@kamothi
Copy link
Member

kamothi commented Oct 26, 2024

/쿠타버스 배포

@kamothi
Copy link
Member

kamothi commented Oct 26, 2024

🌎 배포하였습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 📡 자동 배포
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants