Skip to content
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

Fix NewRelicNavigationObserver class #83

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MiguelBelotto00
Copy link

@MiguelBelotto00 MiguelBelotto00 commented Apr 18, 2024

According to what was discussed in the #68, it is only necessary to verify the names of the routeSettings type since the parameters of go router / router such as MaterialPage and CustomTransitionPage use the name parameter of the routeSettings as a child to send the information.

Changed

  • In the file lib/newrelic_navigation_observer.dart the _addGoRouterBreadcrumb function is no longer used and only the _addBreadcrumb function is used.
  • Readme.md file for the new version according to semver
  • Update version in every file.

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.

None yet

8 participants