Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

openedx/edx-platform

pdpinch pushed 2 commits to master openedx/edx-platform
  • feat: adaptive display of links Do not display the 'Learn more' and 'Share feedback' links for banner that is enable... e4c6956
  • Merge pull request #34432 from DmytroAlipov/add-adaptive-display-of-links c3d68b1

View on GitHub

pdpinch closed a pull request on openedx/edx-platform
feat: adaptive display of links 'Learn more' and 'Share feedback'
## Description Do not display the 'Learn more' and 'Share feedback' links for the banner that is enabled by the context_course.discussions_settings flag if the URLs for these links are not set i...
openedx-webhooks created a comment on a pull request on openedx/edx-platform
Thanks for the pull request, @MoisesGSalas! ### What's next? *Please work through the following steps to get your changes ready for engineering review:* #### :radio_button: Get product approval ...

View on GitHub

MoisesGSalas opened a draft pull request on openedx/edx-platform
chore: use the PEP 508 format without editable VCS requirements
This change makes it possible to use [uv](https://github.com/astral-sh/uv) as a pip replacement in order to perform a faster installation of requirements. `edx_proctoring_proctortrack` already p...
ormsbee created a review comment on a pull request on openedx/edx-platform
It actually does the contentserver thing and streams it through the app code at the very end. I temporarily abandoned the x-accel-redirect thing because I was having issues on the Tutor side of thi...

View on GitHub

ormsbee created a review on a pull request on openedx/edx-platform

View on GitHub

kdmccormick created a review comment on a pull request on openedx/edx-platform
Oh right, content libraries is a "plugin" 😂 Sounds good, thanks for checking.

View on GitHub

kdmccormick created a review on a pull request on openedx/edx-platform

View on GitHub

ormsbee created a review on a pull request on openedx/edx-platform

View on GitHub

ormsbee created a review comment on a pull request on openedx/edx-platform
Oh right, I forgot this was added to the API. Thank you.

View on GitHub

ormsbee created a review comment on a pull request on openedx/edx-platform
Oh right, I should update it.

View on GitHub

ormsbee created a review on a pull request on openedx/edx-platform

View on GitHub

ormsbee created a review comment on a pull request on openedx/edx-platform
Yeah, I'll fix it to clarify. I think of Studio as getting a tiny amount of traffic relative to the LMS, so even big Studio sites don't count as scale in my mind in terms of throughput. I tend to t...

View on GitHub

ormsbee created a review on a pull request on openedx/edx-platform

View on GitHub

ormsbee created a review comment on a pull request on openedx/edx-platform
It is enforced–the URLs are [only configured for Studio](https://github.com/openedx/edx-platform/blob/master/openedx/core/djangoapps/content_libraries/apps.py#L24-L27) and don't work in LMS. (I jus...

View on GitHub

ormsbee created a review on a pull request on openedx/edx-platform

View on GitHub

ormsbee closed a draft pull request on openedx/edx-platform
XBlock asset support for new libraries experience (WIP)
A very hacky pass at getting Learning Core backed static assets to show up for libraries. Things I need to look into: * Showing different versions of the content, since it's rendering on LMS ...
JayceDowns42 starred openedx/edx-platform
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
Done.

View on GitHub

bradenmacdonald created a review on a pull request on openedx/edx-platform

View on GitHub

bradenmacdonald pushed 1 commit to braden/block-draft-published openedx/edx-platform
  • docs: address review comments d79d43f

View on GitHub

bradenmacdonald created a comment on a pull request on openedx/edx-platform
@kdmccormick We don't need to support `api.create_library_block_for_child` since we aren't going to support child blocks in libraries at all AFAIK.

View on GitHub

kdmccormick created a review comment on a pull request on openedx/edx-platform
```suggestion usage_key = video_block.usage_key ``` [nit] you can use the shorthand now if you'd like

View on GitHub

kdmccormick created a review comment on a pull request on openedx/edx-platform
Would it be fair to delete this Historic Information or update it for Learning Core?

View on GitHub

kdmccormick created a review comment on a pull request on openedx/edx-platform
This is the [Django view implemenation of the asset server](https://github.com/openedx/openedx-learning/blob/main/docs/decisions/0015-serving-static-assets.rst#django-view-implemenation), but just ...

View on GitHub

kdmccormick created a review comment on a pull request on openedx/edx-platform
Just checking my understanding: you mean that we won't run this at _LMS_ scale, right? If I understand right, large sites' CMS servers will still hit this view when serving content library assets f...

View on GitHub

kdmccormick created a review comment on a pull request on openedx/edx-platform
If this should only be used by CMS, it might be good to enforce that? I would love if the library API URLs just weren't defined for LMS, but without taking on all of that work, you could just: ...

View on GitHub

bradenmacdonald created a comment on a pull request on openedx/edx-platform
@kdmccormick Thanks! Here is the corresponding ticket for the UI: https://github.com/openedx/frontend-app-authoring/issues/1385

View on GitHub

ormsbee created a review comment on a pull request on openedx/edx-platform
@bradenmacdonald: Okay, I think that's fine. And if it becomes a problem, we can address it then. No need to re-review on my part. Thank you!

View on GitHub

ormsbee created a review on a pull request on openedx/edx-platform

View on GitHub

Load more