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

[fix] : 빈 스케줄 등록과 관련한 에러들을 해결한다 #160

Merged
merged 5 commits into from
Jan 23, 2025
Merged

[fix] : 빈 스케줄 등록과 관련한 에러들을 해결한다 #160

merged 5 commits into from
Jan 23, 2025

Conversation

bbbang105
Copy link
Member

@bbbang105 bbbang105 commented Jan 22, 2025 •
edited
Loading

✅ PR 유형

어떤 변경 사항이 있었나요?

  • 새로운 기능 추가
  • 버그 수정
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 이벤트 생성자이지만, 빈 스케줄을 등록하는 경우를 고려하여 CREATOR_AND_PARTICIPANT 필드를 추가하여 구분하였습니다.
  • 참여 목록에 CREATOR 와 PARTICIPANT 둘 다 들어가는 방법도 고려하였지만, 변경할 코드가 너무 많아진다고 판단하여 위 방법을 택하였습니다.
  • 이벤트 수정 & 삭제 권한은 CREATOR 와 CREATOR_AND_PARTICIPANT 에게만 주어집니다.
  • 전체 스케줄 (날짜 & 요일) 조회 시, 빈 스케줄을 등록한 멤버는 제외되었던 문제를 해결하였습니다.
  • 개인 스케줄 (날짜 & 요일) 조회 시, 빈 스케줄을 등록한 멤버는 404 에러가 발생하였던 문제를 해결하였습니다.

✏️ 관련 이슈

본인이 작업한 내용이 어떤 Issue Number와 관련이 있는지만 작성해주세요

🎸 기타 사항 or 추가 코멘트

@bbbang105 bbbang105 added 🚨 fix 버그 수정 / 에러 해결 😵‍💫 sangho 상호 PR 🔄 refactor 코드 리팩토링 labels Jan 22, 2025
@bbbang105 bbbang105 self-assigned this Jan 22, 2025
@bbbang105 bbbang105 added the 🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정 label Jan 22, 2025
@bbbang105 bbbang105 merged commit 02b766f into develop Jan 23, 2025
2 checks passed
@bbbang105 bbbang105 deleted the feature/#157/empty branch January 23, 2025 00:50
bbbang105 added a commit that referenced this pull request Jan 23, 2025
@bbbang105
[release] : v1.4.3 (#161)
f03c5f4 
* [feat] : 이벤트 정기 삭제 시 S3에 있는 QR 이미지도 함께 삭제한다 (#150)

* [docs] : 리드미를 수정한다

* #146 [feat] : S3에서 파일을 삭제할 수 있다

* #146 [feat] : qr 디렉토리를 추가한다

* #146 [feat] : 이벤트 정기 삭제 시 S3에 있는 QR 이미지도 함께 삭제한다

* #146 [feat] : 이벤트 삭제 시 QR 이미지도 함께 삭제하는 로직을 추가한다

* #146 [chore] : 도커 이미지를 항상 재빌드하도록 변경한다

* [feat] : JwtFilter를 도입한다 (#152)

* #92 [feat] : 인증 및 비인증 엔드포인트를 명확히 분리한다

* #92 [feat] : 헬스체크를 위해 로컬 url을 추가한다

* #92 [chore] : 헬스체크를 제거한다

* #92 [chore] : Actuator 관련 의존성을 추가한다

* #92 [feat] : Actuator를 활용한 헬스 체크 방식으로 변경한다

* #92 [feat] : JwtFilter를 추가한다

* #92 [refactor] : 클레임 값 추출을 공통 메서드로 분리하고, 토큰 검증 메서드를 세분화한다

* #92 [feat] : 토큰 관련 에러코드를 수정한다

* #92 [feat] : JwtFilter에서 사용자 정보를 로드할 수 있도록 한다

* #92 [refactor] : CustomUserDetails을 사용해 유저를 반환하는 방식으로 리팩토링한다

* #92 [feat] : 테스트 코드를 수정한다

* #92 [fix] : 클레임 타입 오류를 해결한다

* #92 [refactor] : 불필요한 검증 로직을 제거한다

* #92 [fix] : PATCH 메서드를 허용한다

* [feat] : 소셜 로그인 실패 시 처리 로직을 추가한다 (#153)

* [fix] : 가장 많이 되는 시간 6개를 찾을 시 바로 종료하도록 수정한다 (#154)

* [feat] : 응답에 이벤트 ID 필드를 추가한다 (#156)

* [fix] : 빈 스케줄 등록과 관련한 에러들을 해결한다 (#160)

* #157 [feat] : 새로운 필드를 추가한다

* #157 [feat] : 이벤트 생성자가 빈 스케줄을 등록할 수 있다

* #157 [fix] : 멤버도 CREATOR로 반환되는 문제를 해결한다

* #157 [remove] : 불필요한 메서드를 제거한다

* #157 [feat] : 멤버가 빈 스케줄을 등록할 수 있다
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@bbbang105 bbbang105

Labels
🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정 🚨 fix 버그 수정 / 에러 해결 🔄 refactor 코드 리팩토링 😵‍💫 sangho 상호 PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[feat] : 빈 스케줄 등록과 관련한 에러들을 해결한다
1 participant
@bbbang105