Skip to content

Conversation

@queengooborg
Copy link
Contributor

This PR adds a feature for the Magnetometer API.

@github-actions github-actions bot added the feature definition Creating or defining new features or groups of features. label Sep 25, 2024
Copy link
Contributor

@autonome autonome left a comment

Choose a reason for hiding this comment

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

The description needs more info, also search features/draft/spec as there's some generated drafts there already which should be removed/updated as part of this change.

Co-authored-by: Daniel D. Beck <daniel@ddbeck.com>
Copy link
Collaborator

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

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

Looking at the other features with the "sensors" group, I think this is good to land for now.

@Elchi3 Elchi3 requested review from autonome and ddbeck October 10, 2024 10:39
@ddbeck
Copy link
Collaborator

ddbeck commented Oct 10, 2024

I'd still prefer to have something done about #1848 (comment) but I won't stand in the way of merging, if no one else cares.

@ddbeck
Copy link
Collaborator

ddbeck commented Nov 18, 2024

@queengooborg do you want to return to this? #1848 (comment) is the last bit outstanding here.

@ddbeck ddbeck removed their request for review November 18, 2024 13:13
@queengooborg
Copy link
Contributor Author

I think that the issue that was opened (#2299) is what is needed as a reminder to come back to this feature. I can add a comment, but I don't think it's necessary and this PR is ready to merge as-is.

@Elchi3 Elchi3 requested a review from ddbeck December 12, 2024 10:23
@ddbeck ddbeck merged commit 8525f90 into web-platform-dx:main Dec 12, 2024
3 checks passed
@queengooborg queengooborg deleted the magnetometer branch December 12, 2024 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature definition Creating or defining new features or groups of features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants