mapbox / mapbox/mapbox-navigation-android
Telemetry bug: issues with `originalRequestIdentifier` and `requestIdentifier`
@RingerJK is already working on this.
Since Jan 24, 2022.
- Dominant language
- Kotlin
- Stars
- 651
- Forks
- 321
- PR merge metrics
- No merged PRs in 30d
Description
While trying to test, noticed that `requestSessionId` is inconsistent (probably unrelated to this PR but wanted to flag so we don't forget) 👀
_First re-route event_

the original request id and the request id are the same but should be different.
_Second re-route event_

the request ids are different, OK.
_Cancel event_

the request ids are different but the request id doesn't match the previous one (from the latest re-route event) 🤔
-----
ref https://github.com/mapbox/mapbox-navigation-android/pull/4693#pullrequestreview-737575430
cc @Guardiola31337
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.