Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
fayyazahmed66 created a comment on a pull request on openedx/edx-platform
I am going to test this as its critical. CC: @farhan
muhammadadeeltajamul opened a pull request on openedx/edx-platform
feat: removed styling from course update notification
Removed all styling from course update notification content. - Tray notification will be plain text - Email notification will also be plain text Ticket: [INF-1622](https://2u-internal.atlass...
muhammadadeeltajamul created a branch on openedx/edx-platform
inf-1622 - The Open edX LMS & Studio, powering education sites around the world!
navinkarkera created a comment on a pull request on openedx/edx-platform
@bradenmacdonald Thanks for the suggestions, updated and addressed all your comments.
navinkarkera created a review comment on a pull request on openedx/edx-platform
Removed next button and updated it to go to component selection on library selection. https://github.com/user-attachments/assets/776403dc-a865-4568-a74a-809786c50efa
navinkarkera created a review comment on a pull request on openedx/edx-platform
Actually, do we even need a `Next` button? Selecting a course should directly take to next page since we only have one item to select in the first slide.
navinkarkera created a review comment on a pull request on openedx/edx-platform
We don't need it anymore, it was required when MFE was pushing the block to course. Now it only sends a message with selection details. I'll update it.
bradenmacdonald created a comment on a pull request on openedx/edx-platform
@kdmccormick @navinkarkera @pomegranited FYI
openedx-webhooks created a comment on a pull request on openedx/edx-platform
Thanks for the pull request, @bradenmacdonald! ### What's next? *Please work through the following steps to get your changes ready for engineering review:* #### :radio_button: Get product approv...
openedx-webhooks created a comment on a pull request on openedx/edx-platform
Thanks for the pull request, @bradenmacdonald! ### What's next? *Please work through the following steps to get your changes ready for engineering review:* #### :radio_button: Get product approv...
bradenmacdonald opened a pull request on openedx/edx-platform
"Bare Bones" UI for Problem Bank [FC-0062]
## Description This implements a minimal UI so that [the WIP Problem Bank XBlock](https://github.com/openedx/edx-platform/pull/35553) is usable: https://github.com/user-attachments/assets/093...
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
Like this: https://github.com/open-craft/edx-platform/commit/5939b5dada9f44f854d11dbe31ebef313349096a Turns out this is actually necessary or sometimes a random message will come in and use up t...
bradenmacdonald created a comment on a pull request on openedx/edx-platform
@navinkarkera Oh, actually one more important thing: can you make sure this new button is only enabled if [`libraries_v2_enabled()`](https://github.com/openedx/edx-platform/blob/83827a3dc2794a70399...
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
Nit: you don't have to check _all_ the fields of `doc` again here. You can just check `doc["display_name"]`, `doc["description"]`, and `doc["published"]`
bradenmacdonald created a review on a pull request on openedx/edx-platform
Thanks, this is great! I tested and it's working well. Just one request: can you please add `published.display_name` and `published.description` to `update_searchable_attributes` in `edx-platfor...
bradenmacdonald created a review on a pull request on openedx/edx-platform
Thanks, this is great! I tested and it's working well. Just one request: can you please add `published.display_name` and `published.description` to `update_searchable_attributes` in `edx-platfor...
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
Also if you use an arrow function ```js window.addEventListener("message", (event) => { ``` then you can remove the `.bind(this)` as it won't be necessary.
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
This can be fixed in a fast-follow PR if you want, but it's not a great UX that the "Next" button is sometimes hidden until you scroll, and the Next + Cancel buttons are in totally different places...
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
I think you're missing code to remove the event listener if the modal is closed/cancelled without picking a component.
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
```suggestion # Set `created_block.upstream` and then sync this with the upstream (library) version. ``` Why do we care about "preserving the former value of `created_block.upstream`...
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
TODO: add a comment that this is not actually an XBlock, unlike the others.
bradenmacdonald created a review on a pull request on openedx/edx-platform
Nice! It's working well. Here are some minor things I've noted that we should clean up:
bradenmacdonald created a review comment on a pull request on openedx/edx-platform
Do we even need to pass `parentLocator` anymore?