Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Misleading documentation for user_directory.search_all_users #13852

Closed
babolivier opened this issue Sep 20, 2022 · 8 comments · Fixed by #14818
Closed

Misleading documentation for user_directory.search_all_users #13852

babolivier opened this issue Sep 20, 2022 · 8 comments · Fixed by #14818
Labels
A-Docs things relating to the documentation good first issue Good for newcomers O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Blocks non-critical functionality, workarounds exist. T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.

Comments

@babolivier
Copy link
Contributor

The documentation says:

Defines whether to search all users visible to your HS when searching the user directory.

It does not make it clear that the users included in that cohort art the users visible by the homeserver at the time the search is performed, i.e. all users who share a room with a user from the homeserver at that time. As opposed to all users the homeserver has ever seen, regardless of whether they share a membership in a room.

While we're doing things around this part of the doc, it has a link to https://matrix-org.github.io/synapse/latest/usage/administration/admin_api/background_updates.html#run which hasn't been linkified - it should probably be turned into a relative link (so that version prefixes in URLs carry over).

@babolivier babolivier added A-Docs things relating to the documentation S-Minor Blocks non-critical functionality, workarounds exist. T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. O-Occasional Affects or can be seen by some users regularly or most users rarely good first issue Good for newcomers labels Sep 20, 2022
@exitflynn
Copy link
Contributor

exitflynn commented Sep 21, 2022

i'd love to work on this issue!
edit: apologies, anyone else feel free to work on this! 🤠

@babolivier
Copy link
Contributor Author

babolivier commented Sep 22, 2022

Feel free to open a PR for it :)

@Abhijnan-Bajpai
Copy link

If this issue is yet to be resolved, could I work on this?

@tjay27
Copy link
Contributor

tjay27 commented Jan 9, 2023

While we're doing things around this part of the doc, it has a link to https://matrix-org.github.io/synapse/latest/usage/administration/admin_api/background_updates.html#run which hasn't been linkified - it should probably be turned into a relative link (so that version prefixes in URLs carry over).

At line 109 from background_updates.md

image
Here, the link to the user directory seems already relative.

Is there something extra I need to add for version prefix carry over? @clokep @babolivier

@DMRobertson
Copy link
Contributor

Brendan was referring to

These indexes are built the first time Synapse starts; admins can
manually trigger a rebuild via the API following the instructions
[for running background updates](../administration/admin_api/background_updates.md#run),
set to true to return search results containing all known users, even if that
user does not share a room with the requester.
, but I'm not sure what he meant by linkified. Looks like it's already a relative URL to me?

@tjay27
Copy link
Contributor

tjay27 commented Jan 9, 2023

Is this right?

If user_directory's search_all_users is

true - searches all users that share a room with the homeserver at that time.
false - search results will only contain users visible in public rooms and users sharing a room with the requester at that time

@tjay27
Copy link
Contributor

tjay27 commented Jan 9, 2023

@DMRobertson
It shows the URL there instead of for running background updates as mentioned in markdown.
That is what he meant by linkify

image

@DMRobertson
Copy link
Contributor

Looks like #14003 fixed the link.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Docs things relating to the documentation good first issue Good for newcomers O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Blocks non-critical functionality, workarounds exist. T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants