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

[PRD-600] feat: Use local search only for long-term sessions #2611

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

paultranvan
Copy link
Contributor

We now rely on the checkbox on login to determine whether or not the local search should be used. If the user enable this checkbox, it means he trusts this device and accept to store local data on it. Otherwise, we do not store any private data, and directly query the stack to build the search index.

Note the update mechanism is the same and rely on realtime. The difference is that we do not replicate the document for non local search.

@paultranvan paultranvan requested a review from acezard as a code owner November 7, 2024 11:29
@paultranvan paultranvan requested review from Ldoppea and removed request for acezard November 7, 2024 11:29
paultranvan added a commit to cozy/cozy-web-data-proxy that referenced this pull request Nov 7, 2024
We now rely on the checkbox on login to determine whether or not the
local search should be used. If the user enable this checkbox, it means
he trusts this device and accept to store local data on it.
Otherwise, we do not store any private data.

See cozy/cozy-libs#2611 for the search part.
@paultranvan paultranvan force-pushed the localAndRemoteSearch branch 7 times, most recently from 1a2caac to 9f8274f Compare November 8, 2024 09:24
We now rely on the checkbox on login to determine whether or not the
local search should be used. If the user enable this checkbox, it means
he trusts this device and accept to store local data on it.
Otherwise, we do not store any private data, and directly query the
stack to build the search index.

Note the update mechanism is the same and rely on realtime. The
difference is that we do not replicate the document for non local
search.
@paultranvan paultranvan merged commit dc2b38e into master Nov 8, 2024
2 checks passed
@paultranvan paultranvan deleted the localAndRemoteSearch branch November 8, 2024 09:51
paultranvan added a commit to cozy/cozy-web-data-proxy that referenced this pull request Nov 8, 2024
We now rely on the checkbox on login to determine whether or not the
local search should be used. If the user enable this checkbox, it means
he trusts this device and accept to store local data on it.
Otherwise, we do not store any private data.

See cozy/cozy-libs#2611 for the search part.
paultranvan added a commit to cozy/cozy-web-data-proxy that referenced this pull request Nov 8, 2024
We now rely on the checkbox on login to determine whether or not the
local search should be used. If the user enable this checkbox, it means
he trusts this device and accept to store local data on it.
Otherwise, we do not store any private data.

See cozy/cozy-libs#2611 for the search part.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant