Skip to content

feat: 공개 API에 이벤트별 필터링 추가 및 최신 이벤트 기본값 적용#35

Open
JaeHyuckSa wants to merge 2 commits intomainfrom
feat/event-filter-public-api
Open

feat: 공개 API에 이벤트별 필터링 추가 및 최신 이벤트 기본값 적용#35
JaeHyuckSa wants to merge 2 commits intomainfrom
feat/event-filter-public-api

Conversation

@JaeHyuckSa
Copy link
Copy Markdown
Member

후원사/발표 API에 ?event= 쿼리 파라미터로 이벤트를 구분하고, 미지정 시 최신 이벤트 데이터를 기본 반환하도록 변경

@JaeHyuckSa JaeHyuckSa self-assigned this Mar 23, 2026
Signed-off-by: JaeHyuckSa <jaehyuck.sa.dev@gmail.com>
from event.filters import EventFilterMixin


class PresentationFilterSet(EventFilterMixin):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

별도 Filter로 구분한 부분 좋은 것 같습니다!

organization = Organization.objects.create(name="Test Organization")
event_1: Event = Event.objects.create(organization=organization, name="Test Event 1")
event_2: Event = Event.objects.create(organization=organization, name="Test Event 2")
event_1: Event = Event.objects.create(
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

현재 해당하는 사항은 아니지만, 3일동안 열리는 행사가 아닌 경우에 대비해서 (파이콘이 2일만 열리거나, Event 객체가 타 행사를 나타낼 경우) event_end_at column도 추후 추가가 필요할 것 같아요(요 부분은 다른 PR에서 수정해도 좋을 것 같습니다)!

Copy link
Copy Markdown
Contributor

@earthyoung earthyoung left a comment

Choose a reason for hiding this comment

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

LGTM 입니다. 고생하셨습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants