Skip to content

Rewrite OLX documentation#1314

Merged
sarina merged 22 commits intomainfrom
sarina/new-olx-documentation
Dec 12, 2025
Merged

Rewrite OLX documentation#1314
sarina merged 22 commits intomainfrom
sarina/new-olx-documentation

Conversation

@sarina
Copy link
Contributor

@sarina sarina commented Nov 6, 2025

Rewrite OLX documentation to focus on:

  • Studio export format
  • Describing Ulmo-current OLX that will actually import

Depends on openedx/training-courses#68

Docs: https://docsopenedxorg--1314.org.readthedocs.build/en/1314/educators/navigation/olx.html

@sarina sarina force-pushed the sarina/new-olx-documentation branch 4 times, most recently from 47fc0e4 to 924ad46 Compare November 15, 2025 05:05
@sarina sarina marked this pull request as ready for review November 15, 2025 20:07
@sarina
Copy link
Contributor Author

sarina commented Nov 15, 2025

@ormsbee @pdpinch @jswope00 I took a swing at rewriting the OLX documentation - updating screenshots, indicating what's no longer supported in the Studio UI, and making a course in OLX that actually imports into Studio (see openedx/training-courses#68 - that needs to merge before this merges)

I don't know how detailed you'd like to be in review. I think at minimum, this is a lot better than what we have now.

@kdmccormick kdmccormick self-requested a review November 21, 2025 19:53
@kdmccormick
Copy link
Member

hey, this hasn't fallen off my radar, just has fallen behind some Ulmo-critical libraries stuff. still planning to review soon so that we can get it merged before the holidays.

@sarina
Copy link
Contributor Author

sarina commented Dec 2, 2025

Thanks. I'd like to get it in before the Ulmo cut if possible, but it shouldn't be terribly difficult to backport if necessary.

Copy link
Contributor

@jswope00 jswope00 left a comment

Choose a reason for hiding this comment

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

This is really impressive!! Some changes requested in the comments.

@sarina sarina force-pushed the sarina/new-olx-documentation branch from 28b4ba6 to e9c73a6 Compare December 10, 2025 17:22
@sarina
Copy link
Contributor Author

sarina commented Dec 10, 2025

@jswope00 could you re-review since your review is blocking?

@jswope00 jswope00 self-requested a review December 12, 2025 15:28
@sarina
Copy link
Contributor Author

sarina commented Dec 12, 2025

@kdmccormick - I'm going to go ahead and merge this, and we can do a review and make further corrections moving forward, so this can get into the Ulmo release.

@sarina sarina merged commit ef01cdd into main Dec 12, 2025
2 checks passed
@sarina sarina deleted the sarina/new-olx-documentation branch December 12, 2025 15:46
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.

Issue with page: "The Structure of a Sample Course" images are out of date

3 participants