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

Move guides to new docs_site #4

Closed
wants to merge 5 commits into from
Closed

Move guides to new docs_site #4

wants to merge 5 commits into from

Conversation

jgayfer
Copy link
Owner

@jgayfer jgayfer commented May 18, 2018

This PR depends on solidusio#2739

This PR moves all of the existing documentation to the new docs site.

There are a few changes that had to be made to the docs in order to have them play nice with Middleman.

  • Change the extension of each document from .md to .html.md
  • Change relative links in each document from .md to .html

Other than what is mentioned above, there are no changes made to the docs themselves. The goal was to keep the docs as pure markdown to ensure they remain platform agnostic should they be used in a different capacity in the future.

@jgayfer jgayfer force-pushed the docs_site branch 2 times, most recently from 73a198c to 30fb811 Compare May 18, 2018 21:27
@jgayfer jgayfer force-pushed the move_docs branch 2 times, most recently from b842470 to 81d257d Compare May 18, 2018 21:35
James Gayfer added 5 commits May 18, 2018 15:16
The docs index wasn't meant to be a long term solution; it was a means
to aid in navigating the docs while on GitHub. Since the docs will now
live in their own site, there won't be a need to navigate the docs on
GitHub.
This is a requirement of Middleman, the platform the docs site is built
on. Otherwise, we can't generate HTML from the markdown.
This change allows for relative links to work in the context of the docs
site.
This change keeps the original top level file/directory names as we
had previously.
@jgayfer
Copy link
Owner Author

jgayfer commented May 22, 2018

Test PR no longer needed.

@jgayfer jgayfer closed this May 22, 2018
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.

1 participant