-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Update firebase.json #7192
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
Update firebase.json #7192
Conversation
firebase.json
Outdated
@@ -301,6 +301,7 @@ | |||
{ "source": "/go/platform-based-key-events", "destination": "https://docs.google.com/document/d/1FDuxxEh810XpY561SgeiyUR0gU2eVz5kRpCFI55ibec/edit", "type": 301 }, | |||
{ "source": "/go/platform-channels-in-federated-plugins", "destination": "https://docs.google.com/document/d/1BSrDA_FjLU-3T0H774RocJ4IYyc0ytMl937rAimGJdg/edit?usp=sharing", "type": 301 }, | |||
{ "source": "/go/platform-interface-breaking-changes", "destination": "https://docs.google.com/document/d/1LD7QjmzJZLCopUrFAAE98wOUQpjmguyGTN2wd_89Srs/edit#heading=h.xabf9lffxrkb", "type": 301 }, | |||
{ "source": "/go/platform-view-focus", "destination": "https://docs.google.com/document/d/1GG8VfYeuVtTQHiMWQXbrSG5HEUorzqU8IGbNySVJ-bY/edit", "type": 301 }, |
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.
maybe add "ios"?
go/ios-platform-view-focus
Also, not sure if it matters, do you know what the /edit postfix does?
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.
or go/flutter-ios-platform-view-focus
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.
It's not consistent in this list, but remove the edit, LGTM
Co-authored-by: Jenn Magder <[email protected]>
Co-authored-by: Jenn Magder <[email protected]>
Once @jmagman's edits are incorporated, LGTM |
to add platform view focus doc
https://docs.google.com/document/d/1GG8VfYeuVtTQHiMWQXbrSG5HEUorzqU8IGbNySVJ-bY/edit
Issues fixed by this PR (if any):
flutter/flutter#34187
Presubmit checklist