Skip to content

docs: [Breaking] split the docs into smaller pages #6079

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

Merged
merged 35 commits into from
Aug 4, 2020
Merged

Conversation

NamanJain8
Copy link
Contributor

@NamanJain8 NamanJain8 commented Jul 27, 2020

This PR splits docs with larger pages into smaller ones.

Splitting work is from https://github.com/ChrisChinchilla/dgraph.
This PR updates the build script of docs to serve docs via new hugo-theme.


This change is Reviewable

Docs Preview: Dgraph Preview

@CLAassistant
Copy link

CLAassistant commented Jul 27, 2020

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the area/documentation Documentation related issues. label Jul 27, 2020
@NamanJain8 NamanJain8 self-assigned this Jul 27, 2020
Copy link
Contributor

@danielmai danielmai left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 127 files at r3, 1 of 6 files at r4.
Reviewable status: 2 of 131 files reviewed, 5 unresolved discussions (waiting on @danielmai, @manishrjain, @MichaelJCompton, @NamanJain8, and @vvbalaji-dgraph)


wiki/scripts/build.sh, line 14 at r5 (raw file):

0.0.0.0:8000}"

Was this changed for a reason?


wiki/scripts/build.sh, line 21 at r5 (raw file):

OLD_THEME="master"
NEW_THEME="naman/docs-merge"

Is the plan to swap this once the docs-merge branch actually merged?


wiki/scripts/build.sh, line 32 at r5 (raw file):

'namandocs'

What is namandocs? Is this branch going to be used moving forward, or was it just for testing?


wiki/scripts/build.sh, line 45 at r5 (raw file):

'v20.03.1'

What's the difference between VERSIONS_ARRAY and OLD_VERSIONS?

I feel that OLD_VERSIONS should also have v20.03.3 and v20.03.4 if OLD_VERSIONS means it's not using the new split-page docs structure.


wiki/themes/hugo-docs, line 1 at r5 (raw file):

Subproject commit d06adcee33319b295238861de2123982b7e79cf0

Can you remove this submodule? If we do need this, then we need to update the wiki README. Otherwise, we can continue with the current approach of not having the hugo-docs theme as a git submodule.

NamanJain8 added a commit to dgraph-io/hugo-docs that referenced this pull request Jul 28, 2020
…ons (#56)

* Changes to theme to suit breaking large content sections see (hypermodeinc/dgraph#6079)
* Remove defunct JavaScript and switch variables used
* Tweak side menus and JavaScript
* Add JavaScript redirect

Co-authored-by: Naman Jain <naman@dgraph.io>
Copy link
Contributor

@danielmai danielmai left a comment

Choose a reason for hiding this comment

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

:lgtm: I'm very much looking forward to this change!

Reviewed 2 of 130 files at r2, 121 of 127 files at r3, 4 of 6 files at r4, 1 of 1 files at r5, 3 of 3 files at r6.
Reviewable status: all files reviewed, 5 unresolved discussions (waiting on @manishrjain, @MichaelJCompton, @NamanJain8, and @vvbalaji-dgraph)

danielmai added a commit to dgraph-io/hugo-docs that referenced this pull request Aug 2, 2020
…nt sections (#56)"

The new theme doesn't work with the current versions of the docs.

Once hypermodeinc/dgraph#6079 is merged, this change can be brought back.

This reverts commit 913be2e.
danielmai added a commit to dgraph-io/hugo-docs that referenced this pull request Aug 2, 2020
…nt sections (#56)" (#61)

The new theme doesn't work with the current versions of the docs.

Once hypermodeinc/dgraph#6079 is merged, this change can be brought back.

This reverts commit 913be2e.
@NamanJain8 NamanJain8 changed the title docs: [Breaking] split docs into smaller pages docs: [Breaking] split the docs into smaller pages Aug 2, 2020
@NamanJain8 NamanJain8 merged commit 88651b4 into master Aug 4, 2020
danielmai pushed a commit that referenced this pull request Aug 4, 2020
This PR splits docs with larger pages into smaller ones.

Splitting work is from https://github.com/ChrisChinchilla/dgraph.
This PR updates the build script of docs to serve docs via new hugo-theme and also add redirects as split pages don't exist in old versions.

Old versions should use "old" theme of "dgraph-io/hugo-docs" while newer branches should use its master.
@NamanJain8 NamanJain8 deleted the namandocs branch August 14, 2020 11:28
@NamanJain8 NamanJain8 restored the namandocs branch August 20, 2020 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Documentation related issues.
Development

Successfully merging this pull request may close these issues.

4 participants