Skip to content

docs: Add new OLX example course#68

Merged
sarina merged 10 commits intomainfrom
sarina/new-olx-course
Nov 25, 2025
Merged

docs: Add new OLX example course#68
sarina merged 10 commits intomainfrom
sarina/new-olx-course

Conversation

@sarina
Copy link
Contributor

@sarina sarina commented Nov 6, 2025

Minimal OLX course, showcasing various problem types, HTML units, video, custom tabs, grading policy. Imports into Studio. Omits certain Studio artifacts that aren't critical to basic course functionality (but could be added in later, if there's appetite for making additional documentation).

@ormsbee
Copy link

ormsbee commented Nov 7, 2025

Really off-the-cuff comment: The OLX url_names are case-sensitive, so I think that we should encourage people who are slinging OLX by hand to use lower case and underscores for their filenames/usage-keys. Otherwise it's really easy to slightly miss the capitalization on mixed case references. For instance, "LTIintroduction.xml‎" lowercases the "i" in "introduction", but "OLXResources.html‎" capitalizes the "R" in "Resources". I don't think we have great error messaging when you have things like this, though there may have been work there on the import linting side of things that I'm not aware of (I believe Arbisoft folks folded in an MIT library to validate OLX on import, but I don't remember any of the details).

@sarina sarina requested a review from ormsbee November 15, 2025 20:18
@sarina
Copy link
Contributor Author

sarina commented Nov 25, 2025

Add a multipart problem

Copy link
Member

@kdmccormick kdmccormick left a comment

Choose a reason for hiding this comment

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

LGTM pending the things we discussed in our meeting, which you've captured in comments 🚀

@sarina sarina merged commit d19272e into main Nov 25, 2025
3 checks passed
@sarina sarina deleted the sarina/new-olx-course branch November 25, 2025 17:32
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.

3 participants