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

Docusaurus versioning #2

Merged
merged 27 commits into from
Jul 20, 2023
Merged

Docusaurus versioning #2

merged 27 commits into from
Jul 20, 2023

Conversation

samricotta
Copy link
Contributor

Within in the sdk repo, versioning is split between vuepress and docusaurus. This PR migrates all versions over to this repo whilst using the custom plugin to build the docs per version.

This PR is to be merged before #1.

There will be another PR that will live in the sdk repo to copy over any changes on MD files using a GitHub Actions workflow which is triggered on any deployment.

@samricotta samricotta force-pushed the sam/versioning branch 2 times, most recently from d241d00 to b888dbf Compare May 10, 2023 14:09
@julienrbrt
Copy link
Member

This PR is based on #1 but the description says it should be merged before #1, is that a typo?

@samricotta
Copy link
Contributor Author

This PR is based on #1 but the description says it should be merged before #1, is that a typo?

I think it will be best to follow your suggestion and configure without domain so I dont think my comment here matters. It was only that I wanted everything set up before going live.

@samricotta samricotta changed the base branch from sam/docs-revamp to main June 11, 2023 21:33
@samricotta samricotta marked this pull request as ready for review June 11, 2023 21:36
@samricotta
Copy link
Contributor Author

A lot of this will have to be checked once it is on staging

Copy link
Member

@julienrbrt julienrbrt left a comment

Choose a reason for hiding this comment

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

Personally, I do not understand the versioning. From v0.46 and v0.47 many things from the core concepts changed as well as modules docs and this isn't reflected here.

Copy link
Member

Choose a reason for hiding this comment

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

Not too sure what this does. Where is that plugin added as well?
Could this get a small doc? I am trying to understand how it created those files, as they aren't in /docs folder.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is just the action they use in docusaurus, i was having trouble last night getting it to build so trying all options. The custom plugin is specified in docusaurus.config.js. It uses all the files from main and any of those that arent the same, they have their own version. Additionally, there is version_config which removes specific files or paths from versions. This way its very clear as to what is in each version when its built

Copy link
Member

@julienrbrt julienrbrt Jun 12, 2023

Choose a reason for hiding this comment

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

I am not seeing it in the docusaurus.config.js. Which main are we talking about? This repo?

It uses all the files from main and any of those that arent the same, they have their own version.

Ok, so the process of adding a custom content for a version is manual right? If so, this means the versioning is definitely incomplete, won't it be a hell to maintain?
Again, only if my understanding is correct: If I want to change the content only for main, does not this mean I need to first copy the old content to each of the versioned SDK versions, where I want the content to stay the same and then edit the latest content?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

lol good catch, I removed this by accident c7d8c0d#diff-a038096cbdea434999e1dce5ab497212f1fe18204dde1a027ce3bdd663261a2aL210

but ill fix this up.

For the second point, the process of adding custom content will be automated. I have been working on a script that for any md file that is updated, removed or added will be copied to the specific version/main within the docs repo.

@@ -13,7 +13,7 @@ const config = {
tagline:
"Cosmos SDK is the world's most popular framework for building application-specific blockchains.",
url: "https://docs.cosmos.network",
baseUrl: "/",
baseUrl: "/cosmos-sdk-docs",
Copy link
Member

Choose a reason for hiding this comment

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

Cool! It could be handy to have this tiny thing in a separate PR so that https://cosmos.github.io/cosmos-sdk-docs can work sooner.

@samricotta
Copy link
Contributor Author

@julienrbrt Yeah this is why it needs to go on staging so i can literally proof read every single page and compare as there are still a lot of manual changes to happen

@tac0turtle
Copy link
Member

Hey checking in on this. Are we good to merge?

@samricotta samricotta merged commit 0d04e6d into main Jul 20, 2023
1 check passed
@samricotta samricotta deleted the sam/versioning branch July 20, 2023 12:21
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.

3 participants