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
feat(server,web): remove external path nonsense and make libraries admin-only #7237
Conversation
Deploying with Cloudflare Pages
|
…e/admin-only-library
…e/admin-only-library
…e/admin-only-library
…e/admin-only-library
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.
The server code LGTM. This is a nice change and should remove a lot of confusion!
Can we split the documentation to a separate PR that will be merged after the next release? |
That, or we do another release and merge this right before. |
I'll split the pr, that'll be the easiest |
…e/admin-only-library
Ready to merge, after the next release we merge #7526 |
There are still some docs changes in this PR |
Fixed, had forgot to push |
We make the library configuration admin-only. This removes a lot of complexity regarding external paths.
There is a new admin panel for external libraries. The configuration pane might be a better place for it, but it won't fit in a config file so I'm unsure.
User library settings have been removed.
We have a picker for library owner: