-
Notifications
You must be signed in to change notification settings - Fork 202
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
Update the recent search disclaimer #4824
Conversation
I think "with anyone" is implied by "never shared", and it doesn't read awkwardly to me, but I'm not a copy editor, so maybe it does not scan correctly to end without specifying the object of the sharing. Regardless of the text we end up with, I would urge brevity as much as possible, and if we feel the need to clarify anything, we can turn the text into a link to the privacy page. "with anyone" might also be technically incorrect. We do send |
This sounds great to me. I agree with keeping the sentence short. |
48e76e6
to
ce72642
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Fixes
Fixes #4741 by @sarayourfriend
Description
This PR updates the text of the recent searches privacy disclaimer.
The current sentence feels unfinished to me, though. Would adding "with anyone" make it clearer, @WordPress/openverse-maintainers?
"Recent searches are saved on your device and are not shared with anyone"
Testing Instructions
Checklist
Update index.md
).main
) or a parent feature branch.ov just catalog/generate-docs
for catalogPRs) or the media properties generator (
ov just catalog/generate-docs media-props
for the catalog or
ov just api/generate-docs
for the API) where applicable.Developer Certificate of Origin
Developer Certificate of Origin