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

login함수 리턴타입 처리의 수정이 필요해보입니다 #337

Closed
YangJonghun opened this issue Oct 5, 2022 · 3 comments · Fixed by #345
Closed

login함수 리턴타입 처리의 수정이 필요해보입니다 #337

YangJonghun opened this issue Oct 5, 2022 · 3 comments · Fixed by #345

Comments

@YangJonghun
Copy link
Contributor

Version of kakao-login libraries

v5.2.2

Version of react-native

RN버전과 관계 없음

Platforms you faced the error (IOS or Android or both?)

TS문제라 플랫폼과는 무관합니다

Expected behavior

login함수의 리턴타입이 통일되거나 웹의 login과 분리해야할것같습니다

Actual behavior

const { accessToken } = (await KakaoLogins.login()) as KakaoLogins.KakaoOAuthToken
앱에서 accessToken을 가져오는 경우 타입캐스팅을 강제해야하는 문제

웹 로그인기능이 추가되면서 login함수의 리턴타입이
Promise<KakaoLogins.KakaoOAuthToken>
에서
Promise<KakaoLogins.KakaoOAuthToken | KakaoLogins.KakaoOAuthWebToken>
로 변경되었는데 스키마가 완전히 달라서 간단한 구조분해할당 지원도 되지 않아 이건 앱/웹 로그인 함수를 분리하거나 스키마를 통일하거나하는 방향으로 수정할 필요가 있을 것 같습니다

Tested environment (Emulator? Real Device?)

@jaeseong-nuvi-labs
Copy link

getKakaoProfile()도 마찬가지의 문제가 발생하는것 같습니다.

@bang9
Copy link
Contributor

bang9 commented Nov 4, 2022

@YangJonghun @jaeseong-nuvi-labs
#345 의 변경사항을 적용해서 테스트해보실 수 있을까요?

@YangJonghun
Copy link
Contributor Author

@bang9
작성해주신 PR #345 의 리뷰로 남기겠습니다!

hyochan pushed a commit that referenced this issue Nov 26, 2022
파일별로 플랫폼별 로직을 분리합니다.

플랫폼별 메소드 인터페이스가 달라서 생기는 문제를, 임시로 오버로드를 통해서 분리하여 해결합니다.

사용하는 프로젝트에서 확장자별 파일을 찾을 수 있도록 package.json 의 main 필드를 업데이트 합니다.

- May resolve #337
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants