-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Clear cache and reload should probably explain what it actually does #5800
Comments
Ah yes, this adds a nuance to #7539 |
@aaronraimist I would like to work on this issue could you give some guidance |
@abhinav-TB This is where the button is created in code. https://github.com/matrix-org/matrix-react-sdk/blob/4f52afdb3c15b5c46cf8a00f0b971637e1147148/src/components/views/settings/tabs/user/HelpUserSettingsTab.js#L273-L277 You can add a sentence or two explaining what clear cache does right above it. It would look something like https://github.com/matrix-org/matrix-react-sdk/blob/4f52afdb3c15b5c46cf8a00f0b971637e1147148/src/components/views/settings/tabs/user/HelpUserSettingsTab.js#L171-L174 If you need more help feel free to join the |
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
I would like to help with this, can anybody Assign it to me ?! |
@tawfiek, done! |
Is it? I don't think I understand... |
compared to this view I think this issue have been solved somewhere |
Yeah, but I don't think there is anything about this... |
I don't know if this is the right place. Please direct me to a more suitable issue. Is it possible to detect when the local state ("cache") is in conflict with what the server thinks is true and do a local state rebuild ("clear cache and reload") automatically? |
Not without basically having to constantly initial sync (same thing the clear cache & reload does) with the server and comparing, this operation is really expensive on the server and processing that much JSON will likely also slow down your user experience. |
@t3chguy Ok, let me rephrase: I'm not looking for a way to make sure the cache is always known to be good. But if some operation reveals that the cache is indeed out-of-sync. It would help even if it wasn't perfect. |
You can of course write heuristics, but getting a good heuristic for this which doesn't have too many false positives will probably be an artform in itself. You're guessing at best, unless you ask the server for what it thinks and comparing, which will increase server load a lot. |
There are two clear cases for cache inconsistency that are user facing:
These can't be resolved with any other mean than clearing the cache AFAIK so when the client and server disagree on this the user could be presented with a choice: "Seems like your client state is not in sync with the server, we can reset your client automatically but it will take a while, proceed?" (rephrased better). |
It would be nice if this could also warn users how long the operation could take. I'm running one right now that's over an hour in. The window has been pure white with the same spinner in the center the entire time, so I assume it's working. |
Well the warning would have to be "If your server has insufficient resources it might take longer than you remain alive" given there's no way to know how long it will take |
That sounds good to me. In my case, I had to kill the process after letting it run for about 26 hours. It seems there was a malfunction but I can't be sure. |
Description
Currently there is no explanation what the "Clear cache and reload" button in user settings actually does. What is stored in the cache? What happens if you clear it? Currently the button is red, which indicates it might be dangerous, but the user has no way of deciding if they want to "take the risk". As per #5724 (comment) it could be made yellow/orange instead, but some explanation of what the button does, and why you should be careful with it, would be nice to have.
Version information
For the web app:
The text was updated successfully, but these errors were encountered: