-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[google_sign_in] Migrate to new pubspec plugins schema #2371
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, make sure to either to first add an android folder for the web plugin, or land this without endorsing web
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Please get @hterkelsen's approval as well
## 4.1.0 | ||
|
||
* Support web by default. | ||
* Use the new plugins pubspec schema. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: remove this line
Please bump the minimum version of <!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<meta name="google-signin-client_id" content="159623150305-q05bbbtsutr02abhips3suj7hujfk4bg.apps.googleusercontent.com" />
<title>Google Sign-in Example</title>
</head>
<body>
<script src="main.dart.js" type="application/javascript"></script>
</body>
</html> |
@hterkelsen done! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
flutter/flutter#46255