search

openedx / edx-platform

The Open edX LMS & Studio, powering education sites around the world!
https://openedx.org
GNU Affero General Public License v3.0
7.44k stars 3.88k forks source link

Support static assets in the Learning Core runtime #34737

Closed kdmccormick closed 2 weeks ago

kdmccormick commented 6 months ago

@ormsbee Opening this issue now so that I have something to link TODO comments to :)

Exact details TBD. ADR with details: https://github.com/openedx/openedx-learning/blob/main/docs/decisions/0015-serving-static-assets.rst

kdmccormick commented 2 months ago

In review: https://github.com/openedx/openedx-learning/pull/217

bradenmacdonald commented 2 months ago

https://github.com/openedx/frontend-app-course-authoring/pull/1242 will soon merge, and then we'll pretty much have everything in place on the frontend to be able to test this: e.g. get a simple HTML XBlock that references a static image to render on the frontend. (Would still require using the python API to upload the image and reference it from the block's OLX though.)

I think that is the simplest case, before we move on to things like copying a block into a modulestore course.

ormsbee commented 1 month ago

Closing this now that https://github.com/openedx/openedx-learning/pull/217 has merged (which gets us enough of this functionality for Sumac).

ormsbee commented 1 month ago

Oh wait, I misread this. Re-opening, since the static asset implementation for edx-platform is incomplete.

ormsbee commented 1 month ago

Next PR:

Ian2012 commented 2 weeks ago

I think this one is closed, right?

ormsbee commented 2 weeks ago

@Ian2012: Yeah, I feel like this is over-broad at this point. Basic support has been added, though not to the full spec of the ADR. It'll be more useful to break it down into the pieces we care about–it's possible the full ADR version won't happen until after Teak, and that's probably okay.