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

[FIX] 기능 명세서 수정사항 반영 - 공지사항 #30

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

eeddiinn
Copy link
Contributor

@eeddiinn eeddiinn commented Jul 13, 2024

📌 관련 이슈

closed #27

📝 작업 내용

  • NO.12 세부 공지사항
    • response dto 값들 변경 -> category, contentSummary, memberId, noticeLike 삭제 및 createdAt 추가
  • NO.13 공지 좋아요, NO.14 공지 좋아요 취소
    • 좋아요 이미 눌렀을 때 에러 메시지 추가
  • NO.15 공지 저장, NO.16 공지 저장 취소
    • 저장 이미 눌렀을 때 에러 메시지 추가
    • 저장했을 때 저장 수 1 증가, 저장 취소 시 저장 수 1 감소
  • NO.17 저장한 공지들 보기
    • 일시(startTime, endTime)대신 공지 등록 날짜, 이미지 추가
    • 이미지는 여러 개여도 1개만 뜨도록 하고, 없을 시 null
    • 최근 저장 누른 순서대로 정렬
  • NO.18 공지 조회수 증가
  • NO.19 공지사항 등록
    • noticeSave 초기 값 0 되도록 수정
    • 공지사항 등록 시 이미지를 여러 개 올릴 수 있도록 하고, 이미지가 만약 없을 시 null 뜨도록 함
    • validation 추가 : 제목 1000자, 사진 최대 5개

💬리뷰 요구사항

  • Postman으로 여러 경우에서 테스트 해봤는데 잘 됐습니다 !! 그래도 혹시 빠진 부분있는지 확인 해주세용

@eeddiinn eeddiinn added 🐞Fix 코드 수정, 버그/오류 해결 🐰예진 🔥Pull Request This will not be worked on labels Jul 13, 2024
@eeddiinn eeddiinn requested a review from softmoca July 13, 2024 21:05
@eeddiinn eeddiinn self-assigned this Jul 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞Fix 코드 수정, 버그/오류 해결 🐰예진 🔥Pull Request This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 기능 명세서 수정사항 반영 - 공지사항
1 participant