Skip to content

Conversation

@Jimin730
Copy link
Contributor

관련 Issue (필수)

주요 변경 사항 (필수)

  • 회원 탈퇴 시 providerId 에 탈퇴일시를 포함하여 포맷할 수 있도록 변경
  • Member providerId 필드 @column 속성에 updatable 속성 제거

리뷰어 참고 사항

탈퇴 시 DB에 저장된 providerId

스크린샷 2025-09-11 오후 6 19 10

추가 정보

없음

PR 작성 체크리스트 (필수)

  • 제목이 Issue와 동일함을 확인했습니다.
  • 리뷰어를 지정했습니다.
  • 프로젝트를 연결했습니다.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Summary of Changes

Hello @Jimin730, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 회원 탈퇴 시 기존 회원의 providerId 데이터 처리 방식을 개선합니다. 탈퇴한 회원의 providerId에 탈퇴 일시를 추가하여 고유성을 유지하고, providerId 필드의 업데이트 가능성을 열어 데이터 무결성을 강화하는 것이 주 목적입니다.

Highlights

  • 회원 탈퇴 시 providerId 포맷 변경: 회원 탈퇴 시 기존 회원의 providerId 필드에 탈퇴 일시를 포함하여 포맷하도록 변경되었습니다. 이는 탈퇴한 회원의 providerId가 재사용될 경우 발생할 수 있는 충돌을 방지하고 데이터의 고유성을 유지합니다.
  • Member providerId 필드 updatable 속성 제거: Member 엔티티의 providerId 필드에 설정되어 있던 @column(updatable = false) 속성이 제거되었습니다. 이로써 providerId 필드의 업데이트가 가능해졌습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in issue comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@chanwonlee chanwonlee left a comment

Choose a reason for hiding this comment

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

코드 확인했습니다! 빠르게 구현 해주셔서 정말 감사합니다!

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

회원 탈퇴 시 개인정보 보호를 위해 providerId에 탈퇴 시각을 포함하여 업데이트하는 기능 추가와 관련 테스트 코드 작성이 확인되었습니다. providerId를 업데이트할 수 있도록 @Column에서 updatable = false 속성을 제거한 변경은 올바릅니다. 전반적으로 PR의 목적에 맞게 코드가 잘 수정되었으나, 성능 및 테스트 정확성 측면에서 두 가지 개선점을 발견하여 리뷰 댓글로 남겼습니다. DateTimeFormatter를 상수로 만들어 재사용하고, 테스트 코드의 로그아웃 검증 로직을 수정하는 것을 권장합니다.

@Jimin730 Jimin730 force-pushed the feature/156-member-delete branch from b1ce114 to da6e49e Compare September 11, 2025 09:23
@Jimin730 Jimin730 merged commit f492506 into develop Sep 11, 2025
3 checks passed
@Jimin730 Jimin730 deleted the feature/156-member-delete branch September 11, 2025 09:25
@github-project-automation github-project-automation bot moved this to Done in 💙 BE Sep 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[FEATURE] 회원 탈퇴 시 기존 회원 providerId 데이터 포맷

3 participants