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.

Sign up for GitHub

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

Jump to bottom

[개선] 공지사항 파일 등록 #109

Merged
merged 15 commits into from
Jul 27, 2024
Merged

[개선] 공지사항 파일 등록 #109

merged 15 commits into from
Jul 27, 2024

Conversation

jyj1289
Copy link
Member

@jyj1289 jyj1289 commented Jul 21, 2024 •
edited
Loading

🎫 관련 이슈

close #106


📄 개요

공지사항에 파일 업로드 기능을 만들었습니다.


🔨 작업 내용

  • 파일 업로드 API를 만들었습니다.
  • tbl_notice 테이블에 fileUrl 컬럼을 추가했습니다.
  • 생성, 수정, 조회에 fileUrl을 넣을 수 있도록 수정했습니다.

🏁 확인 사항

  • 테스트를 완료했나요?
  • API 문서를 작성했나요?
  • 코드 컨벤션을 준수했나요?
  • 불필요한 로그, 주석, import 등을 삭제했나요?

🙋🏻 덧붙일 말

@jyj1289
feat(#107): fileUrl 컬럼 추가
83c5ae7 
- 공지사항에 파일 Url을 등록할 수 있도록 컬럼을 추가했어요.
@jyj1289
feat(#107): DTO 수정
d1256af 
- 공지사항에 파일 Url을 등록할 수 있도록 DTO fileUrl을 추가했어요.
@jyj1289
perf(#107): CreateNoticeUseCase수정
469ed75 
- 공지사항에 파일 Url을 등록할 수 있도록 UseCase를 수정했어요.
@jyj1289
feat(#107): folderConstant 추가
2f75ff5 
- NOTICE_FILE을 추가했어요.
@jyj1289
perf(#107): update 메서드 수정
68adb1e 
- fileUrl을 받도록 변경했어요.
@jyj1289
feat(#107): file 업로드 메서드 추가
e67aa43 
- 메서드를 추가했어요.
@jyj1289
fix(#106): NotBlank 삭제
ff9ad9e 
- fileUrl을 안넣을 수도 있기 때문에 NotBlank를 삭제했어요.
@jyj1289
perf(#106): NoticeResponse 수정
6f2fc16 
- FileUrl이 나오도록 수정했어요.
@jyj1289
perf(#106): UpdateNoticeUseCase 수정
999b5cf 
- FileUrl도 받도록 수정했어요.
@jyj1289
feat(#106): UploadFileUseCase 추가
b2b15a2 
- 공지 파일을 만드는 유즈케이스를 추가했어요.
@jyj1289
test(#106): UploadFileUseCase 테스트
6503e7f 
- 파일 업로드 테스트 코드를 작성했어요.
@jyj1289
test(#106): UploadFile 테스트 코드 작성
e864a40 
- 파일 업로드 유즈케이스 및 컨트롤러 테스트 코드를 작성했어요.
- 원래 공지사항을 생성, 조회, 수정하는 테스트에 formUrl을 넣어 수정했어요.
- NoticeFixture에 formUrl을 넣도록 수정했어요.
@jyj1289
docs(#106): notice.adoc 추가
b82af99 
- formUrl과 관련된 내용을 문서에 추가했어요.
@jyj1289
fix(#106): 버킷 파일이름을 원본 파일이름으로 저장하도록 변경
3260df7 
- 유저의 uuid로 저장하면 하나의 파일밖에 저장하지 못한다는 점 때문에 파일이름으로 변경했어요.
@jyj1289 jyj1289 requested a review from a team as a code owner July 21, 2024 08:02
@jyj1289 jyj1289 linked an issue Jul 21, 2024 that may be closed by this pull request
[개선] 공지사항 파일 등록 #106
Closed
@github-actions GitHub Actions
Copy link

이렇게 리뷰해주세요.

🖐️ 규칙

  1. 왜 개선이 필요한지 충분히 설명해주세요.
  2. 답을 알려주기보다는 스스로 고민하고 개선 방법을 선택할 수 있게 해주세요.
  3. 코드를 깔끔하게 유지하고, 일관되게 구현하도록 안내해 주세요.
  4. 리뷰 과정이 숙제검사가 아닌 학습과정으로 느낄 수 있게 리뷰해 주세요.
  5. 리뷰를 위한 리뷰를 하지 마세요. 피드백 할 게 없으면 칭찬해 주세요.

🥬 우선순위

리뷰 중요도에 따라 다음과 같은 prefix를 붙여주세요.

예시 - P5) 쓸모없는 주석 삭제해주세요.

  • P1: 꼭 반영해주세요 (Request changes)
  • P2: 적극적으로 고려해주세요 (Request changes)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

@gimhanul
Copy link
Member

form은 원서의 도메인으로 이미 사용하고 있는데 form으로 파일 첨부 도메인 이름을 사용하면 헷갈리지 않을까요?

@junghongseop
fix(#106): 공지사항 파일을 버킷에 등록시 파일이름을 uuid로 수정
228654a 
- 파일이름으로 저장을 하였을 때, 동일한 파일이 등록되면 덮어쓰기가 되는 오류가 있어 uuid로 저장하도록 변경했어요.
@junghongseop junghongseop reopened this Jul 25, 2024
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jul 25, 2024

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@jyj1289
Copy link
Member Author

jyj1289 commented Jul 25, 2024

form은 원서의 도메인으로 이미 사용하고 있는데 form으로 파일 첨부 도메인 이름을 사용하면 헷갈리지 않을까요?

혹시 어디쪽인지 말씀해주실 수 있으실까요? NoticeFixture에 formUrl 추가했다는 커밋 메시지는 fileUrl을 잘못 적은겁니다 ㅠㅠ

@gimhanul
Copy link
Member

image
요거 보고 이야기한 거긴한데 아닌가욥?

@jyj1289
Copy link
Member Author

jyj1289 commented Jul 26, 2024 •
edited
Loading

image 요거 보고 이야기한 거긴한데 아닌가욥?

앗 그것도 오타입니다... 다음부터는 2번 더 확인하겠습니다...

@jyj1289 jyj1289 merged commit 1d73c86 into develop Jul 27, 2024
3 checks passed
@jyj1289 jyj1289 deleted the perf/#106 branch July 27, 2024 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[개선] 공지사항 파일 등록
3 participants
@jyj1289 @gimhanul @junghongseop