Skip to content

Updating the Android deployment page#13242

Open
sfshaza2 wants to merge 1 commit intomainfrom
fix-12137
Open

Updating the Android deployment page#13242
sfshaza2 wants to merge 1 commit intomainfrom
fix-12137

Conversation

@sfshaza2
Copy link
Copy Markdown
Contributor

@sfshaza2 sfshaza2 commented Apr 1, 2026

Fixes #12137

@sfshaza2 sfshaza2 requested a review from jwill April 1, 2026 02:02
@sfshaza2 sfshaza2 requested a review from a team as a code owner April 1, 2026 02:02
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Android deployment documentation by refining terminology, updating link references, and adding instructions for updating the application ID and namespace. The review feedback identifies a broken Markdown link reference caused by a label mismatch and suggests expanding the application ID update example to include Java file paths for better completeness.

[flutter_launcher_icons]: {{site.pub}}/packages/flutter_launcher_icons
[launchericons]: {{site.material}}/styles/icons
[configuration qualifiers]: {{site.android-dev}}/guide/topics/resources/providing-resources#AlternativeResources
[config qual]: {{site.android-dev}}/guide/topics/resources/providing-resources#AlternativeResources
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The link reference label [config qual] does not match the identifier [config-qual] used on line 50. Markdown link references must match exactly (including spaces and hyphens) to resolve correctly.

Suggested change
[config qual]: {{site.android-dev}}/guide/topics/resources/providing-resources#AlternativeResources
[config-qual]: {{site.android-dev}}/guide/topics/resources/providing-resources#AlternativeResources

Comment on lines +499 to +502
For example, if your new ID is `com.example.myapp`,
move your `MainActivity` file to
`android/app/src/main/kotlin/com/example/myapp/MainActivity.kt`
and ensure the first line is `package com.example.myapp`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Since the instructions mention both Kotlin and Java files, it's helpful to provide the example path for both languages to ensure clarity for all users.

Suggested change
For example, if your new ID is `com.example.myapp`,
move your `MainActivity` file to
`android/app/src/main/kotlin/com/example/myapp/MainActivity.kt`
and ensure the first line is `package com.example.myapp`.
For example, if your new ID is com.example.myapp,
move your MainActivity file to
android/app/src/main/kotlin/com/example/myapp/MainActivity.kt
(or android/app/src/main/java/com/example/myapp/MainActivity.java)
and ensure the first line is package com.example.myapp.

@flutter-website-bot
Copy link
Copy Markdown
Collaborator

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

https://flutter-docs-prod--pr13242-fix-12137-b6100ycj.web.app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Application Building forever

2 participants