Skip to content

Add Wear navigation 3 snippets#922

Open
ithinkihaveacat wants to merge 4 commits into
android:mainfrom
ithinkihaveacat:migrate-nav3-snippets
Open

Add Wear navigation 3 snippets#922
ithinkihaveacat wants to merge 4 commits into
android:mainfrom
ithinkihaveacat:migrate-nav3-snippets

Conversation

@ithinkihaveacat
Copy link
Copy Markdown
Contributor

@ithinkihaveacat ithinkihaveacat commented May 15, 2026

The Migration.kt looks a bit weird because it's composed of very short before/after code fragments.

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 introduces Navigation 3 to the Wear OS module by adding the necessary dependencies and implementing a sample application that demonstrates screen navigation and ViewModel integration. A review comment suggests moving the hardcoded version of the compose-navigation3 dependency into the version catalog to maintain consistency with the project's dependency management.

Comment thread wear/build.gradle.kts Outdated
- Create Migration.kt to house snippets for the migrate-to-navigation3.md guide.

- Include both Navigation 2 and Navigation 3 code examples for comparison.

- Ensure all snippets compile successfully against minSdk 33.
@ithinkihaveacat ithinkihaveacat marked this pull request as ready for review May 15, 2026 01:32
@ithinkihaveacat ithinkihaveacat requested review from a team, kkuan2011 and yrezgui as code owners May 15, 2026 01:32
@snippet-bot
Copy link
Copy Markdown

snippet-bot Bot commented May 15, 2026

Here is the summary of changes.

You are about to add 9 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@ithinkihaveacat ithinkihaveacat enabled auto-merge (squash) May 15, 2026 08:50
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.

2 participants