-
Notifications
You must be signed in to change notification settings - Fork 28.9k
Closed
Labels
P3Issues that are less important to the Flutter projectIssues that are less important to the Flutter projectc: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.good first issueRelatively approachable for first-time contributorsRelatively approachable for first-time contributorsp: google_sign_inThe Google Sign-In pluginThe Google Sign-In pluginpackageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.platform-webWeb applications specificallyWeb applications specifically
Description
The js-interop layer of the web version of Google Sign In used to be generated by the js_facade_gen script, but it is no longer.
Proposal
Move the js-interop layer (gapi.dart
, gapiauth2.dart
...) outside of generated
(or rename generated
to js_interop
or similar), so the files don't receive any special treatment.
Also: fix tests, and whatever issues the analyzer may start having with the change, of course.
Metadata
Metadata
Assignees
Labels
P3Issues that are less important to the Flutter projectIssues that are less important to the Flutter projectc: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.good first issueRelatively approachable for first-time contributorsRelatively approachable for first-time contributorsp: google_sign_inThe Google Sign-In pluginThe Google Sign-In pluginpackageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.platform-webWeb applications specificallyWeb applications specifically