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 agree, but gonna leave it for now.
ormsbee created a review on a pull request on openedx/edx-platform
Only strongly requested changes are a comment for `can_delete` and re-applying `.location` -> `.usage_key`. Others are optional nits.
ormsbee created a review on a pull request on openedx/edx-platform
Only strongly requested changes are a comment for 'can_delete` and re-applying `.location` -> `.usage_key`. Others are optional nits.
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
Sorry, my bad. Just a rebase issue indeed.
edx-secure pushed 1 commit to 2u/release openedx/edx-platform
- feat: add "beta" label to libraries v2 content + item bank buttons (#35690) 784b2b8
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.
ormsbee created a review comment on a pull request on openedx/edx-platform
[nit, optional]: The usage of max_count == -1 as "grab all the things" is both frequent and obscure to anyone just glancing at the code for the first time. It might be worth making a constant or a ...
kdmccormick created a review comment on a pull request on openedx/edx-platform
I assume this was just a rebase thing. It was `.location` on my branch (carried over from the original block) until I updated it to `.usage_key` earlier today.
ormsbee created a review comment on a pull request on openedx/edx-platform
[question] Why `.location` instead of `.usage_key`? I thought `.location` was deprecated?
ormsbee created a review comment on a pull request on openedx/edx-platform
[nit, optional]: You can also do this as `elif 0 < len(self.children) < self.max_count`
ormsbee created a review comment on a pull request on openedx/edx-platform
[request] Please add a comment explaining this special-casing of itembank behavior.
edx-secure created a branch on openedx/edx-platform
private_to_public_08c8e9d214a0 - The Open edX LMS & Studio, powering education sites around the world!
edx-secure created a branch on openedx/edx-platform
private_to_public_784b2b8c3c18 - The Open edX LMS & Studio, powering education sites around the world!
kdmccormick created a comment on a pull request on openedx/edx-platform
Thanks for the fast reviews all!
renovate[bot] pushed 2 commits to renovate/squirejs-0.x openedx/edx-platform
renovate[bot] pushed 2 commits to renovate/jasmine-monorepo openedx/edx-platform
sohailfatima pushed 1 commit to fsohail/INF-1569/bulk-email-rebrand openedx/edx-platform
- feat: updated bulk email context for updated footer fb338e4
kdmccormick pushed 1 commit to master openedx/edx-platform
- feat: ItemBankBlock (#35553) This is a new XBlock that presents a random subset of its children. As of this commit, ... 08c8e9d