Skip to content

feat: update example to use Navigator 2.0 #3385

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

loqmanali
Copy link

This commit updates the main example to use the Navigator 2.0 API from GetX. This makes the example more modern and consistent with the example_nav2 project.

The following changes were made:

  • example/lib/main.dart was updated to use GetMaterialApp.router with a routerDelegate and routeInformationParser.
  • The commented-out main function in example/lib/main.dart was removed.
  • example/lib/routes/app_pages.dart was updated to be compatible with the new navigation API.

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.

Every PR must update the corresponding documentation in the code, and also the readme in english with the following changes.

Pre-launch Checklist

  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making or feature I am adding, or @jonataslaw said the PR is test-exempt.
  • All existing and new tests are passing.

This commit updates the main example to use the Navigator 2.0 API from GetX. This makes the example more modern and consistent with the `example_nav2` project.

The following changes were made:

*   `example/lib/main.dart` was updated to use `GetMaterialApp.router` with a `routerDelegate` and `routeInformationParser`.
*   The commented-out `main` function in `example/lib/main.dart` was removed.
*   `example/lib/routes/app_pages.dart` was updated to be compatible with the new navigation API.
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.

1 participant