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

[admin-settings] "Unsaved changes" shown without doing any changes #8548

Closed
hurradieweltgehtunter opened this issue Mar 7, 2023 · 1 comment · Fixed by #8555
Closed

[admin-settings] "Unsaved changes" shown without doing any changes #8548

hurradieweltgehtunter opened this issue Mar 7, 2023 · 1 comment · Fixed by #8555
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working

Comments

@hurradieweltgehtunter
Copy link
Contributor

Steps to reproduce

  1. Go to admin settings
  2. Click on pen icon ("edit user") in table row (not right click -> edit)
  3. Right sidebar opens

Expected behaviour

No hint or "No changes" is displayed.

Actual behaviour

Hint "Unsaved changes" is displayed.

The described behaviour seems not to be consistent. I tried different click paths and sometimes the behaviour changes ("No changes" is displayed). It also differs for example if I right click on user -> edit -> or right click -> details -> edit user --> "No changes" is displayed.

Another action when this happens (sometimes):

  1. Go to admin settings
  2. Click on pen icon in table row (not right click -> edit)
  3. Right sidebar opens
  4. add or remove a group to the user
  5. Click "save"
  6. Click on another user
  7. "Unsaved changes" is displayed.

Environment general

https://ocis.ocis-wopi.latest.owncloud.works

ownCloud Infinite Scale
Version: 2.0.0+2bf458385
Web-Client Version: 7.0.0-rc.16

@hurradieweltgehtunter hurradieweltgehtunter added the Type:Bug Something isn't working label Mar 7, 2023
@JammingBen
Copy link
Contributor

Phew, good find!

Some technical background, whoever tackles this issue: We load additional data when selecting (=clicking) a user. I assume that sometimes this data is being loaded when the sidebar panel is displayed (good), and sometimes it's not yet loaded (bad). In that case we need to implement a proper loading state.

This also explains why this issue is not happening via context menu -> edit: there is enough time to fully load the additional data when going that route.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants