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

Avoid using the permissions task source directly #144

Closed
johannhof opened this issue Dec 14, 2022 · 1 comment · Fixed by #184
Closed

Avoid using the permissions task source directly #144

johannhof opened this issue Dec 14, 2022 · 1 comment · Fixed by #184
Assignees
@johannhof
Labels
integration Integration with other specs resolve before graduation These issues need to be resolved before the spec graduates from the CG

Comments

@johannhof
Copy link
Member

As @annevk notes in w3c/permissions#407

I would kinda expect the flow to be that other specifications ask something from Permissions with a set of completion steps and Permissions queues a task and runs those completion steps at the end of that task.

We should remove our usage of the permissions task source once those changes are made in Permissions.
@annevk annevk added integration Integration with other specs resolve before graduation These issues need to be resolved before the spec graduates from the CG labels Dec 14, 2022
@johannhof johannhof mentioned this issue Dec 14, 2022
Merged
3 tasks
@mreichhoff mreichhoff mentioned this issue Jan 19, 2023
Open
@annevk annevk mentioned this issue Sep 4, 2023
Closed
@johannhof
Copy link
Member Author

I chatted about this again with @annevk and we think that might be able to use the network task source, with the reasoning that the updates should be queued alongside things affected by cookie changes. I'll go ahead and make this change, happy to revisit this decision if someone ends up having a strong opinion.

@johannhof johannhof self-assigned this Sep 4, 2023
johannhof added a commit to johannhof/storage-access that referenced this issue Sep 7, 2023
@johannhof
Switch to the networking task source
705b9ea 
Fixes privacycg#144, also fixes privacycg#182 on the side
@johannhof johannhof mentioned this issue Sep 7, 2023
Merged
johannhof added a commit that referenced this issue Oct 9, 2023
@johannhof
Switch to the networking task source (#184)
71a6071 
Fixes #144, also fixes #182 on the side
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johannhof johannhof

Labels
integration Integration with other specs resolve before graduation These issues need to be resolved before the spec graduates from the CG
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Switch to the networking task source johannhof/storage-access
2 participants
@annevk @johannhof