-
Notifications
You must be signed in to change notification settings - Fork 184
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
[QA] Accounts page does not finish loading #530
Comments
This is a known issue, but since we don't have a changelog for ocis, it is not visible. https://github.com/owncloud/ocis/blob/master/changelog/unreleased/permission-checks-in-ocis-accounts.md |
We decided to rather have this visually broken state and have permission checks on the accounts API endpoints, instead of not having permission checks and the accounts UI accessible for any user. There are plans for hiding the accounts UI for users that don't have account management permissions. It is a stretch goal of the current sprint and tracked in this ticket: owncloud/product#167 |
Account page loads fine for admin user moss. But is readonly. |
The role assignment should work already. Features for enabling and disabling users as well as deleting and creating users are part of the current sprint (already implemented, currently bubbling up into ocis). |
Behavior from the open post keeps happening when user is not an admin user, so i guess we should do: 1 - Do not show accounts section (button) for non-admin users |
Closing here as the main issue is fixed. Thanks! The remaining part is in #804 |
Seen with docker run --rm -ti -p 9200:9200 owncloud/ocis:1.0.0-rc1 and
ocis-1.0.0-rc1 via https://github.com/owncloud-docker/compose-playground/blob/master/examples/hetzner-deploy/make_ocis_eos_compose_test.sh
An ever loading blue bar is shown.
Log snippet when visiting the URL with shift reload:
The text was updated successfully, but these errors were encountered: