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: MO 카카오톡으로 링크를 열 때, 홈으로 가는 오류 수정 #1697

Merged
merged 9 commits into from
Dec 17, 2024

Conversation

pepperdad
Copy link
Member

@pepperdad pepperdad commented Dec 3, 2024

🤫 쉿, 나한테만 말해줘요. 이슈넘버

🧐 어떤 것을 변경했어요~?

  • 카카오톡 -> 외부 브라우저로 갈 때, /intro페이지에서 외부 브라우저가 열리게 되는데, 이 상황에서 카카오톡의 인앱의 sessionStorage에 path가 저장되고, 외부 브라우저에서는 sessionStorage에 path가 없어서, 로그인 이후에도 홈으로 가는 상황입니다.
  • 따라서, 외부 브라우저로(/intro 페이지로) returnUrl을 파라미터로 전달하여, 외부 브라우저에서도 sessionStorage에 path가 저장될 수 있도록 변경하였습니다.

🤔 그렇다면, 어떻게 구현했어요~?

  • 카카오톡을 통해 외부 브라우저 호출시, returnUrl 전달
  • 외부 브라우저에서 URL 파라미터(returnUrl)을 저장하여 로그인 후 리다이렉트 처리

❤️‍🔥 당신이 생각하는 PR포인트, 내겐 매력포인트.

아래 스크린샷처럼 세션 스토리지에 저장된 path를 불러오는 것을 확인했습니다! 그러나, 모바일에서 로그인이 dev 서버로 병합되어야 테스트가 가능할 것 같습니다!

어푸되면 머지 후 추가로 테스트 하겠습니다!

📸 스크린샷, 없으면 이것 참,, 섭섭한데요?

IMG_4105

Copy link

height bot commented Dec 3, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link

github-actions bot commented Dec 3, 2024

✨✨ 스토리북으로 확인하기 ✨✨

Copy link

github-actions bot commented Dec 3, 2024

🚀 프리뷰 배포 확인하기 🚀

https://ee12a07f.sopt-internal-dev.pages.dev

@pull-request-size pull-request-size bot added size/M and removed size/S labels Dec 3, 2024
@pull-request-size pull-request-size bot added size/S and removed size/M labels Dec 9, 2024
Copy link
Member

@seojisoosoo seojisoosoo left a comment

Choose a reason for hiding this comment

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

정말 빠르시군요!!! 최고입니다!!!!🚀

@pepperdad pepperdad self-assigned this Dec 14, 2024
@pepperdad pepperdad added this pull request to the merge queue Dec 17, 2024
Merged via the queue into main with commit a7aae8e Dec 17, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: MO 카톡 브라우저 거친 후, 플그 홈으로만 가는 오류 수정
2 participants