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

Handle authorization response and return result object. #426

Merged

Conversation

brnnmrls
Copy link
Member

@brnnmrls brnnmrls commented May 20, 2024

Once the authorization response is processed, the next step is to handle the response by passing back the necessary tokens to the requesting party. A new type is made to encapsulate the completion of the VwG flow. Items that this result object holds are expirationDate, accessTokenString, refreshTokenString, and verifiedAccountDetails. A new method -[GIDVerifyAccountDetail addCompletionCallback:] will create a result object with these properties filled. This class also holds -[GIDVerifiedAccountDetailResult refreshTokensWithCompletion:] to allow developers refresh tokens themselves.

…st strict mocks to less restrictive. Also, fix nullability warnings in GIDFakeMainBundle.
@brnnmrls brnnmrls linked an issue Jun 5, 2024 that may be closed by this pull request
Base automatically changed from briannamorales/processAuthorizationResponse to briannamorales/vwg-flow June 5, 2024 17:54
@brnnmrls brnnmrls marked this pull request as ready for review June 5, 2024 23:44
@brnnmrls brnnmrls requested a review from mdmathias June 5, 2024 23:44
@brnnmrls brnnmrls linked an issue Jun 6, 2024 that may be closed by this pull request
…lso, add `hash` methods to overridden `isEqual:` methods.
@brnnmrls brnnmrls requested a review from mdmathias June 13, 2024 15:11
@brnnmrls brnnmrls merged commit 7e02188 into briannamorales/vwg-flow Jun 17, 2024
10 of 11 checks passed
@brnnmrls brnnmrls deleted the briannamorales/handleAuthorizationResponse branch June 17, 2024 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants