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

Don't require Same Site Cookies on accessibility app assets #12669

Merged
merged 1 commit into from
Nov 29, 2018

Conversation

tcitworld
Copy link
Member

@tcitworld tcitworld commented Nov 26, 2018

Follows #11878

(EDIT : Github web Editor is handy for tiny editions, but DCO doesn't like it of course.)

@tcitworld tcitworld added the 3. to review Waiting for reviews label Nov 26, 2018
@tcitworld tcitworld added this to the Nextcloud 15 milestone Nov 26, 2018
@tcitworld tcitworld requested review from rullzer and blizzz November 26, 2018 15:23
@rullzer
Copy link
Member

rullzer commented Nov 27, 2018

How is accessibility triggered on public pages? As this is a user setting I think.

@tcitworld
Copy link
Member Author

CSS is embed only when user is logged in, but JS is always here : https://github.com/nextcloud/server/blob/master/apps/accessibility/lib/AppInfo/Application.php

@rullzer
Copy link
Member

rullzer commented Nov 27, 2018

/backport to stable15

@backportbot-nextcloud
Copy link

backport to stable15 in #12731

@MorrisJobke
Copy link
Member

(EDIT : Github web Editor is handy for tiny editions, but DCO doesn't like it of course.)

It's easy - just add the "Signed-off-by: YOUR NAME [email protected]" to the commit message and it should be fine ;)

@tcitworld
Copy link
Member Author

@MorrisJobke Yeah, I might as well have a WebExtension doing that for me. :D

(and I'm not the only one todogroup/gh-issues#50 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants