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] 공지사항 등록 API #9

Closed
wants to merge 18 commits into from
Closed

Conversation

eeddiinn
Copy link
Contributor

@eeddiinn eeddiinn commented Jul 8, 2024

📌 관련 이슈

closed #7

📝 작업 내용

  • 공지사항 등록을 할 수 있는 API를 구현했습니다.
    • request : 제목, 대상, 일시(시작 날짜, 끝 날짜), 등록할 이미지 리스트( 1~10개 사이 선택) , 내용
    • response : 공지사항의 idx값
  • accessToken이 아직은 구현이 안되어서 임시로 설정해서 구현하였고,
    user또한 DB에 테스트 유저 (idx 값 : 1 ) 1명을 만들어서 테스트 하였습니다.
    image
    image

💬리뷰 요구사항

  • (이미지 업로드하는 메서드가 S3에 있긴한데
    클라이언트한테 어떻게 이미지를 받을지 몰라서 일단은 클라이언트가 파일을 보내면 이미지 업로드하는 api를 호출해서
    url을 클라이언트가 얻으면, 이 url을 request로 받아 공지로 등록할 수 있도록 구현하였습니다.)
    -> 아 이 부분 생각해봤는데 파일을 보내면 업로드하는 api를 호출하는 것 자체가 불필요하다고 생각하여 그냥 request로 파일을 받으면 로직에서 s3에 업로드하여 url로 저장할 수 있도록 수정하겠습니다 !!
    • 수정 완료 !!!
    • 추후 바꿀 부분이 있으면 이미지 관련은 나중에 의논해서 바꾸면 될 것 같습니다 !!!
  • 그리고 id값이 Integer보다는 Long값이 더 낫다고해서 Long을 설정해두었습니다.
    • 현재 Integer로 충분한 경우라도, 미래에 데이터가 더 많이 증가할 것을 대비할 수 있어서 ?!
  • accessToken부분은 회원가입/로그인 구현이 완료되면 수정할 계획입니당 ☺️

@eeddiinn eeddiinn added ✨ Feature 새로운 기능 구현 🐰예진 📬API 🔥Pull Request This will not be worked on labels Jul 8, 2024
@eeddiinn eeddiinn requested a review from softmoca July 8, 2024 16:22
@eeddiinn eeddiinn self-assigned this Jul 8, 2024
@eeddiinn eeddiinn closed this Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📬API ✨ Feature 새로운 기능 구현 🐰예진 🔥Pull Request This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 공지사항 등록 API 구현
2 participants