-
Notifications
You must be signed in to change notification settings - Fork 136
Reconciles duplicate pages #2718
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
base: main
Are you sure you want to change the base?
Conversation
@shainaraskas -- what are your thoughts on all the pages within the User roles section (and other pages within the Deploy and manage section of the docs-content repo) that link to this page? Should they be updated to point to the page in the Reference section or is it okay to leave them as is? |
@yetanothertw I think the xlinks definitely need to be updated. here's what I'd do:
|
the reason I'd kill the page is that this page is too tiny and useless to be valuable, which means it would suck to land on it. we should prob always kill pages that are under 10 lines like this unless they disambiguate child pages. |
I do like your additions though ... if you want to, you can keep the page (would still prob recommend killing it) but I would change the xlinks because in most cases we're trying to draw their attention to a specific permission / the list of permissions |
My only concern about removing it entirely is the gap it leaves in the TOC: At the moment it's sort of a logical equivalent to the Kibana privileges page and if we remove it, it might be confusing, might look like a knowledge gap. 🤷♀️ |
Re diffing, it's the opposite, it seems there are additions in the page in the Reference section but not in Deploy and manage. |
@yetanothertw yeah that's why we would put the crosslink in (it would sit in the same spot) |
I also would move your additions over to the other copy of the page :) |
OK, I see what you mean.
|
) The [Elasticsearch privileges page in the Deploy and manage section](https://www.elastic.co/docs/deploy-manage/users-roles/cluster-or-deployment-auth/elasticsearch-privileges) is being replaced with its [counterpart in the Reference section](https://www.elastic.co/docs/reference/elasticsearch/security-privileges). This PR replaces all links from within the narrative docs to the correct (deduplicated and reconciled) page. Relates to #2718 and #2218
To remove duplication and ensure that information stays accurate, the Elasticsearch privileges page won't include detailed information about each privilege and their descriptions, and instead will refer users to Elasticsearch privileges in the Reference section.
Fixes #2218