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

idea: DTO for updating entry cache should also include number of entries #1824

Open
rmunn opened this issue May 28, 2024 · 0 comments
Open

Comments

@rmunn
Copy link
Collaborator

rmunn commented May 28, 2024

Is your feature request related to a problem? Please describe.
Sometimes the entry cache is incomplete, and Language Forge doesn't detect that situation. This can lead to users being afraid that they've lost data, because they only see 6 entries when there should be 31.

Describe the solution you'd like
We could have the entry update DTO send a count of the entries the server knows about. Then when the client has finished applying the update (including any new entries), check if the entry counts match. If they don't match, discard the current cache and re-download the entire dictionary from the server again.

Describe alternatives you've considered
The "reset browser cache" button that we've placed in the advanced configuration page allows redownloading the dictionary, but that's a manual process. We'd like to automatically catch the situation and fix it.

Additional context
#1024 is an older issue related to incomplete-cache problems; this may be a duplicate of that one, or it might turn out this is a different idea and #1024 should be kept around.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

1 participant