Engine: reorganize install topic#10479
Merged
usha-mandya merged 3 commits intodocker:masterfrom Apr 6, 2020
Merged
Conversation
thaJeztah
force-pushed
the
reorganize_engine_section
branch
from
761054f to
e51d09f
Compare
This was referenced Mar 20, 2020
thaJeztah
force-pushed
the
reorganize_engine_section
branch
4 times, most recently
from
2f3238d to
8e59149
Compare
- Move getting started overview to /get-started/overview/ - Move engine installation files under /engine/ - Redirect the top-level /install/ to /get-docker/ - Updated titles in left-hand navigation - Added back some pages to the navigation that were currently not included. - Reduce some steps in the installation pages - Move devicemapper prerequisites to the devicemapper storage driver page. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
thaJeztah
force-pushed
the
reorganize_engine_section
branch
from
8e59149 to
426f539
Compare
Member
Author
|
@usha-mandya updated this PR, this should be ready for review now |
usha-mandya
reviewed
Apr 6, 2020
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
thaJeztah
force-pushed
the
reorganize_engine_section
branch
from
426f539 to
569f448
Compare
usha-mandya
approved these changes
Apr 6, 2020
Member
usha-mandya
left a comment
usha-mandya
left a comment
There was a problem hiding this comment.
LGTM. Thank you @thaJeztah
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
rewrite the engine "overview" page
This page is currently not a good introduction to the engine, and overlaps with the "global" introduction to Docker / overview in the getting started: http://localhost:4000/engine/docker-overview/
This PR makes some changes to the page to reduce the overlap, but may still need some TLC
(note that that page's URL is currently under "engine" - probably will fix that in this PR as well, and move it to either/docker-overview/or/get-started/overview/)With this PR, the navigation looks like this: