Skip to content

Conversation

hannah-hyj
Copy link
Member

@hannah-hyj hannah-hyj commented Dec 13, 2024

Add doc about the breaking change and migration guide.
Related PR: flutter/engine#52350

Presubmit checklist

  • This PR is marked as draft with an explanation if not meant to land until a future stable release.
  • This PR doesn’t contain automatically generated corrections (Grammarly or similar).
  • This PR follows the Google Developer Documentation Style Guidelines — for example, it doesn’t use i.e. or e.g., and it avoids I and we (first person).
  • This PR uses semantic line breaks of 80 characters or fewer.

@hannah-hyj hannah-hyj requested review from a team, parlough and sfshaza2 as code owners December 13, 2024 20:55
Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

Thanks, @hannah-hyj! A few things to fix.

@sfshaza2
Copy link
Contributor

Also, @hannah-hyj, can you add this to the index file?


* Android Manifest file

```dart
Copy link
Contributor

Choose a reason for hiding this comment

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

@hannah-hyj, I believe the build is failing because you label this code as dart, when it's yaml. Can you fix this and also add "yaml" to the iOS example below. If I'm right, this should fix the website build.

@sfshaza2 sfshaza2 added the review.await-update Awaiting Updates after Edits label Dec 13, 2024
@flutter-website-bot
Copy link
Collaborator

flutter-website-bot commented Dec 13, 2024

Visit the preview URL for this PR (updated for commit 0687357):

https://flutter-docs-prod--pr11493-deeplinkflag-breakingchange-rs4js96k.web.app

Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

At last, the build passes! :D LGTM. (And I meant to say that if you put the links inline you use regular parenthetical brackets, not square brackets, nor curly brackets. :)

@sfshaza2 sfshaza2 merged commit ba95a54 into main Dec 14, 2024
9 checks passed
@sfshaza2 sfshaza2 deleted the deeplinkflag-breakingchanges branch December 14, 2024 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review.await-update Awaiting Updates after Edits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants