Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
I was going to ask the same thing, because the canonical way to iterate over children is just to use `self.get_chidlren()`. But I saw the "all descendants" - I think the intention here was somethin...
ormsbee created a comment on a pull request on openedx/edx-platform
I guess I'm betting that if we can at least store them in a separated space, we'll have enough metadata information to rationalize this better in the future.
openedx-webhooks created a comment on a pull request on openedx/edx-platform
Thanks for the pull request, @pomegranited! ### What's next? *Please work through the following steps to get your changes ready for engineering review:* #### :radio_button: Get product approval ...
pomegranited opened a draft pull request on openedx/edx-platform
DO NOT MERGE -- testing Meilisearch in tutor core
**DO NOT MERGE** This PR exists solely to deploy a sandbox for https://github.com/overhangio/tutor/pull/1141. Part of: https://github.com/openedx/modular-learning/issues/236 **Settings** `...
edx-requirements-bot created a comment on a pull request on openedx/edx-platform
These Packages need manual review..</br> - **[MAJOR]** `edx-django-utils` changes from `6.1.0` to `7.0.0`
edx-requirements-bot created a comment on a pull request on openedx/edx-platform
List of packages in the PR without any issue.</br> - `boto3` changes from `1.35.42` to `1.35.45` - `botocore` changes from `1.35.42` to `1.35.45` - `coverage` changes from `7.6.3` to `7.6.4` - `c...
edx-requirements-bot opened a pull request on openedx/edx-platform
chore: Upgrade Python requirements
Python requirements update. Please review the [changelogs](https://openedx.atlassian.net/wiki/spaces/TE/pages/1001521320/Python+Package+Changelogs) for the upgraded packages.
edx-requirements-bot created a branch on openedx/edx-platform
repo-tools/upgrade-python-requirements-e47fdbf - The Open edX LMS & Studio, powering education sites around the world!
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
Nit: `legacy` doesn't appear in the event name so this is slightly confusing to me "Check that an analytics event (called edx.librarycontentblock.content.____ for backwards compatibility) was pu...
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
```suggestion the user about problems with settings (max_count). ```
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
```suggestion # Ensure we're starting with clean validation ```
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
I'm actually removing most of the validation messages in my UI PR, except the one for when `max_count > num_children`. Though it would be good to add validation that max_count isn't some invalid va...
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
As far as I know, this function will just skip over comments, so I'm not sure what this test is actually testing that's any different than if the comments didn't exist. We don't need to verify that...
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
Nit: make this a static attribute of `ItemBankTestBase` perhaps?
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
I don't like how this relies weird export behavior coming from `XmlMixin`, such that `add_xml_to_node` has a side-effect of writing definitions to the memory filesystem. According to the XmlMixin d...
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
This one also seems to be removable without causing any issues, at least for `ItemBankBlock`. Not sure if `LegacyLibraryContentBlock` needs it.
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
It seems you can remove `XmlMixin` now. `ItemBankBlock` works fine without it AFAICT, although you may have to update some test cases that rely on its weird default behavior of writing the XML to a...
bradenmacdonald created a review on a pull request on openedx/edx-platform
This is looking great, and working really well in my testing so far. Didn't run into any bugs.
bradenmacdonald created a review on a pull request on openedx/edx-platform
This is looking great, and working really well in my testing so far. Didn't run into any bugs.
edx-secure pushed 2 commits to 2u/release openedx/edx-platform
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.
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.
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.
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.
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 staging environment in preparation for a release to production.