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: navigator.connection not working as intended #38907

Merged
merged 1 commit into from Jun 23, 2023

Conversation

ckerr
Copy link
Member

@ckerr ckerr commented Jun 22, 2023

Backport of #38491.

See that PR for details.

No interesting changes relative to 38491. Manual intervention was required because of minor code shear from #38301.

Notes: Fixed an issue where navigator.connection returned incorrect data.

CC @codebytere

* fix: navigator.connection not working as intended

* chore: make network quality methods private
@ckerr ckerr added the backport This is a backport PR label Jun 22, 2023
@trop trop bot added 25-x-y semver/patch backwards-compatible bug fixes labels Jun 22, 2023
@ckerr ckerr added target/23-x-y PR should also be added to the "23-x-y" branch. target/24-x-y PR should also be added to the "24-x-y" branch. labels Jun 22, 2023
@codebytere codebytere merged commit 679f473 into 25-x-y Jun 23, 2023
15 checks passed
@codebytere codebytere deleted the fix-navigator-connection--25-x-y branch June 23, 2023 08:05
@release-clerk
Copy link

release-clerk bot commented Jun 23, 2023

Release Notes Persisted

Fixed an issue where navigator.connection returned incorrect data.

@trop
Copy link
Contributor

trop bot commented Jun 23, 2023

I was unable to backport this PR to "23-x-y" cleanly;
you will need to perform this backport manually.

@trop trop bot removed the target/23-x-y PR should also be added to the "23-x-y" branch. label Jun 23, 2023
@trop
Copy link
Contributor

trop bot commented Jun 23, 2023

I was unable to backport this PR to "24-x-y" cleanly;
you will need to perform this backport manually.

@trop trop bot removed the target/24-x-y PR should also be added to the "24-x-y" branch. label Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
25-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants