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

Shortens Scenario Guides, incorporates vmware guide #54554

Merged
merged 2 commits into from
Apr 4, 2019

Conversation

acozine
Copy link
Contributor

@acozine acozine commented Mar 28, 2019

SUMMARY

Supersedes #53569.

Divides the Scenario Guides into 3 groups: cloud, network, and virtualization/containerization.
Incorporates VMware content into Scenario Guides section.
Removes unused VMware pages from the VMware guide.
Updates main TOC and left nav bar for cleaner navigation.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

docs.ansible.com: vmware guide, other Scenario Guides (for cloud platforms, network technologies, and virtualization/containerization solutions).

@acozine acozine added docs This issue/PR relates to or includes documentation. docsite This issue/PR relates to the documentation website. labels Mar 28, 2019
@ansibot ansibot added affects_2.8 This issue/PR affects Ansible v2.8 cloud core_review In order to be merged, this PR must follow the core review workflow. docker support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Mar 28, 2019
Copy link
Contributor

@gundalow gundalow left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor

@gundalow gundalow left a comment

Choose a reason for hiding this comment

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

Looks good. Once this has been merged please remind me to update BOTMETA.yml

@acozine acozine added the vmware VMware community label Mar 29, 2019
@acozine
Copy link
Contributor Author

acozine commented Mar 29, 2019

Temporarily published to http://docs.testing.ansible.com/ansible/latest/index.html

@acozine
Copy link
Contributor Author

acozine commented Mar 29, 2019

@gundalow hmmm, we don't have a page called index.html for the scenario guides section now. I retained the guides.html page in this PR because that page exists now (https://docs.ansible.com/ansible/latest/scenario_guides/guides.html), so keeping it will help with version-switching.

I don't know how many hits the guides.html page gets now. To navigate to the missing index.html page now, you have to type in the URL.

@gundalow
Copy link
Contributor

gundalow commented Mar 29, 2019

I don't think we need index.html, a server side redirect from / to guides.html should be fine

@gundalow
Copy link
Contributor

gundalow commented Apr 2, 2019

After merging we need to update BOTMETA to add labels to the VMware section

@acozine acozine merged commit a4d0bc2 into ansible:devel Apr 4, 2019
@acozine acozine deleted the scenario_guides_update branch April 22, 2019 16:18
@acozine acozine mentioned this pull request May 1, 2019
@ansible ansible locked and limited conversation to collaborators Jul 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.8 This issue/PR affects Ansible v2.8 cloud core_review In order to be merged, this PR must follow the core review workflow. docker docs This issue/PR relates to or includes documentation. docsite This issue/PR relates to the documentation website. support:core This issue/PR relates to code supported by the Ansible Engineering Team. vmware VMware community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants