yoonjaepark / flutter_naver_login

A Flutter plugin for Naver Sign In.
BSD 2-Clause "Simplified" License
46 stars 69 forks source link

imp: ios 플러그인의 getCurrentAccessToken 메서드에서도 expiresAt을 리턴하도록 추가, refreshAccessTokenWithRefreshToken 메서드 추가 #62

Closed versatile309 closed 2 years ago

versatile309 commented 2 years ago

안녕하세요! 우선 라이브러리를 만들고 운영해주셔서 감사합니다:) 다름이 아니라 ios에서 expiresAt이 빈값으로 넘어와서 isValid 메서드가 깨지는 현상이 있어서 ios 플러그인 코드에 expiresAt을 추가했습니다. android와 동일한 포멧으로 타임스탬프를 리턴합니다. 제가 Object-C로 개발해 본 적이 없어서 적절한 코드인지는 모르겠네요. 리뷰 해주시면 감사하겠습니다~

다른 repo에 pr을 올리는게 처음이라 코드랑, 양식 모드 부족한 부분이 있으면 언제든 피드백 부탁드립니다!

yoonjaepark commented 2 years ago

@versatile309 님 소중한 pr감사합니다. 주말에 다른부분과 같이 적용시켜서 업데이트 하곘습니다 :)

versatile309 commented 2 years ago

@yoonjaepark 빠른 답변 감사합니다! 두번째 commit으로 refreshToken을 통해 accessToken을 갱신하는 메서드도 추가했는데요! 같이 리뷰해주시면 감사하겠습니다~ 🙏

versatile309 commented 2 years ago

안녕하세요! 혹시 진행상황 알 수 있을까요?