Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[extension_google_sign_in_as_googleapis_auth] Update example...#311

Merged
ditman merged 2 commits intoflutter:masterfrom
kevmoo:extension_google_sign_in_as_googleapis_auth_fixes
Mar 15, 2021
Merged

[extension_google_sign_in_as_googleapis_auth] Update example...#311
ditman merged 2 commits intoflutter:masterfrom
kevmoo:extension_google_sign_in_as_googleapis_auth_fixes

Conversation

@kevmoo
Copy link
Contributor

@kevmoo kevmoo commented Mar 14, 2021

...to use scope from the People API
Remove x bit from readme and pubspec.yaml
Add link to root repo README

Pre-launch Checklist

  • The title of the PR starts with the name of the package surrounded by square brackets, e.g. [shared_preferences]
  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide].
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
  • I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
  • I updated CHANGELOG.md to add a description of the change.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the [CLA].
  • All existing and new tests are passing.

@kevmoo kevmoo requested a review from ditman March 14, 2021 04:24
Copy link
Contributor Author

Choose a reason for hiding this comment

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

email doesn't seem to be needed...

Copy link
Member

Choose a reason for hiding this comment

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

I thought both 'profile' and 'email' were required to sign in, but not seeing profile there is suspicious :P let me run this example real quick!

https://developers.google.com/identity/protocols/oauth2/scopes#google-sign-in

Copy link
Member

Choose a reason for hiding this comment

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

('profile', 'email' and 'openid' seem to get added by the google sign-in API automatically if absent from the original request, good!)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've been running it all weekend – no issues!

kevmoo and others added 2 commits March 14, 2021 14:56
Make GoogleSignIn instance final
To use scope from the People API
Remove x bit from readme and pubspec.yaml
Add link to root repo README
@google-cla

This comment has been minimized.

@ditman

This comment has been minimized.

Copy link
Member

@ditman ditman left a comment

Choose a reason for hiding this comment

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

Let's go! Thanks for the fixes @kevmoo!

@kevmoo
Copy link
Contributor Author

kevmoo commented Mar 15, 2021

@ditman – am I okay to land this?

@ditman ditman merged commit 344f575 into flutter:master Mar 15, 2021
@ditman
Copy link
Member

ditman commented Mar 15, 2021

@ditman – am I okay to land this?

@kevmoo I've just clicked the button, you're always welcome to merge :P

@kevmoo kevmoo deleted the extension_google_sign_in_as_googleapis_auth_fixes branch March 15, 2021 20:59
@ditman
Copy link
Member

ditman commented Mar 15, 2021

The latest changes have been published to pub.dev as 2.0.2: https://pub.dev/packages/extension_google_sign_in_as_googleapis_auth/changelog#202

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.

2 participants