Skip to content

Conversation

@elive7
Copy link
Contributor

@elive7 elive7 commented Mar 26, 2025

🍃 Pull Requests

⛳️ 작업한 브랜치

👷 작업한 내용

  • 클라측에서, 이벤트 배너가 끝나고 나면 기본 배너가 보일 수 있도록 하는 자동화 기능을 요청하여 그에 맞게 코드를 수정했습니다. (1번 이벤트 배너 기간 동안은 2번 배너가 보이지 않았다가, 1번 이벤트 배너의 기간이 끝나면 자동으로 2번 배너가 노출되도록 자동화 요청)
    image
    image
  • 배너의 노출 시작일인 startDate를 추가하고, 배너 생성시 필요에 따라 기존 배너의 startDate를 새롭게 생성되는 배너의 dueDate로 설정했습니다.
    • 배너 목록 조회시 현재 시간이 startDate와 dueDate 사이인 이벤트만 조회됩니다.
    • 또한 숨길 배너의 id가 요청에 주어지지 않으면 yml 속 값을 참조하여 기본 배너가 숨겨지게 됩니다.

🚨 참고 사항

  • yml이 업데이트 되었습니다.
  • 더 좋은 함수나 변수 네이밍, 로직이 있다면 추천해주세요!! 👊

elive7 added 4 commits March 26, 2025 21:25
- 배너에 시작 날짜를 추가하여, 이벤트 목록 조회시 startDate와 dueDate 사이에 있는 배너만 조회되도록 수정하였습니다.
- 앞에서 정의한 startDate를 이용하여, 이벤트 배너의 기간이 끝나기 전까지는 기본 배너(또는 특정 배너)가 보이지 않도록 설정했습니다.이벤트 배너의 기간이 끝나면 기본 배너가 다시 노출되게 됩니다.
- BannerCreateRequest의 형태가 바뀐 관계로 test도 수정해주었습니다.
- 또한 hideOthher가 true일 때 기존 배너의 시작일이 바뀌는지도 테스트를 통해 확인해보았습니다.
@elive7 elive7 requested a review from jinkonu March 26, 2025 13:50
@elive7 elive7 self-assigned this Mar 26, 2025
@elive7 elive7 added feat 🔥 new feature 🧠 test add or fix test labels Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat 🔥 new feature 🧠 test add or fix test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants