Skip to content

Conversation

@youjin09222
Copy link
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
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
Contributor

Choose a reason for hiding this comment

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

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

Copy link
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
Contributor

Choose a reason for hiding this comment

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

그렇구나 딩딩딩딩딩

Copy link
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.

4 participants