Skip to content

Conversation

sealesj
Copy link
Contributor

@sealesj sealesj commented Dec 1, 2023

In order to resolve the dashboard sign-in issue, bump google_sign_in_web to 0.12.1

List which issues are fixed by this PR. You must list at least one issue.
Resolves: flutter/flutter#139382

Pre-launch Checklist

  • 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 the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@sealesj sealesj requested a review from keyonghan December 1, 2023 20:08
@ditman
Copy link
Member

ditman commented Dec 1, 2023

@sealesj please, don't land this, I'm about to publish a PR for 0.12.3 which doesn't break cocoon :)

@sealesj
Copy link
Contributor Author

sealesj commented Dec 1, 2023

Awesome! Thanks for letting me know

@ditman
Copy link
Member

ditman commented Dec 1, 2023

It seems the fixed version is going to be: 0.12.2+1.

auto-submit bot pushed a commit to flutter/packages that referenced this pull request Dec 2, 2023
When initializing a `CodeClient`, it is mandatory to pass at least one `scope` (or the JS initialization will crash).

This PR ensures that the `CodeClient` is not created unless `initialScopes.isNotEmpty`, and lets the user know when attempting to use said client that it hasn't been initialized properly.

## Issues

* Fixes: flutter/flutter#139382
* Fixes: flutter/flutter#62474
* Closes: flutter/cocoon#3304 (makes this PR unneeded)

## Tests

* Tested locally with `cocoon`.
@sealesj sealesj deleted the bump_google_sign_in_web branch December 4, 2023 20:37
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.

[infra][cocoon] Unable to use the dashboard
2 participants