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

feat: kakao login scopes #411

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

nnnnoel
Copy link

@nnnnoel nnnnoel commented Jul 8, 2024

Copy link
Member

@hyochan hyochan left a comment

Choose a reason for hiding this comment

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

안녕하세요! PR 감사드립니다.
불필요한 띄어쓰기만 우선 제거해주시면 매우 감사하겠습니다!!

let appKey = try? KakaoSDK.shared.appKey();

Copy link
Member

Choose a reason for hiding this comment

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

불필요한 띄어쓰기를 되돌려 주시면 감사하겠습니다.

@objc(logout:rejecter:)
func logout(_ resolve: @escaping RCTPromiseResolveBlock,
rejecter reject: @escaping RCTPromiseRejectBlock) -> Void {
rejecter reject: @escaping RCTPromiseRejectBlock) -> Void {
Copy link
Member

Choose a reason for hiding this comment

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

여기도요!

@hyochan hyochan added the 🎯 feature New feature label Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎯 feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants