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

US158420 - Add option to allow usage of clipboard in iframes #163

Merged
merged 3 commits into from
Oct 11, 2023

Conversation

EvanLemieux
Copy link
Contributor

@EvanLemieux EvanLemieux commented Oct 11, 2023

Enabling clipboard-write permission to allow us to use the clipboard API in folio-app

https://web.dev/articles/async-clipboard#permissions_policy_integration

Relevant LMS PR:
https://github.com/Brightspace/lms/pull/41825

@EvanLemieux EvanLemieux requested review from a team and grant-cleary October 11, 2023 15:54
Copy link

@eKoopmans eKoopmans left a comment

Choose a reason for hiding this comment

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

Neat!

@dlockhart
Copy link
Member

Can you add this to the README?

@dlockhart
Copy link
Member

I got a notification that you pushed a change to the README... but I'm not seeing it here in the review. Is GH bugging out?

@EvanLemieux
Copy link
Contributor Author

I got a notification that you pushed a change to the README... but I'm not seeing it here in the review. Is GH bugging out?

Yeah appears to be, looks like it's processing my push
image

@EvanLemieux
Copy link
Contributor Author

Anyone know if this is safe to merge? Or do I need to wait for Github to stop bugging out?

Copy link
Member

@dlockhart dlockhart left a comment

Choose a reason for hiding this comment

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

I just manually checked the README change and it LGTM... yolo and hope it all merges?

@EvanLemieux
Copy link
Contributor Author

Couldn't merge so pushed an empty commit and that seemed to kick it into gear. 🤷‍♂️

@EvanLemieux EvanLemieux merged commit 4232f93 into master Oct 11, 2023
1 check passed
@EvanLemieux EvanLemieux deleted the elemieux/add-clipboard-permission branch October 11, 2023 20:08
@ghost
Copy link

ghost commented Oct 11, 2023

🎉 This PR is included in version 0.41.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ghost ghost added the released label Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants