Skip to content

[Fix/#84] : 구글 로그인 오류 해결#1

Merged
gaeulzzang merged 13 commits intodevelopfrom
feature/#84-google-login
Apr 10, 2025
Merged

[Fix/#84] : 구글 로그인 오류 해결#1
gaeulzzang merged 13 commits intodevelopfrom
feature/#84-google-login

Conversation

@youjin09222
Copy link
Copy Markdown
Contributor

@youjin09222 youjin09222 commented Apr 8, 2025

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

  • merge할 브랜치의 위치를 확인해 주세요.(main❌/develop⭕)
  • 리뷰가 필요한 경우 리뷰어를 지정해 주세요.
  • 리뷰는 PR이 올라오면 최대한 빠르게 진행합니다.
  • P1 단계의 리뷰는 빠르게 확인 후 반영합니다.
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

  • closed #84

📎 𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

  • 구글 로그인 오류 해결
  • 로그인 UI 변경
  • 토큰 재발급 오류 해결

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

KakaoTalk_20250409_044809932.mp4

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

@youjin09222 youjin09222 added 🔨 [FIX] 버그 및 오류 발생 및 해결 🐰 유진 뉴진스 ♻️ [REFACTOR] 코드 리팩토링 💟 [UI] UI 작업 labels Apr 8, 2025
@youjin09222 youjin09222 self-assigned this Apr 8, 2025
Copy link
Copy Markdown
Contributor

@gaeulzzang gaeulzzang left a comment

Choose a reason for hiding this comment

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

image
하.... 이건 문제없이 잘 돌아가서 다행이다 ㅋㅋㅋㅋ큐ㅠㅠ
워크플로우에 슬랙 연동된게 있어서 제가 제거해버렸으니 참고해주세요!! 수고했어잉 ❤️

@SerialName("accessToken") val accessToken: String,
@SerialName("refreshToken") val refreshToken: String,
@SerialName("authType") val authType: String
@SerialName("authId") val authId: String
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P3: 이게 달라서 재발급이 안됐던걸까유?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

요것도 그렇고, SUCCESS 코드가 200으로 되어있어서 재발급이 안된거였으요..

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

그렇구나 딩딩딩딩딩

Copy link
Copy Markdown
Member

@Eonji-sw Eonji-sw left a comment

Choose a reason for hiding this comment

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

LGTM! 수고많으셨어요

@gaeulzzang gaeulzzang merged commit b24f731 into develop Apr 10, 2025
1 check passed
@gaeulzzang gaeulzzang deleted the feature/#84-google-login branch April 10, 2025 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 [FIX] 버그 및 오류 발생 및 해결 🐰 유진 뉴진스 ♻️ [REFACTOR] 코드 리팩토링 💟 [UI] UI 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants