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/#195] 삼성인터넷/크롬 선택지 없이 바로 삼성인터넷으로 우회 #200

Merged
merged 1 commit into from
Oct 15, 2023

Conversation

simeunseo
Copy link
Member

@simeunseo simeunseo commented Oct 11, 2023

#195

todo

  • 삼성인터넷/크롬 선택지 없이 바로 삼성인터넷으로 열리도록 한다.

📌 내가 알게 된 부분

  • 스킴이라는거... 생소하네요 자료 찾기가 어려운 것 같아요 특히 삼성인터넷이다보니...

📌 공유하고 싶은 부분

  • 카톡 인앱브라우저를 피하는 기능을 담당하는 changeBrowser.ts라는 파일의 inappbrowserout이라는 function에서 navigator.userAgent라는 값으로 현재 어떤 인앱으로 들어왔는지를 확인합니다. kakaotalk이라면 외부 브라우저를 호출하고, 이후 안드로이드라면 location.href = 'intent://' + target_url.replace(/https?:\/\//i, '') + '#Intent;scheme=http;package=com.android.chrome;end';라는 코드를 통해 크롬을 띄우는 것이 원래 코드였습니다. 그런데 의도와는 다르게 크롬이 아니라 크롬/삼성인터넷 중 선택하는 모달창이 나왔었죠. 그 모달창에서 삼성인터넷을 선택하면 다시 같은 분기가 실행되어 같은 모달창이 뜨는 것이었습니다. 그 모달을 띄울지 말지를 조정하는 방법을 찾지 못했고 (스킴을 바꿔야하는걸까요.. 바꾼다면 어떻게...) 차선책으로 아예 안드로이드일 경우에- 라는 if분기를 삭제하니 모달이 뜨지 않고 바로 삼성인터넷으로 연결이 되었습니다.
  • 혹시모르니 여러 브라우저에서 qa가 필요할 것 같습니다.

@simeunseo simeunseo added fix fix 은서 은서의 개발 라벨 labels Oct 11, 2023
@simeunseo simeunseo self-assigned this Oct 11, 2023
@vercel
Copy link

vercel bot commented Oct 11, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
asap ✅ Ready (Inspect) Visit Preview Oct 11, 2023 2:05pm

@simeunseo simeunseo merged commit 894faf8 into develop Oct 15, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix fix size/M 은서 은서의 개발 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant