Skip to content

[feat] 사용자 회원탈퇴, 닉네임 수정 #40

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 4 commits into from
Jul 15, 2025
Merged

[feat] 사용자 회원탈퇴, 닉네임 수정 #40

merged 4 commits into from
Jul 15, 2025

Conversation

@jiwon1217
Copy link
Collaborator

@jiwon1217 jiwon1217 commented Jul 15, 2025 •
edited
Loading

🛰️ Issue Number

🪐 작업 내용

  • DELETE /user/me: 회원탈퇴 기능
  • UPDATE /user/me: 닉네임 수정 기능
  • 프론트엔드에서 form 요청 시 메서드가 변환되지 않는 문제가 발생할 수 있어서 WebConfig에 코드를 추가했습니다. js로 method를 지정해서 요쳥을 보낸다면 해당 코드는 필요없지만, 혹시 모를 form 요청에 대비해서 만들어 둔 것이라고 생각하시면 됩니다 !!
@Bean
    public HiddenHttpMethodFilter hiddenHttpMethodFilter() {
        return new HiddenHttpMethodFilter();
    }

📚 Reference

✅ Check List

  • 코드가 정상적으로 컴파일되나요?
  • 테스트 코드를 통과했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

@jiwon1217 jiwon1217 added the enhancement New feature or request label Jul 15, 2025
@jiwon1217 jiwon1217 self-assigned this Jul 15, 2025
sehee123
sehee123 approved these changes Jul 15, 2025
View reviewed changes
Copy link
Collaborator

@sehee123 sehee123 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 🙌

LimKangHyun
LimKangHyun approved these changes Jul 15, 2025
View reviewed changes
Copy link
Collaborator

@LimKangHyun LimKangHyun left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!👍

silver-eunjoo
silver-eunjoo approved these changes Jul 15, 2025
View reviewed changes
Copy link
Collaborator

@silver-eunjoo silver-eunjoo left a comment

Choose a reason for hiding this comment

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

왕! 고생하셨습니다 !

dlsrks1021
dlsrks1021 reviewed Jul 15, 2025
View reviewed changes
backend/src/main/java/io/f1/backend/domain/user/api/UserController.java Outdated
import org.springframework.web.bind.annotation.RequestBody;
import org.springframework.web.bind.annotation.RequestMapping;

@Controller
Copy link
Collaborator

@dlsrks1021 dlsrks1021 Jul 15, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

[L5-참고의견]
Controller로 어노테이션이 되어 있다면 뷰 탐색 동작이 일어날 것 같습니다.
확인해 주시면 감사하겠습니다.

Copy link
Collaborator Author

@jiwon1217 jiwon1217 Jul 15, 2025

Choose a reason for hiding this comment

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

감사합니다 ! 리뷰 반영했습니다

@jiwon1217 jiwon1217 merged commit e4a1912 into dev Jul 15, 2025
2 checks passed
@jiwon1217 jiwon1217 deleted the feat/32 branch July 15, 2025 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dlsrks1021 dlsrks1021 dlsrks1021 approved these changes

@sehee123 sehee123 sehee123 approved these changes

@silver-eunjoo silver-eunjoo silver-eunjoo approved these changes

@LimKangHyun LimKangHyun LimKangHyun approved these changes

Assignees

@jiwon1217 jiwon1217

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@jiwon1217 @dlsrks1021 @sehee123 @silver-eunjoo @LimKangHyun