-
Notifications
You must be signed in to change notification settings - Fork 94
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
Fix#396 - Rename tutorials + Fix names in sidebar #397
Conversation
@@ -1,5 +1,3 @@ | |||
# Tutorial: Making a Simple UI Change |
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.
You've deleted the header of the page, so what is the page going to look like on the GitHub wiki? Will readers know what the page refers to?
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.
Github wiki automatically renders the file name as the top level header (I learnt about it just recently) - https://github.com/oppia/oppia/wiki/Making-a-simple-UI-change. If you checkout the wiki page, you could see two headers which is making it look a bit odd.
_Sidebar.md
Outdated
@@ -52,7 +52,7 @@ | |||
* [[Debug frontend code|Debug-frontend-code]] | |||
* [[Debugging Stories|Debugging-Stories]] | |||
* [[Testing email functionality|Testing-email-functionality]] | |||
* [[Tutorial - Learn-to-figure-out-the-reproduction-steps-for-server-error|Tutorial:-Learn-to-figure-out-the-reproduction-steps-for-server-error]] | |||
* [[Tutorial - Learn to figure out the reproduction steps for server error|Tutorial-Learn-to-Figure-Out-the-Reproduction-Steps-for-Server-Error]] |
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.
for a server error
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.
Done, thanks!
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.
Thanks, LGTM.
Fix #396
Rename tutorials - Remove colons from the name of the tutorials
Fix the tutorial links name in Sidebar.