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

fix: ResignAPI 호출 시 UserDefaults MemberId nil 값 뜨는 이슈 수정해요 #700

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

Do-hyun-Kim
Copy link
Collaborator

🔵PR을 올리기 전 아래 사항을 확인해주세요.

  • 구현한 로직과 기능이 올바르게 작동되는지 충분히 테스트해주세요.
  • 코드의 성능이나 메모리 효율성이 적절하게 고려되었는지, 불필요한 코드가 없는지 검토해주세요.
  • 이번 PR에서 구현된 주요 기능이나 해결된 문제에 대해 자세히 서술해주세요.
    (위 내용은 지워주세요)

😽개요

  • MyUserDefault 내부 메서드인 loadFamilyId, loadFamilyName Type Annotation String으로 수정해요

🛠️작업 내용

  • 기존 UserDefaults memberId 값이 nil 값으로 반환 되고 있었던걸 familyId 원본 값으로 반환하도록 수정하였습니다.
  • ManagementViewController 에서 toSetting 메서드에 memberid 값이 없어서 설정 아이콘을 눌러도 PrivacyViewController로 이동하지 않았는데 UserDefaults를 수정하면서 이동하도록 수정되었습니다.

✅테스트 케이스

  • ManagementViewController에서 설정 아이콘 클릭 시 PrivacyViewController로 이동하는지 확인해요
  • 회원 탈퇴 API 요청 시 정상적으로 요청되는지 확인해요
  • memberId 값이 있는지 확인해요

🙏🏻아래와 같이 PR을 리뷰해주세요.

  • PR 내용이 부족하다면 보충 요청해주세요.
  • 코드 스타일이 팀의 규칙에 맞게 작성되었는지, 일관성을 유지하고 있는지 확인해주세요.
  • 코드에 대한 문서화나 주석이 필요한 부분에 적절하게 작성되어 있는지 확인해주세요.
  • 구현된 로직이 효율적이고 올바르게 작성되었는지, 아키텍처를 잘 준수하고 있는지 검토해주세요.
  • 네이밍, 포매팅, 주석 등 코드의 일관성이 유지되고 있는지 확인해주세요.

@Do-hyun-Kim Do-hyun-Kim requested a review from akrudal November 6, 2024 15:03
@Do-hyun-Kim Do-hyun-Kim self-assigned this Nov 6, 2024
@Do-hyun-Kim Do-hyun-Kim changed the title fix: MyUserDefaults memberId, userName Type Annotation String으로 수정 fix: ResignAPI 호출 시 UserDefaults MemberId nil 값 뜨는 이슈 수정해요 Nov 6, 2024
@Do-hyun-Kim Do-hyun-Kim merged commit 5878518 into develop Nov 6, 2024
Do-hyun-Kim added a commit that referenced this pull request Nov 12, 2024
* origin/develop:
  fix: SplashViewReactor 화면 전환 issue 수정 (#702)
  fix: MyUserDefaults memberId, userName Type Annotation String으로 수정 (#700)
  fix: 로그인 버튼 클릭시 touch event cancel 현상 수정해요 (#695)
  fix: QA issue를 수정합니다(#696)
  feat: AlertService 구현 (#692)
  refactor: Calendar 관련 뷰 컨트롤러, 리액터 등 부가 코드 리팩토링 (#682)
@akrudal akrudal deleted the fix/#699-userdefault-type-issue-modify branch December 17, 2024 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant