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

Clear cache and reload should probably explain what it actually does #5800

Open
pafcu opened this issue Dec 10, 2017 · 20 comments
Open

Clear cache and reload should probably explain what it actually does #5800

pafcu opened this issue Dec 10, 2017 · 20 comments
Assignees
Labels
P1 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@pafcu
Copy link
Contributor

pafcu commented Dec 10, 2017

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

  • Platform: web

For the web app:

  • Browser: Firefox 57
  • OS: Linux
  • URL: riot.im/develop
@lampholder lampholder added Help Wanted Extra attention is needed P1 ui/ux S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect and removed ui/ux labels Dec 12, 2017
@lampholder
Copy link
Member

Ah yes, this adds a nuance to #7539

@anoadragon453
Copy link
Member

image

It really should get its own section. I've also had reports of users hitting "Clear cache and reload" accidentally when they go to hit "Submit debug logs", which is probably not what we want if we want users to catch hard-to-find bugs.

@abhinav-TB
Copy link

abhinav-TB commented Jan 1, 2021

@aaronraimist I would like to work on this issue could you give some guidance

@aaronraimist
Copy link
Collaborator

@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 #element-dev:matrix.org room.

@jryans jryans removed the Z-UI/UX label Mar 9, 2021
@salmankhan432423

This comment has been minimized.

1 similar comment
@salmankhan432423

This comment has been minimized.

@tawfiek
Copy link

tawfiek commented Mar 24, 2021

I would like to help with this, can anybody Assign it to me ?!

@SimonBrandner
Copy link
Contributor

@tawfiek, done!

@tawfiek
Copy link

tawfiek commented Mar 24, 2021

I think it's fixed already !!
Do I still need to explain what this btn does ?!
Screenshot from 2021-03-24 12-12-23

@SimonBrandner
Copy link
Contributor

I think it's fixed already !!

Is it? I don't think I understand...

@tawfiek
Copy link

tawfiek commented Mar 24, 2021

image

It really should get its own section. I've also had reports of users hitting "Clear cache and reload" accidentally when they go to hit "Submit debug logs", which is probably not what we want if we want users to catch hard-to-find bugs.

compared to this view I think this issue have been solved somewhere :octocat:

@SimonBrandner
Copy link
Contributor

"Clear cache and reload" button in user settings actually does. What is stored in the cache? What happens if you clear it?

Yeah, but I don't think there is anything about this...

@ptman
Copy link
Contributor

ptman commented Feb 17, 2022

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?

@t3chguy
Copy link
Member

t3chguy commented Feb 17, 2022

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.

@ptman
Copy link
Contributor

ptman commented Feb 17, 2022

@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.

@t3chguy
Copy link
Member

t3chguy commented Feb 17, 2022

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.

@hifi
Copy link

hifi commented Feb 17, 2022

There are two clear cases for cache inconsistency that are user facing:

  • joining a room you're already on
  • leaving a room you've already left

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).

@Penguin-Guru
Copy link

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.

@t3chguy
Copy link
Member

t3chguy commented Apr 8, 2022

It would be nice if this could also warn users how long the operation could take.

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

@Penguin-Guru
Copy link

It would be nice if this could also warn users how long the operation could take.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.