Skip to content

Remove v1 Android create templates#61203

Merged
xster merged 2 commits intoflutter:masterfrom
xster:remove-v1-templates
Jul 11, 2020
Merged

Remove v1 Android create templates#61203
xster merged 2 commits intoflutter:masterfrom
xster:remove-v1-templates

Conversation

@xster
Copy link
Member

@xster xster commented Jul 10, 2020

flutter.dev/go/android-migration-summary

Remove the creation template stubs for v1 embedding apps and plugins targeting v1 embeddings.

v1 usage ratio is now 26% for apps.

This PR removes the feature flag for app and plugin creations. It does not affect reporting which is determined independently. It does not affect current v1 embedding apps or v1 targeting plugins. Current v1 apps using current v1 targeting plugins will continue to generate the correct registrations. The generation is also determined independently.

The soft breaking change from this PR is that if a plugin developer creates a new plugin today and publishes it, v1 embedding app developers will see a prompt to update their app's embedding. Though the message and the trigger has been there all year, this increases the likelihood of that message being seen.

Breaking change website PR: flutter/website#4328

@fluttergithubbot fluttergithubbot added c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels. labels Jul 10, 2020
@xster xster marked this pull request as draft July 10, 2020 02:05
@xster xster requested review from blasten and jmagman July 10, 2020 19:39
@xster xster marked this pull request as ready for review July 10, 2020 19:39
@blasten
Copy link

blasten commented Jul 10, 2020

nice! I was gonna ping you about when to delete the old embedding from flutter/engine.

@xster
Copy link
Member Author

xster commented Jul 10, 2020

well that's probably gonna be the very last step

@xster
Copy link
Member Author

xster commented Jul 10, 2020

well, it's a chicken and egg issue. I need to know the time when this is merged so I'll merge this first then merge the breaking change announce website.

@xster xster merged commit 8bd2e65 into flutter:master Jul 11, 2020
@xster xster deleted the remove-v1-templates branch July 11, 2020 19:15
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants