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 구현 #297

Merged
merged 9 commits into from
Feb 11, 2025
Merged

[FEAT] 약관 동의 조회 API 구현 #297

merged 9 commits into from
Feb 11, 2025

Conversation

rinarina0429
Copy link
Member

@rinarina0429 rinarina0429 commented Feb 10, 2025

Related Issue

Key Changes

약관 동의를 조회하는 API를 구현했습니다.
사용자가 조회하기 위함이 아닌 클라이언트 측에서 기존 유저가 새롭게 동의를 받아야하는지 확인하기 위한 api 입니다.

  • 약관 동의 여부를 저장하기 위해 User 안에 isTermsAgreed, isPrivacyConsented, isMarketingConsented를 추가했습니다.
  • 필드 추가에 따라 생성자를 수정하였습니다.
  • GET /users/consent-settings 로 조회할 수 있습니다.

To Reviewers

  • 생성자만 수정해주면 되는 것으로 확인했는데 혹시 회원가입 시 필드 추가로 인한 오류가 일어날만한 예외사항이 있을지 한번 더 체크 부탁드립니다아
  • 아직 DB에는 필드 추가를 적용하지 않았습니다!

References

@rinarina0429 rinarina0429 self-assigned this Feb 10, 2025
@github-actions github-actions bot requested review from ChaeAg and Kim-TaeUk February 10, 2025 06:59
Copy link
Member

@ChaeAg ChaeAg left a comment

Choose a reason for hiding this comment

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

굿입니다! 👍
현재는 약관이 세개뿐이지만 더 많아지거나 변경될 시 수정이 용이하도록 별도의 테이블로 분리하는 것도 좋을 것 같네요!

@rinarina0429 rinarina0429 merged commit ae3d952 into dev Feb 11, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 약관 동의 조회 API 구현
2 participants