-
Notifications
You must be signed in to change notification settings - Fork 117
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
Load viewer scripts depending on share ownership #2745
Conversation
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
e2e tests for this would also be nice to have, but require some more effort. I'll try to work on that on the sides a bit but would be good to get this in already to fix this issue. |
/backport to stable25 |
/backport to stable24 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did not try it but looks good.
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
32e2801
to
bb89b79
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
/backport to stable25 |
/backport to stable24 |
The backport to stable25 failed. Please do this backport manually. |
The backport to stable24 failed. Please do this backport manually. |
Summary
This changes makes sure that the viewer integration will no longer be loaded for all share links but only for those where the share owner should have access to office. This only applies if office is restricted to by only used by certain groups.
Checklist