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

광고 구좌 제작 및 csv 다운로드 api 버전 업 #823

Merged
merged 8 commits into from
Aug 27, 2024
Merged

Conversation

borimong
Copy link
Contributor

🚩 관련 이슈

📋 작업 내용

  • 기능 구현 중 csv api 업데이트 요청이 들어왔는데, chore 한 작업이라 하나의 pr 에서 작업했습니다.
  • 기존에는 embla-carousel-react 만 설치되어 있었는데, autoplay 플러그인을 설치하면서 embla-carousel 에 있는 옵션이 필요하여 추가 설치하였습니다.
  • 세부 기능 중 구현되지 않은 사항들이 몇 있습니다. 이후 pr 에서 보충하도록 하겠습니다.

📌 PR Point

  • 무슨 이유로 어떻게 코드를 변경했는지
  • 어떤 위험이나 우려가 발견되었는지
  • 어떤 부분에 리뷰어가 집중해야 하는지
  • 개발하면서 어떤 점이 궁금했는지

📸 스크린샷

@borimong borimong requested a review from R1mmm as a code owner August 27, 2024 11:06
Copy link

height bot commented Aug 27, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@borimong borimong merged commit dcf2326 into develop Aug 27, 2024
2 checks passed
@borimong borimong deleted the feat/#818 branch August 27, 2024 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

광고 구좌 ui
1 participant