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 : 공지, 홍보, 부스 다중 이미지, 썸네일 추가 #12

Merged
merged 9 commits into from
Sep 30, 2023

Conversation

pump9918
Copy link
Collaborator

##공지, 홍보 올렸습니다

  • 병현이랑 합의 후 이미지 필드를 개별 모델 클래스 아래에 넣는 제 방식을 쓰기로 했습니다.
  • 중복되는 썸네일 코드 삭제했습니다
  1. models.py
  • 모델에서 이미지 경로 다듬기 위해 Notification_image와 Promotion_image가 수정됐습니다.
  1. paginations.py
  • 페이지네이션 10개씩 넣었습니다
  1. admin.py
  • admin으로만 테스트했는데 한 게시물에 이미지 5개까지 넣을 수 있게 했습니다.

4 serializers.py

  • thumbnail을 가상필드로 사용해서 한 게시물에 올린 이미지들 중 첫번째 이미지를 thumbnail에도 자동 저장되게 했습니다
  1. views.py
  • Mixin 사용했고, Promotion만 첫번째 이미지 url 불러오는걸로 설정 따로 했습니다
  • 필터 적용해서 type으로 구분되게 해놨습니다
  • List와 detail 시리얼라이저 분류하고 date 양식에 맞게 필터링했습니다
  1. url.py
  • url 설정했습니다

api 테스트를 위한 swagger를 추가했습니다
project 폴더 충돌 해결
feat: 공지, 홍보 페이지네이션 추가
feat : 공지, 홍보 필터링 적용
fix : notification에 썸네일 주가
feat : 공지, 홍보 date 추가
feat : 부스 썸네일, 다중이미지 추가
fix : 중복되는 썸네일 코드 삭제
Copy link
Collaborator

@JongbeomLee623 JongbeomLee623 left a comment

Choose a reason for hiding this comment

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

Good

Copy link
Collaborator

@lsjmc0224 lsjmc0224 left a comment

Choose a reason for hiding this comment

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

LGTM

@lsjmc0224 lsjmc0224 added the safe Safe to deploy label Sep 30, 2023
@lsjmc0224 lsjmc0224 merged commit e77d219 into LikeLion-at-DGU:main Sep 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe Safe to deploy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants