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] 디스코드 계정 개수 제한 적용 #53

Merged
merged 1 commit into from
Nov 15, 2024
Merged

Conversation

iiqcov
Copy link
Contributor

@iiqcov iiqcov commented Nov 15, 2024

#️⃣ 관련 이슈

💡 작업내용

디스코드 개정 개수 1개로 제한 수정

  • 사용자당 하나의 계정만 존재하기 때문에 OneToOne 관계로 전환하고 하드 딜리트 구현으로 변경하였습니다.
  • 연결 시 이미 연결이 존재한다면 예외를 반환합니다.
  • 어떤 디스코드인지 필요하지 않게됨에 따라 컨트롤러의 경로변수 로직을 제거하였습니다.

디스코드 서비스 클래스 분리 구현

  • 디스코드 레포지토리와 연결되는 도메인 서비스 로직을 따로 구현하였습니다.

디스코드 인가코드 디코딩

  • 구글과 같은 오류가 발생하여 디코딩 로직을 추가하였습니다.

📸 스크린샷(선택)

📝 기타

@iiqcov iiqcov added the ✨ feature 기능 구현 label Nov 15, 2024
@iiqcov iiqcov requested a review from cowboysj November 15, 2024 09:55
@iiqcov iiqcov self-assigned this Nov 15, 2024
Copy link
Member

@cowboysj cowboysj left a comment

Choose a reason for hiding this comment

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

수고하셨습니당~!

@iiqcov iiqcov merged commit d03a1a6 into develop Nov 15, 2024
2 checks passed
@iiqcov iiqcov deleted the feat/#52 branch November 23, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants