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

Clarify how user search works #1525

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

weyfonk
Copy link

@weyfonk weyfonk commented Oct 25, 2024

This adds a few points which should shed light on how to search users using a drop-down list in Rancher, whether to add users to clusters or to projects.
Context: rancher/dashboard#12358.

Disclaimer: I have not found a way to test rendering for this, as I kept stumbling upon this issue when building contents locally.

Copy link
Contributor

@btat btat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just a few minor nits. Adding my review as "request changes" since this will need to be ported to other versions/languages of our docs. Let us know if help is needed with that.

This adds a few points which should shed light on how to search users
using a drop-down list in Rancher, whether to add users to clusters or
to projects.
This explains the difference that it makes as opposed to substring-based
search.
This improves formatting of notes on searching users, by making them
appear in notes blocks.
This applies suggestions from code review.
@weyfonk weyfonk force-pushed the document-user-search-behaviour branch from bf27827 to 1de75f7 Compare November 7, 2024 08:51
@weyfonk
Copy link
Author

weyfonk commented Nov 7, 2024

LGTM, just a few minor nits. Adding my review as "request changes" since this will need to be ported to other versions/languages of our docs. Let us know if help is needed with that.

Thanks for your feedback, I think I've addressed all of it.
Then come a couple of questions:

  • does porting to other versions mean submitting PRs to separate branches, such as v2.8.10 and v2.9.4? Or rather making similar changes to versioned_docs? Either way, I'm not entirely sure which versions of Rancher this applies to, beside the most recent minor.
  • how do translations work? I can see contents in i18n, the only directory there being zh so I'd definitely need help with that 😅

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.

2 participants