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.

Sign up for GitHub

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

Jump to bottom

테마별 향수 추천 API 추가 #320

Merged
merged 2 commits into from
Dec 23, 2023
Merged

테마별 향수 추천 API 추가 #320

merged 2 commits into from
Dec 23, 2023

Conversation

oliviarla
Copy link
Contributor

@oliviarla oliviarla commented Dec 23, 2023 •
edited
Loading

What is this PR? 👀

  • Issue ticket number:

Changes ✏️

  • 테마 별로 향수 4개를 추천해주는 부분에 대한 API를 추가했습니다.
  • 향수개수는 4개로 항상 정해져있을 것 같아 중간 테이블을 만들지 않고 ,로 구분된 String을 담는 형태로 구현해보았습니다. (뭔가 개발은 편한데 마음은 안편하네요....😓)

Test checklist 🧪

  • [ ]

@oliviarla oliviarla added the 30분 후 merge! 다음 작업과 연관이 있어 30분 후 리뷰가 없을 경우 merge하는 PR입니다. label Dec 23, 2023
@big-cir
Copy link
Contributor

big-cir commented Dec 23, 2023

고생하셨습니다!
DB에는 4개의 향수가 수정/삭제 없이 항상 고정인가요!?

@oliviarla
Copy link
Contributor Author

@Big-Cir97
테마별 향수 추천이라는 내용 자체가 계절별, 분기별로 갱신되는데 최근것만 가져오는 (공지사항같은) 데이터라서, 향수 삭제가 발생하면 글 자체를 바꿔야 할 것 같긴 해요.! 물론 향수 삭제도 빈번히 발생할 것 같진 않습니다..!

@big-cir
Copy link
Contributor

big-cir commented Dec 23, 2023

@oliviarla
감사합니다!

dygma0
dygma0 approved these changes Dec 23, 2023
View reviewed changes
Copy link
Contributor

@dygma0 dygma0 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 :)

@oliviarla oliviarla merged commit 006011e into develop Dec 23, 2023
2 checks passed
@oliviarla oliviarla deleted the perfume branch December 23, 2023 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dygma0 dygma0 dygma0 approved these changes

@big-cir big-cir Awaiting requested review from big-cir

Assignees
No one assigned
Labels
30분 후 merge! 다음 작업과 연관이 있어 30분 후 리뷰가 없을 경우 merge하는 PR입니다.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@oliviarla @big-cir @dygma0