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

feat: 페스티벌 정적 데이터 관련 API 구현 #29

Merged
merged 8 commits into from
Aug 9, 2024

Conversation

punchdrunkard
Copy link
Member

@punchdrunkard punchdrunkard commented Aug 7, 2024

요약

온보딩 등에 사용되는 페스티벌 관련 정적 데이터 컨트롤러 구현

작업 내용

기타 (논의하고 싶은 부분)

  • 고정 데이터주는 컨트롤러는 따로 나눠야 할 지 고민입니다! (Festival 뿐만 아니라 다른 엔티티랑도 엮여 있습니다..)

image

  • 단순히 고정 데이터를 주는 기능에 (화면에 뿌리기 위한 카테고리를 조회하기) 테스트를 작성하는게 묘하게 비효율적이라고 느껴졌습니다... 단순히 고정 데이터 전체를 반환하는 api 의 경우 어떤식으로 처리해야할까요?

타 직군 전달 사항

close #28

@punchdrunkard punchdrunkard added the type: feat 새로운 기능 구현 label Aug 7, 2024
@punchdrunkard punchdrunkard requested a review from yueunfive August 7, 2024 12:56
@punchdrunkard punchdrunkard self-assigned this Aug 7, 2024
Copy link

github-actions bot commented Aug 7, 2024

Unit Test Results

12 tests   12 ✔️  2s ⏱️
  8 suites    0 💤
  8 files      0

Results for commit aa457d8.

@punchdrunkard punchdrunkard requested a review from hadevyi August 7, 2024 13:14
Copy link
Contributor

@yueunfive yueunfive left a comment

Choose a reason for hiding this comment

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

컨트롤러 분리�는 다은님 편한 대로 진행해주시면 될 것 같습니다
테스트 관련해서는 아는 바가 없다보니 생산적인 피드백을 드리기가 어려울 것 같네요..😅

@punchdrunkard punchdrunkard merged commit 026f9d8 into main Aug 9, 2024
3 checks passed
@punchdrunkard punchdrunkard deleted the feat/#28 branch August 9, 2024 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feat 새로운 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 온보딩 정적 데이터 조회 api 구현
2 participants