Skip to content

[FEAT] 파일 및 폴더 이름 중복 체크 #176

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
Merged
merged 44 commits into from
Nov 22, 2024
Merged

[FEAT] 파일 및 폴더 이름 중복 체크 #176

merged 44 commits into from
Nov 22, 2024

Conversation

Chan531
Copy link
Contributor

@Chan531 Chan531 commented Nov 19, 2024

✨ Related Issue

📝 기능 구현 명세

🐥 추가적인 언급 사항

  • 테스트 완료했습니다!
  • 중복 로직 관련 논의 사항은 슬랙 참고해주세요!
  • 파일 업로드는 원래 아이디를 리스트로 만들어 응답했는데 필요가 없을 거 같아 클라와 논의 후 없앴습니다.

Chan531 added 30 commits November 19, 2024 13:20
@Chan531
refactor(DocumentController): url 변경
2ece31a 
teamId와 foderId가 url에 드러나도록 수정
@Chan531
chore(DocumentController): created URI 수정
543fa14
@Chan531
chore(DocumentController): folderId 타입 수정
2450ed9
@Chan531
feat(DocumentService): 폴더 검증 로직 추가
57f419e 
폴더의 팀 id 검증
@Chan531
feat(DocumentService): 파일 이름 중복 검증 로직 추가
5e3865a
@Chan531
feat(ErrorCode): 중복된 파일 이름 관련 에러 코드 추가
44c0260
@Chan531
feat(DocumentService): 파일 이름 중복 여부 검증
1d57b96 
중복된 파일 이름이 존재한다면 예외 발생
@Chan531
chore(DocumentService): 미사용 메소드 제거
0e0af79
@Chan531
feat(Folder): 팀 아이디 검증 기능 추가
1880878
@Chan531
refactor(DocumentService): 팀 아이디 검증 책임 변경
35acb3c
@Chan531
refactor(DocumentController): folderId 파라미터 추가
2241f3f
@Chan531
refactor(DocumentService): folderId 파라미터 적용
dec324a
@Chan531
refactor(DocumentService): 파일 아이디 리스트 생성 로직 메소드 분리
1601f99
@Chan531
style(DocumentService): 메소드 선언 순서 변경
6a57b01
@Chan531
refactor(DocumentsCreateRequest): 폴더 id 요청 제거
739d99d
@Chan531
refactor(FolderController): url 변경
728eccc 
teamId와 folderId가 url에 드러나도록 수정
@Chan531
style(DocumentController): 미사용 import문 제거
83d625c
@Chan531
refactor(FolderController): teamId 수정
1d93bdd
@Chan531
refactor(FolderController): 폴더 id 파라미터 추가
4d56713
@Chan531
refactor(FolderService): 폴더 찾기 시, 팀 아이디 검증 로직 추가
37c005a
@Chan531
refactor(FolderController): url에서 path 제거
9338065
@Chan531
refactor(FolderService): path 전달 방식 수정
28400df
@Chan531
refactor(FolderCreateRequest): 부모 폴더 id 제거
b9561b4
@Chan531
refactor(FolderService): 폴더 path 추출 로직 메소드 분리
f521403
@Chan531
feat(Folder): 자식 폴더 path 반환
aa0600b
@Chan531
feat(ErrorCode): 중복된 폴더 이름 관련 에러 코드 추가
f50af72
@Chan531
feat(FolderService): 폴더 이름 중복 여부 검증
1056a11 
중복된 폴더 이름이 존재한다면 예외 발생
@Chan531
feat(ErrorHandler): FolderException 추가
af6c18d
@Chan531
chore(ErrorHandler): 메소드명 수정
8e7e44c 
대문자 시작 메소드 소문자로 수정
@Chan531
fix(FolderService): 자식 주소를 반환하도록 수정
6bab42d
@Chan531 Chan531 requested a review from paragon0107 November 19, 2024 08:02
@Chan531 Chan531 self-assigned this Nov 19, 2024
paragon0107
paragon0107 reviewed Nov 19, 2024
View reviewed changes
paragon0107
paragon0107 reviewed Nov 19, 2024
View reviewed changes
Copy link
Contributor

@paragon0107 paragon0107 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

고생하셨습니다!.
추가적으로 현재 데이터베이스 상에서는 따로 유니크 작업이 안되어 있는걸로 아는데 동시성 이슈로 인해 유니크 제약 조건을 걸어야 할 것 같습니다. 다른 기능들도 마찬가지이고 현재 폴더및 파일 기능에서 또한, Path + 폴더 네임으로 복합 유니크 제약 조건을 걸어놔야 할 것 같습니다!

paragon0107
paragon0107 reviewed Nov 20, 2024
View reviewed changes
paragon0107
paragon0107 approved these changes Nov 20, 2024
View reviewed changes
Copy link
Contributor

@paragon0107 paragon0107 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

이따 메서드에 대한 네이밍 정도만 한번 이야기해보는데 네이밍 관련 부분이니까 일단 승인해놓겠습니다!
수고하셨습니다~

@Chan531 Chan531 merged commit 261a6a4 into develop Nov 22, 2024
1 check passed
@Chan531 Chan531 deleted the feat/#175-file-folder branch November 22, 2024 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@paragon0107 paragon0107 paragon0107 approved these changes

Assignees

@Chan531 Chan531

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FEAT] 파일 및 폴더 이름 중복 체크

2 participants

@Chan531 @paragon0107