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

Inital setup of docusaurus #3165

Merged
merged 16 commits into from
Oct 19, 2018
Merged

Conversation

sveinpg
Copy link
Contributor

@sveinpg sveinpg commented Oct 7, 2018

#3161

Added inital setup of docusaurus.

Missing files in docs:

  • README.md
  • PATRONS.md
  • CHANGELOG.md

Docusaurus requires docs to be placed in /docs. Should we move these files?

Also, since docusaurus currently does not support nested sidebars, I moved Structuring Reducers one level up in the sidebar.

@markerikson
Copy link
Contributor

Hmm. Well, we can't move the actual README, because that's gotta stay at the root of the repo. PATRONS could be moved, and tbh CHANGELOG is kinda pointless.

Tell you what. Let's not include PATRONS or CHANGELOG in the actual published docs. Leave the files there for historical purposes, but no point in publishing them.

For the README... do we even want the entire README as a docs page itself? I'm not sure we do. Perhaps we should have a cut-down version of it in the docs.

*/

// This copyright info is used in /core/Footer.js and blog RSS/Atom feeds.
copyright: `Copyright © ${new Date().getFullYear()} Your Name or Your Company Name`,
Copy link
Member

Choose a reason for hiding this comment

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

Need to fill this in.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What do you want it to say? "Redux JS"?

Copy link
Member

Choose a reason for hiding this comment

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

As per the license (which covers the existing docs), Copyright (c) 2015-present Dan Abramov.

Copy link
Member

Choose a reason for hiding this comment

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

Actually, maybe we should get a little more specific. Dan doesn't own copyright for things he didn't contribute. Copyright (c) 2015-present Dan Abramov and the Redux documentation authors.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's nice. Fixed now.

.gitignore Outdated

website/translated_docs
website/build/
website/yarn.lock
Copy link
Member

Choose a reason for hiding this comment

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

Please don't use Yarn. We use npm here. Also, don't ignore the lock file generated.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Noted

@sveinpg
Copy link
Contributor Author

sveinpg commented Oct 8, 2018

Should we create an issue to cover what needs to be done?

Needs to be done:

  • Initial setup of docusaurus
  • Remove blog feature
  • Remove sample content and pages
  • Update footer links
  • Add copyright to footer
  • Fix internal links (like TOC)
  • Create cut-down verison of the README
  • Create frontpage design (what should be included on the frontpage?)
  • Update readme to reference the new website.
  • Remove template README from /website(?)
  • Set up search

@netlify
Copy link

netlify bot commented Oct 8, 2018

Deploy preview for redux-docs ready!

Built with commit 89faca9

https://deploy-preview-3165--redux-docs.netlify.com

@reduxjs reduxjs deleted a comment from netlify bot Oct 8, 2018
@markerikson
Copy link
Contributor

Looks like Docusaurus just merged in the nested categories feature in facebook/docusaurus#892, and v1.4.1 should be out with that in the next few days.

Copy link
Member

@timdorr timdorr left a comment

Choose a reason for hiding this comment

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

Sorry for the duplicate comments. There isn't a way to edit multiple files in a single commit from the web UI, so I'm just noting the places that are affected in the review UI for now.

---
id: core-consepts
title: Core consepts
sidebar_label: Core consepts
Copy link
Member

Choose a reason for hiding this comment

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

Spelling: "concepts"

---
id: prerequisite-consepts
title: Prerequisite Consepts
sidebar_label: Prerequisite Consepts
Copy link
Member

Choose a reason for hiding this comment

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

Spelling: "concepts"

<div>
<h5>Docs</h5>
<a href={this.docUrl("introduction/core-consepts")}>
Core consepts
Copy link
Member

Choose a reason for hiding this comment

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

Spelling: "concepts"

<ProjectTitle />
<PromoSection>
<Button href={docUrl("introduction/core-consepts", language)}>
Core consepts
Copy link
Member

Choose a reason for hiding this comment

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

Spelling: "concepts"

"docs": {
"Introduction": [
"introduction/motivation",
"introduction/core-consepts",
Copy link
Member

Choose a reason for hiding this comment

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

Spelling: "concepts"

],
"Structuring Reducers": [
"recipes/structuring-reducers",
"recipes/reducers/prerequisite-consepts",
Copy link
Member

Choose a reason for hiding this comment

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

Spelling: "concepts"


// For no header links in the top nav bar -> headerLinks: [],
headerLinks: [
{ doc: "introduction/core-consepts", label: "Docs" },
Copy link
Member

Choose a reason for hiding this comment

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

Spelling: "concepts"

@selbekk
Copy link
Contributor

selbekk commented Oct 9, 2018

What a great god damn effort @sveinpg 👏 👏 👏

@timdorr
Copy link
Member

timdorr commented Oct 9, 2018

The only thing I don't like about this site so far: The sidebar position resets after navigating to a new page. If I'm way down at the API reference and click over from createStore to the Store, the sidebar is back at the top.

Also, for the "README", I'd like to see that be a "Getting Started" page that can replace the "Core Concepts" button on the front page. And that should contain just the bare minimum to make a store and dispatch actions. Simple and straightforward for those that don't want to read through many pages of docs 😄

@endiliey
Copy link
Contributor

@timdorr

There is a workaround for the sidebar scroll position

facebook/docusaurus#823 (comment)

@yangshun
Copy link
Contributor

@sveinpg Hi there, Docusaurus maintainer here. We managed to get the subcategories feature shipped in facebook/docusaurus#1026 and you should be able to keep the subcategories for "Structuring Reducers". Could you point your Docusaurus version to trunk for the moment? If there are things that Redux needs but Docusaurus doesn't support we could do it over the weekend. Else if all is good, I'll release Docusaurus 1.5.0 over the weekend which will include the subcategories feature.

Let me know what you think! We're here to support Redux's migration to Docusaurus 😄

@sveinpg
Copy link
Contributor Author

sveinpg commented Oct 12, 2018

@yangshun How do I point my docusaurus version to trunk?

"rename-version": "docusaurus-rename-version"
},
"devDependencies": {
"docusaurus": "^1.4.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

@sveinpg

Change this line to:

"docusaurus": "git://github.com/facebook/docusaurus.git#127500532103ee9096de9f202866546969ea107d"

Refer to https://stackoverflow.com/a/18049685/1751946 for more info.

@yangshun
Copy link
Contributor

Docusaurus v1.5.0 has been released - https://github.com/facebook/Docusaurus/blob/master/CHANGELOG.md#150---2018-10-13

@sveinpg was trying out the trunk version yesterday and he showed me that the subcategories worked ok!

unknown-1

Thanks for your patience!

cc @markerikson @timdorr

@markerikson
Copy link
Contributor

Okay, I just took a look at the current contents. It looks great!

I think this is more than sufficient to merge into the docs-rebuild branch, and we can open up some more issues to figure out what else actually needs to be done.

@sveinpg , thank you VERY much for some excellent work!

@markerikson markerikson merged commit 944df4a into reduxjs:docs-rebuild Oct 19, 2018
markerikson pushed a commit that referenced this pull request Dec 3, 2018
* Added metadata to the docs

* Added  initial docusaurus setup

* Removed markdown block

* Updated copyright

* Removed yarn.lock from gitignore

* Updated copyright

* .

* Add copyright to footer

* Added white version of redux-icon to the header

* Updated image urls to use https

* Fixed spelling

* Added script to scroll active sidebar in to viewport.

* Moved content from README to a getting started section.

* Replaced core concepts link with getting started

* Updated docusaurus to 1.5.0

* Moved Structuring Reducers to a subcategory in the sidebar
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.

6 participants