Skip to content

Telemetry polishing - #184

Merged
EmilianoSanchez merged 24 commits into
pluggable_storage_mtk_producer_apifrom
telemetry_polishing
Sep 29, 2022
Merged

Telemetry polishing#184
EmilianoSanchez merged 24 commits into
pluggable_storage_mtk_producer_apifrom
telemetry_polishing

Conversation

@EmilianoSanchez

Copy link
Copy Markdown
Contributor

Javascript commons library

What did you accomplish?

  • Update TelemetryCacheInMemory to mark it as empty when data is popped, in order to avoid POST requests with empty telemetry data on browser pagehide events.

How do we test the changes introduced in this PR?

  • Unit tests

Extra Notes

@EmilianoSanchez EmilianoSanchez changed the title fixed an issue with telemetry in Browser listener, not being marked a… Telemetry polishing Sep 27, 2022
@EmilianoSanchez
EmilianoSanchez changed the base branch from SDKS-5521-MTK-polishing to pluggable_storage_mtk_producer_api September 29, 2022 16:56
@EmilianoSanchez
EmilianoSanchez merged commit d7cd200 into pluggable_storage_mtk_producer_api Sep 29, 2022
@EmilianoSanchez
EmilianoSanchez deleted the telemetry_polishing branch September 29, 2022 16:57
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