-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
gitbooks not showing link for "Core concepts" #482
Conversation
While reading the online book there is no link for next chapter after the Getting started one because Core concepts has no file attached to it.
It seems like there is still not exist the link though. |
@ktsn I am really sorry I didn't understand what your comment meant. |
I mean that this situation persists when I check the docs with your patch in local. |
Did the gitbook's build fail? I can't really find the root cause of this not working without seeing the output of gitbooks. Do you have gitbooks installed on local? |
The build succeeded but there is still no link for next page in the Getting Started page. Or you just intended to add the link at "Core Concept" in ToC? |
Well, I am not sure what I did :-) So let me pinpoint the problem so you can fix it. When I am reading the gitbooks book, I don't have a (next) button for every page except http:https://vuex.vuejs.org/en/getting-started.html has a next button. This is because the next section has no link, thus no next. |
OK, I found the cause. We should update |
Yes, you are right. I have used gitbooks earlier once, https://github.com/thewhitetulip/web-dev-golang-anti-textbook/blob/master/SUMMARY.md What I had done to mitigate this issue is add a new file for the one not showing the link, like a landing page. Can we do the same? Or what we can do is, shift the Do let me know what you wish to do. We can close this issue and start a new one. |
Thanks for suggesting your solutions.
I'm not sure if I understand you correctly but do you mean adding |
Yes, we can add a new core-concepts.md which will just say something like , " In this section we will study the core components". with a link to the child chapters :-) |
Great! Can we add it on this PR? |
Done! @ktsn Please check. |
docs/en/core-concepts.md
Outdated
- Getters | ||
- Mutations | ||
- Actions | ||
- Modules |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great if these items have the link to the corresponding chapter 🙂
docs/en/README.md
Outdated
@@ -7,7 +7,7 @@ | |||
- [Installation](installation.md) | |||
- [What is Vuex?](intro.md) | |||
- [Getting Started](getting-started.md) | |||
- Core Concepts | |||
- [Core Concepts](state.md) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we also update this link to core-concepts.md
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, sorry. I had intended it to have core-concepts.md link only. Somehow missed it.
docs/en/core-concepts.md
Outdated
@@ -0,0 +1,10 @@ | |||
# Core concepts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a trivial thing but camelizing all words is better like other chapter title.
Core Concepts
👍 Done |
Thanks! LGTM 🙂 |
I had to search for what LGTM meant. 😄 |
Hey @ktsn , is there a reason it's not merged yet? Are you waiting for @yyx990803 approval? @thewhitetulip there is a conflict, could you please update this PR? |
Sorry for leaving this for a long time. This look great, thank you! |
Finally! |
While reading the online book there is no link for next chapter after the Getting started one because Core concepts has no file attached to it.