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 native modules #65

Merged
merged 1 commit into from
Oct 8, 2019
Merged

fix native modules #65

merged 1 commit into from
Oct 8, 2019

Conversation

bang9
Copy link
Contributor

@bang9 bang9 commented Oct 8, 2019

@heyman333 작업하면서 몇가지 상이한 프로퍼티 이름을 변경했습니다.
프로미스 PR 까지 합쳐서, 버전을 2.0.0 으로 릴리즈하고, breaking changes 추가하는게 좋을 것 같아요!


Use Native Map and Dictionary for javascript object

불필요한 JSON Parsing 과정 제거

before
Native String -> Convert to Javascript Object -> Javascript Object -> Use

after
Native Object -> Javascript Object -> Use

Rename profile properties

카카오 응답결과와 상이한 변수명들 변경

profile_image_path -> profile_image_url
thumbnail_image_path -> thumbnail_image_url
kakaotalk_user -> is_kakaotalk_user
email_verified -> is_email_verified

Support real-time profile

최신 카카오 SDK 에서 제공하는, 실시간 프로필 지원 (nickname profile image thumbnail image)

Update doc

iOS is_kakaotalk_user 외 세가지 항목 지원 목록 업데이트
is_kakaotalk_user, is_email_verified 등 권한이 필요한 항목들 공식문서에 따른 설명 추가

- Use Native Map and Dictionary for javascript object
- Rename profile properties (profile_image_url, thumbnail_image_url, is_kakaotalk_user, is_email_verified)
- Support real-time profile (nickname, profile image, thumbnail image)
@heyman333 heyman333 requested review from hyochan and sunkibaek and removed request for hyochan October 8, 2019 06:46
@heyman333
Copy link
Contributor

@bang9 감사합니다!
README에 있는 예제코드도 변경 가능할까요?

@heyman333 heyman333 requested review from heyman333 and removed request for sunkibaek and hyochan October 8, 2019 09:53
@bang9
Copy link
Contributor Author

bang9 commented Oct 8, 2019

@bang9 감사합니다!
README에 있는 예제코드도 변경 가능할까요?

변경되어있습니다!

@heyman333 heyman333 merged commit 7c38f7e into crossplatformkorea:master Oct 8, 2019
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 this pull request may close these issues.

None yet

2 participants