Repository navigation
docs(android): Add Navigation 3 integration docs - #19804
0xadam-brown wants to merge 1 commit into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
This comment was marked as resolved.
This comment was marked as resolved.
5a63904 to
cab4b4f
Compare
Document Navigation 3 support on the Android navigation integration page and clarify how it differs from the existing Navigation 2 integration. Align the examples with the current SDK API shape, including route name extraction, optional argument extraction, back stack capture options, and route data scrubbing.
cab4b4f to
53c392e
Compare
|
The plan checks request volume on the updated Navigation docs page, client errors on that page, and Android not-found counts. An issue escalates when those errors rise or not-found counts for Android docs rise. Services: Mention PlanWhat changedRequests to sentry-docs must deploy before this page is live. The site prebuilds pages, so a production deploy ships the page. RiskA broken MDX file can drop this page from the static set. Readers then get a not-found page. New headings, code tabs, and images can throw in the page renderer. The new Compose Navigation link can point at a missing page. Intended effectThe new Navigation 3 copy is not queryable. Span volume on this URL shows that the rebuilt page is served. Absent looks like that span count at zero after deploy.
Regression watchThis change can 404 the Navigation page or throw in its renderer. It can also break the linked Compose Navigation page.
Not observableThe Navigation 3 heading text and screenshot pixels are not queryable. Page-level LCP and CLS for this URL are not queryable. The metrics API rejected a URL filter. Whether package |
Warning
do not merge before sentry-jave 8.60.0 release
DESCRIBE YOUR PR
Documents Android Navigation 3 (aka, "Nav3") support and clarifies how it differs from the existing Navigation 2 integration.
Also updates the Android features page so our Nav3 docs are easier to discover from high-level SDK feature docs.
Nav3 support was implemented by the PR stack rooted at sentry-java #6128, and later modified by sentry-java #6197 and sentry-java #6201.
IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
Select exactly one option. For deadlines, replace
YYYY-MM-DDwith the due date. You can update this information later by editing the PR description.We have a soft release coming up on October 7. Not extremely urgent, but all the better if we're able to get these docs published as soon as we can after that date. Thanks in advance! 🙌
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes:
LEGAL BOILERPLATE
Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.
EXTRA RESOURCES