-
Notifications
You must be signed in to change notification settings - Fork 111
[WIP] Split up _topics/automation_management_providers #1854
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: master
Are you sure you want to change the base?
Conversation
03654cc
to
1984480
Compare
Okay @Fryguy this is ready to review |
- title: Ansible Tower | ||
path: "/docs/reference/latest/managing_providers/automation_management_providers/ansible_tower.html" | ||
desc: "Managing Ansible Tower Providers" |
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.
Should this be Ansible Automation Platform now?
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.
Also do we have AWX? Or perhaps this should be Ansible Automation Platform / AWX?
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.
Doesn't have to be for this PR - the existing stuff is all Ansible Tower anyway, and this is a clean reorg
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.
Yeah we can definitely do that, I was avoiding moving&changing in the same PR but that makes sense
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.
Agreed - let's make it a different PR
We might need "prior" values for the reverse linkages and canonical URLs, but honestly I'm not sure as it might break things |
@agrare What does the "Radjabov" link link to on, say, the Ansible Tower page. |
1984480
to
875891b
Compare
Currently all automation providers are in one big `managing_providers/_topics/automation_management_providers.md` file where the other manager types each have their own dedicated pages.
875891b
to
8413864
Compare
Checked commit agrare@8413864 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.62.0, and yamllint **
|
This pull request is not mergeable. Please rebase and repush. |
3aad695
to
530fdad
Compare
107b1b1
to
78836a2
Compare
78836a2
to
4fbd352
Compare
Currently all automation providers are in one big
managing_providers/_topics/automation_management_providers.md
file where the other manager types each have their own dedicated pages.TODO
managing_providers/_topics
undermanaging_providers/automation_managers/_topics