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

[Proposal] 명시적 init API로의 변환 #333

Open
mym0404 opened this issue Sep 29, 2022 · 0 comments
Open

[Proposal] 명시적 init API로의 변환 #333

mym0404 opened this issue Sep 29, 2022 · 0 comments

Comments

@mym0404
Copy link
Contributor

mym0404 commented Sep 29, 2022

Version of kakao-login libraries

5.2.0

Version of react-native

0.70.1

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

Android

Description

해당 라이브러리는 Android auto initializationiOS auto initialization 을 통해 암시적으로 자동 초기화를 합니다.

Set-up step에서 별다른 언급 없이
@string/kakao_app_key를 android string resource에, KAKAO_APP_KEY Info.plist에 명시하게 함으로써 key들을 overwrite시켜 동작합니다.

그리고 둘다 초기화 시점이 Native module class의 language constructor 인데, 대략 다음과 같이 Application.onCreate 에서 호출하라는 document된 카카오의 설명은 그렇다 치더라도
image

RN new architecture의 turbo module이 지원해준다는 lazy loading이 활성화될 시 어떤 문제점이 생길지 알 수 없는 상황입니다.

또한, 위와 같이 초기화 순서가 꼬일 시에 Kakao SDK중 login만 사용하지 않는 제가 관리하는 프로젝트 같은 경우 SDK initialization을 먼저 해둔다음 다시 초기화가 덮어씌워져버리면(카카오 SDK가 어떻게 이를 처리하는지는 잘 모르겠네요) setup guide만 보고 설치한 사람들은 혼란이 올 수 있습니다.

실제로 갑자기 코드는 변경한게 없는데 RN을 업그레이드 해보니 dooboolab이 띄워져서 조금 당황스럽군요 🥲
image

Proposal

breakchange가 되더라도 initialization을 명시적으로 코드에 포함해야하게 수정하는 방향으로 제안드립니다.

@mym0404 mym0404 changed the title [Proposal] Auto initialization이 필요한가요? [Proposal] 명시적 init API로의 변환 Sep 29, 2022
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

No branches or pull requests

1 participant