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

[MS-202] Feat: Create Banner API #108

Merged
merged 2 commits into from
Sep 28, 2024
Merged

[MS-202] Feat: Create Banner API #108

merged 2 commits into from
Sep 28, 2024

Conversation

tjdgns8439
Copy link
Member

요약 🎀

  • Banner 생성 및 조회 API 작성했습니다.

상세 내용 🌈

조회 API
스크린샷 2024-09-26 오전 1 35 40

생성 API
스크린샷 2024-09-26 오전 1 36 04

  • 생성은 imageUrl과 targetUrl만 주면 됩니다!
  • 귀찮겠지만 배너의 첫 Active 상태는 false입니다! 배너 잘못 올린 게 그대로 노출되면 안되기 때문에 한번 더 검증하라고 그렇게 만들었어요!
  • 생성 후 DB에서 수동으로 바꿔주게끔 되어있는데 우리는 따로 앱 자체에 관리자 역할이 구분되어 있지 않기 때문에 앱이 아닌 다른 방식으로 배너를 활성화시키게끔 유도하기 위해서 이런 방식을 채택했습니다!
  • 지금은 DB에서 수동으로 바꿔주어야 하지만 조금 고민해보고 PM 에게 친화적인 방법을 고려해보겠습니다! 일주일에 한 번 바꿀까 말까일 것 같기 때문에 그냥 이대로 갈 것 같기도 하고.. 하지만 보안상 좋지 않으니 고민해보겠습니다!

질문 및 이외 사항 🚩

리뷰어에게 ✨

@tjdgns8439 tjdgns8439 self-assigned this Sep 25, 2024
Copy link
Contributor

@YooJisu826 YooJisu826 left a comment

Choose a reason for hiding this comment

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

지금 방식은 imgUrl이랑 linkUrl밖에 없어서 클릭했을 때 무조건 웹으로 이동하는 형식으로 가야겠네요?!
앱 내에서 특정 화면으로 이동하는 딥링크는 쓰는 일이 없으려나요? 만약에 쓴다면 또 프론트와 협의해야할 듯 합니다
지금은 이대로도 문제 없어보입니다 고생하셨씁니다 LGTM

private final GetBannerService getBannerService;

@Operation(summary = "배너 리스트 조회",
description = "actibe상태의 배너 리스트를 조회합니다."
Copy link
Contributor

Choose a reason for hiding this comment

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

오타있습니다!

@tjdgns8439 tjdgns8439 merged commit fa48f75 into develop Sep 28, 2024
1 check passed
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