-
Notifications
You must be signed in to change notification settings - Fork 0
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
Migrate docs from Coursemology Help repo #1
base: master
Are you sure you want to change the base?
Conversation
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 upgrade to Docusaurus 3.7? The version 2.4.1 we're using was released back in March 2023 and it's too outdated.
- Please set up the GitHub Actions action to automatically deploy the site. I'll defer to your judgement whether to have the action run every time a commit or tag is made. Check this official action out, but if it doesn't work, the one I used before is this, but it's unofficial. I prefer sticking to official things if possible. Let me know how it turned out!
sidebar_label: 1b. Account Settings | ||
--- | ||
|
||
<span style={{ color: 'red' }}><b>NOTE:</b> New sign-ups are set to students by default.</span> |
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 notes, please use admonitions. Check for the rest of the documentation site for notes or additional tips and mark them up accordingly.
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.
|
||
 | ||
|
||
Note that as a student, you cannot create a course! |
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.
Use admonitions for notes or additional tips, and drop the "Note that" or similar sounding phrases for brevity. There's no need to parenthesise sentences if they are already part of an additional information.
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.
|
||
You may perform the following actions on **Manage Users** page. | ||
|
||
[[Invite Users Individually]](#invite-users-individually) [[Invite Users From File]](#invite-users-using-excel-file) [[Upgrade User Roles]](#upgrade-user-roles) |
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.
There's no need to separately create links and <a>
tags for navigation like this. Just use the #
headings and follow Docusarus' heading IDs.
There's also no need for this "table of contents" if you correctly use the #
headings. Docusaurus will generate the correct table of content on the right sidebar of the page.
If you really want to have the table of content on the page itself, use TOCInline
.
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 (removed the tags and let Docusaurus heading ids handle)
--- | ||
sidebar_position: 1 | ||
slug: /instructor-guide/setup/create-account | ||
title: 1a. Creating your Coursemology account |
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.
I personally don't see the point of numbering the sections hardly like this. It's so easy to keep them out of sync and cripples once you hit more than 26 sections for alphabets.
If it was up to me, I'd remove these things and just have the titles. Docusaurus will automatically generate the right tree structure for everything anyway.
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.
I think it's fine as is for now, the numbered titles is convenient for keeping track of their progress if they choose to go through the whole thing.
title: Student Guide | ||
--- | ||
|
||
Here you can find the essentials of how to use Coursemology! |
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.
If there's nothing for students then we should just remove it to reduce confusion.
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.
We are planning to add student-focused documentation in the near future, and we will certainly do so before the new guides are published on gh pages.
title: Instructor Guide | ||
--- | ||
|
||
Here you can find the essentials of how to use Coursemology! |
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.
This isn't very helpful, especially on mobile when the left sidebar that shows the entire guide tree isn't visible by default. See if there's anything that can be done with TOCInline
, or just tell the user to click on the hamburger icon or navigate to the left sidebar.
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.
There was a table on the original docs: I've readded it (as a Markdown table for now, so it renders as an actual table)
docusaurus.config.js
Outdated
@@ -93,6 +79,10 @@ const config = { | |||
label: "GitHub", |
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.
I think we can remove this. Our repository is irrelevant to the instructors.
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 now, I removed it. In future, we may want to consider directing users (instructors and students) to our Github repo to report issues.
sidebar_label: 6a. Automated Feedback | ||
--- | ||
|
||
You may use Codaveri for automated grading feedback for coding questions. |
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.
We need to first explain to them what Codaveri is. We can do it here in an admonition, or a separate index page for the Codaveri section.
|
||
You may use Codaveri for automated grading feedback for coding questions. | ||
|
||
First, you will need to enable **Codaveri Evaluation and Feedback** under **Course Settings**. |
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.
This should say "in Course Settings > Components". They wouldn't know where it is otherwise.
Probably a good idea to have a page on Components, then can say if they can't find certain features anywhere, they can check the Components page to see if said component is enabled. Then at least they know that Coursemology features are componentised, and if they don't like certain features, they can choose to disable it. Some instructors will ask us how to disable Forums or some gamification features because they find them intrusive or unnecessary for their course.
README.md
Outdated
|
||
Translation keys for text in React components (marked by <Translate/> elements) can be generated by the command | ||
``` | ||
$ yarn write-translations --locale zh |
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.
Add these commands to scripts
in package.json
instead.
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.
Upgraded to Docusaurus 3.7 (and all other Remaining todos:
|
zh
locale (incomplete).