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.

Sign up for GitHub

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

Jump to bottom

Rewrite requestMIDIAccess to hook into permissions #220

Open
miketaylr opened this issue Oct 20, 2021 · 8 comments
Open

Rewrite requestMIDIAccess to hook into permissions #220

miketaylr opened this issue Oct 20, 2021 · 8 comments
Assignees
@hoch
Labels
class: substantive https://www.w3.org/2023/Process-20230612/#correction-classes status: ready for editing Enough information should be available to implement this change in the spec
Milestone
CR

Comments

@miketaylr
Copy link
Contributor

As a followup to #219 (review)
@miketaylr miketaylr mentioned this issue Oct 20, 2021
Closed
@cwilso
Copy link
Contributor

cwilso commented Oct 26, 2021

What remains on top of #294?

@miketaylr
Copy link
Contributor Author

Sorry for not writing a better description!

Essentially, https://webaudio.github.io/web-midi-api/#dom-navigator-requestmidiaccess needs to hook into the Permissions algorithms.

@marcoscaceres wrote some more explicit suggestions in #219 (review).

@miketaylr
Copy link
Contributor Author

I will attempt to not mess this up.

miketaylr added a commit to miketaylr/web-midi-api that referenced this issue Oct 27, 2021
@miketaylr
Issue WebAudio#220 - Link to freshly exported [=express permission=]
01d2c5e
miketaylr added a commit to miketaylr/web-midi-api that referenced this issue Oct 27, 2021
@miketaylr
Issue WebAudio#220 - Add software to MidiPermissionDescriptor
b0be5fc
miketaylr added a commit to miketaylr/web-midi-api that referenced this issue Oct 27, 2021
@miketaylr
Issue WebAudio#220 - MidiPermissionDescriptor replaces MIDIOptions
9cceb88
miketaylr added a commit to miketaylr/web-midi-api that referenced this issue Oct 27, 2021
@miketaylr
Issue WebAudio#220 - Link up some [=system exclusive=]s
3d823b6
miketaylr added a commit to miketaylr/web-midi-api that referenced this issue Oct 27, 2021
@miketaylr
Issue WebAudio#220 - Rewrite the requestMIDIAccess algorithm (part 1)
3feb652
@hoch
Copy link
Member

hoch commented Apr 6, 2023

Teleconference 4/6:
Remove "particularly if system exclusive access is requested." in the second paragraph of https://www.w3.org/TR/webmidi/#requestmidiaccess.

@hoch hoch self-assigned this Apr 6, 2023
@miketaylr
Copy link
Contributor Author

I will attempt to not mess this up.

(It looks like I never finished...)

@hoch
Copy link
Member

hoch commented Apr 6, 2023

No worries! We'll take it from here. :)

@miketaylr
Copy link
Contributor Author

Yay, thanks! I'll go ahead and close my WIP PR now.

@hoch hoch added the status: ready for editing Enough information should be available to implement this change in the spec label Sep 11, 2023
@mjwilson-google mjwilson-google added the class: substantive https://www.w3.org/2023/Process-20230612/#correction-classes label Sep 13, 2023
@mjwilson-google mjwilson-google added this to the CR milestone Sep 14, 2023
@mjwilson-google mjwilson-google added status: needs WG review Needs to be discussed with the Audio Working Group before proceeding and removed status: ready for editing Enough information should be available to implement this change in the spec labels Oct 15, 2023
@mjwilson-google
Copy link
Contributor

Audio WG 2023-10-18 conclusion:

@mjwilson-google mjwilson-google added status: ready for editing Enough information should be available to implement this change in the spec and removed status: needs WG review Needs to be discussed with the Audio Working Group before proceeding labels Oct 18, 2023
@mjwilson-google mjwilson-google mentioned this issue Jan 3, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hoch hoch

Labels
class: substantive https://www.w3.org/2023/Process-20230612/#correction-classes status: ready for editing Enough information should be available to implement this change in the spec
Projects
None yet
Milestone
CR
Development

No branches or pull requests
4 participants
@miketaylr @cwilso @hoch @mjwilson-google