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.

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

fix: 카카오 회원가입시 user id 수정 (#408, #412) #411

Merged
merged 20 commits into from
Apr 14, 2023
Merged

fix: 카카오 회원가입시 user id 수정 (#408, #412) #411

merged 20 commits into from
Apr 14, 2023

Conversation

hyun99999
Copy link
Member

@hyun99999 hyun99999 commented Apr 14, 2023 •
edited
Loading

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • 카카오로 회원가입시에 user 의 고유값으로 이메일을 전달하고 있었음. 이것을 고유아이디로 수정하였습니당

📮 관련 이슈

@hyun99999
fix: framework not found 에러 해결 (#397))
87cf3ef 
- pod deintegrate
@hyun99999
chore: pod update (#397)
a845b42
@hyun99999
fix: url path 변경 (#397)
ad0095b
@hyun99999
fix: UpdateNote DataModel 변경 (#397)
ed4402a 
- isForce 자료형 변경.
@hyun99999
feat: HomeViewController 에서 업데이트 API 구현 (#397)
2517971
@hyun99999
add: ModuleFactory 에 화면전환 추가 (#397)
1684cc8 
- UpdateViewController 화면전환
@hyun99999
add: UpdateVC 화면전환에 modalTransitionStyle 추가
f2be46b
@hyun99999
feat: 버전 확인 후 업데이트 유무 체크 (#397)
e306a6e
@hyun99999
feat: 버전 체크 후에 화면전환 메서드 구현 (#397)
3979d34
@hyun99999
feat: touchCancelButton 이벤트 구현 (#397)
631c5c0
@hyun99999
feat: touchUpdateButton 이벤트 구현 (#397)
9effbf7
@hyun99999
feat: touchCheckBox 이벤트 주석 제거 (#397)
cf5083c
@hyun99999
design: 업데이트의 강제 유무에 따른 디자인 수정 (#397)
0a1aca2 
- '확인했어요'라벨과 체크박스 display
@hyun99999
add: UpdateViewController 변수 추가 (#397)
52d1c89
@hyun99999
feat: 업데이트 내역 텍스트 다루기 (#397)
84108bc
@hyun99999
feat: touchCheckBox 이벤트 기능 추가 (#397)
ce2cb55 
- 업데이트를 확인한 유무를 전달하는 변수 토글.
@hyun99999
refactor: forceUpdateAgreement 의 접근제어자 수정 (#397)
d7a6ee9 
- 강제업데이트인 경우에 해당 변수를 사용하지 않기 때문에 lazy
- 파일내에서만 사용하기 때문에 private
@hyun99999
Merge branch 'develop' of https://github.com/TeamNADA/NADA-iOS-ForRel…
3f7933e 
…ease into feature/#397
@hyun99999
fix: 카카오 회원가입시 user id 수정 (#408)
6fe01ac
@hyun99999 hyun99999 added Hyungyu 🐯 현규 교수님 작업 Fix 코드 수정 및 버그, 오류 해결 labels Apr 14, 2023
@hyun99999 hyun99999 requested a review from dlwns33 April 14, 2023 05:51
@hyun99999 hyun99999 self-assigned this Apr 14, 2023
dlwns33
dlwns33 approved these changes Apr 14, 2023
View reviewed changes
Copy link
Member

@dlwns33 dlwns33 left a comment

Choose a reason for hiding this comment

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

image

NADA-iOS-forRelease/Sources/ViewControllers/Login/LoginViewController.swift
Comment on lines +115 to +116
if let userID = user?.id {
self.postUserSignUpWithAPI(socialID: String(userID), socialType: "KAKAO")
Copy link
Member

Choose a reason for hiding this comment

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

확인했습니다!

Copy link
Member Author

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋㅋㅋㅋㅋ 정상영업..합니다..

Copy link
Member Author

Choose a reason for hiding this comment

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

image

@hyun99999 hyun99999 merged commit 518a2c7 into develop Apr 14, 2023
@hyun99999 hyun99999 deleted the feature/#408 branch April 14, 2023 06:04
@hyun99999 hyun99999 changed the title fix: 카카오 회원가입시 user id 수정 (#408) fix: 카카오 회원가입시 user id 수정 (#408, #412) Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dlwns33 dlwns33 dlwns33 approved these changes

Assignees

@hyun99999 hyun99999

Labels
Fix 코드 수정 및 버그, 오류 해결 Hyungyu 🐯 현규 교수님 작업
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fix: GenericResponse 데이터 모델 수정 fix: 카카오 회원가입시 user id 수정
2 participants
@hyun99999 @dlwns33