Skip to content

Correct Chromium data for Bluetooth APIs and make consistent#11715

Merged
foolip merged 8 commits intomdn:mainfrom
queengooborg:api/Bluetooth/corrections
Aug 12, 2021
Merged

Correct Chromium data for Bluetooth APIs and make consistent#11715
foolip merged 8 commits intomdn:mainfrom
queengooborg:api/Bluetooth/corrections

Conversation

@queengooborg
Copy link
Contributor

@queengooborg queengooborg commented Jul 23, 2021

This PR is a continuation off of #10197. While I was updating some of the Bluetooth APIs, I realized that I hadn't made the other APIs' style consistent with our changes. Additionally, I noticed a few errors in the data that we had. This PR corrects the erroneous data and makes our styling consistent across all of the Bluetooth APIs when representing the OS-specific support.

Fixes #11532.

@github-actions github-actions bot added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Jul 23, 2021
@queengooborg queengooborg requested a review from foolip July 25, 2021 15:52
Copy link
Contributor

@foolip foolip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I sent #11947 and started suggesting reverting the WebView changes, but there are too many. Can you search-replace them? Landing #11947 first will make it trivial to see if all are fixed, since the lint would fail otherwise.

queengooborg and others added 3 commits August 11, 2021 20:57
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
"chrome": [
{
"version_added": "56",
"notes": "macOS only."
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is adding the old style structure. I'll fix in in a follow-up PR.

},
"chrome": [
{
"version_added": "56",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed with https://mdn-bcd-collector.appspot.com/tests/api/BluetoothUUID on macOS. 70 was wrong.

@foolip foolip merged commit 16a1b9a into mdn:main Aug 12, 2021
@queengooborg queengooborg deleted the api/Bluetooth/corrections branch August 12, 2021 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make Bluetooth API OS support notes consistent across files

2 participants