Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
pomegranited created a review comment on a pull request on openedx/edx-platform
Should use [CoursewareSearchIndexer.INDEX_NAME](https://github.com/openedx/edx-platform/blob/e28a01e2bf78d5750c649edf5e8d48776cc38d8d/cms/djangoapps/contentstore/courseware_index.py#L343) instead o...
pomegranited created a review comment on a pull request on openedx/edx-platform
Lol.. sounds like a Google situation, where the working approach is deprecated, but the new approach isn't developed yet 😄 So long as we can pass these settings from the env to the LMS + Studio...
pomegranited created a review comment on a pull request on openedx/edx-platform
We could at least convert them to UTC timestamps, so at least the deserialized time is accurate, even if the timezone isn't preserved?
pomegranited created a review comment on a pull request on openedx/edx-platform
Hmm ok.. I guess I don't have any experience with use cases where different timezones might be used for different dates throughout the code. But I'm happy to trust your experience :)
edx-secure created a branch on openedx/edx-platform
private_to_public_e28a01e2bf78 - The Open edX LMS & Studio, powering education sites around the world!
edx-secure created a branch on openedx/edx-platform
private_to_public_c3d68b13f84c - The Open edX LMS & Studio, powering education sites around the world!
renovate[bot] pushed 2 commits to renovate/edx-paragon-2.x openedx/edx-platform
renovate[bot] pushed 2 commits to renovate/squirejs-0.x openedx/edx-platform
muhammadadeeltajamul pushed 1 commit to master openedx/edx-platform
- temp: logs for unsubscribe event (#35652) e28a01e
ormsbee created a review comment on a pull request on openedx/edx-platform
I'll try to have a real review in the next day, but a quick question: where does this icon come from?
jignaciopm created a comment on an issue on openedx/edx-platform
Hi @arbrandes I have been following up on this issue and am in charge of it from Edunext. The first thing I have done is test the proposed [workaround](https://discuss.openedx.org/t/image-zoom-t...
ormsbee created a review comment on a pull request on openedx/edx-platform
Should this be `/api/contentstore/v2/downstreams/{usage_key_string}` here (and elsewhere in this docstring)?
ormsbee created a review on a pull request on openedx/edx-platform
Just a minor question about the API endpoint in the docstring (whether it's intended to be `/api/contentstore/v2/...`). Please consider the remaining logging request as optional.
ormsbee created a review on a pull request on openedx/edx-platform
Just a minor question about the API endpoint in the docstring (whether it's intended to be `/api/contentstore/v2/...`). Please consider the remaining logging request as optional.
ormsbee created a review comment on a pull request on openedx/edx-platform
Fair point. The main reason I bring it up is because in this case, it might be confusing to see a bunch of 404s in the logs for this, when it's really an underlying authz check that's failing.
ormsbee created a comment on a pull request on openedx/edx-platform
@bradenmacdonald: Yeah, this is still missing the part where it hooks into the XBlock runtime. I'll start a PR for that tomorrow (hopefully with @Ian2012)
bradenmacdonald created a review on a pull request on openedx/edx-platform
LGTM. This is working in the Authoring MFE via the "Assets" list under "Details > Advanced Details", but it doesn't seem to be working in the preview yet - an HTML block with `<img alt="Picture"...
bradenmacdonald created a review on a pull request on openedx/edx-platform
LGTM. This is working in the Authoring MFE via the "Assets" list under "Details > Advanced Details", but it doesn't seem to be working in the preview yet - an HTML block with `<img alt="Picture"...
ormsbee created a comment on a pull request on openedx/edx-platform
@kdmccormick: I think I've addressed your comments/suggestions.
edx-pipeline-bot created a comment on a pull request on openedx/edx-platform
**2U Release Notice**: This PR has been deployed to the edX production environment.
