Open edX — Activity since 2025-04-07

» eduNEXT/platform-plugin-ecommerce-api | edx/commerce-coordinator | openedx/openedx-events | openedx/openedx-filters | openedx/xblock-lti-consumer | openedx/docs.openedx.org | openedx/openedx-learning | Tutor PRs

• eduNEXT/platform-plugin-ecommerce-api repo issues, pull requests, and releases [back to top]

  Nothing happening

• edx/commerce-coordinator repo issues, pull requests, and releases [back to top]

  • fix: concurrency issue in fulfillment task #376 [syedsajjadkazmii]

    The fulfill_placed_order_signal_task Celery task is being triggered twice, which should not be happening. This...

    • abdullahwaheed

  • feat: added management command to handle discount code category enum updates and added channel category #378 [abdullahwaheed]

    Update the category custom field and create a channel custom field in the cart discount Merge checklist: Check off...

    • NawfalAhmed

  • fix: improve ct call in program price api #379 [syedsajjadkazmii]

    Fix errors and put retries on CT calls in program price api view Merge checklist: Check off if complete or not...

    • mubbsharanwar: When the function will be terminated, I mean where is the condition that returns the flow to the...

      • syedsajjadkazmii: its here...

    • mubbsharanwar: LGTM

    • shafqatfarhan

      • shafqatfarhan: Suggested change ...

      • shafqatfarhan: Suggested change ...

  • fix: fallback for user first name empty #380 [syedsajjadkazmii]

    fallback for user first name empty Merge checklist: Check off if complete or not applicable: Documentation updated...

    • shafqatfarhan

  • Revert "fix: fallback for user first name empty" #381 [syedsajjadkazmii]

    Reverts #380

    • shafqatfarhan

  • Revert "Revert "fix: fallback for user first name empty"" #382 [syedsajjadkazmii]

    Reverts #381

    • shafqatfarhan

  • feat: extract ct data into csv command #367 [syedsajjadkazmii]

    Management command to extract different products types data from CT

    • NoyanAziz: CTCustomAPIClient already in exists in the repo

      • shafqatfarhan: @syedsajjadkazmii Can you please look into it?

    • shafqatfarhan: @syedsajjadkazmii Please rebase and check the comments. We have added a new mgmt command,...

• openedx/openedx-events repo issues, pull requests, and releases [back to top]

  • v10.0.0 v10.0.0 7bcc56b [navinkarkera]

    ## What's Changed * chore: Upgrade Python requirements by @edx-requirements-bot in...

  • refactor!: use LibraryCollectionLocator and LibraryContainerLocator in event data structures [FC-0083] #487 [pomegranited] open-source-contribution FC

    Description Refactors the LibraryCollectionData and LibraryContainerData structures to use LibraryCollectionLocator...

    • rpenido: LGTM 👍 Thank you for your work, @pomegranited! I tested this using the CI pipeline I read...

    • pomegranited: Hi @mariajgrimaldi :) This PR addresses your comment on #479 (comment) Are you available to review?

    • pomegranited: Hi @felipemontoya , do you have time to help get this change merged? Thank you!

    • navinkarkera: @pomegranited Looks good! 👍 I tested this: (Verified tests locally) I read through the code ...

  • feat: add support for complex types in dicts and lists #483 [BryanttV] open-source-contribution

    Description This PR adds support for complex types in dictionaries and lists. e.g. List[Dict[str, str]: List of...

    • bmtcril: I'm going to attempt testing this locally against the redis and kafka backends today, sorry for the delay

• openedx/openedx-filters repo issues, pull requests, and releases [back to top]

  Nothing happening

• openedx/xblock-lti-consumer repo pull_requests [back to top]

  Nothing happening

• openedx/docs.openedx.org repo pull_requests [back to top]

  • Failed add_video_quiz.rst #994 [leiramagpantay] open-source-contribution

    • sarina

  • Passed configure_prerequisite_content.rst #995 [leiramagpantay] open-source-contribution

    • sarina

  • Failed contribute_course_wiki.rst #968 [leiramagpantay] open-source-contribution

    • sarina

  • add data references #991 [sarina]

    Remove duplicate maintenance table Add data references (outdated) docs

    • sarina: @bmtcril - this is a reasonably straight port of these docs, with added warnings and references...

    • bmtcril: Just one note, at a glance everything else seems ok. TIL Wiki doesn't have a DEPR!

      • bmtcril: This file seems to only be relevant to the edX data package

  • Fix styling on removed flags list #996 [sarina]

  • Add Sumac Release Note on LTI Certification #997 [sarina]

    Sumac is now LTI certified (https://site.imsglobal.org/certifications/axim-collaborative/open-edx-platform) so add a...

  • Fixes for Set Up Course Section #944 [jswope00] open-source-contribution

    …page.rst

    • sarina: @jswope00 - do you want me to address my comment or are you going to be able to follow up on this?

    • sarina

  • Add seealso tables for OLX pages #1000 [sarina]

    Addresses #826

  • Configure search result optimizations #1001 [sarina]

    Configure search results Deprioritize old release pages (Juniper is always a high result for me) Priviledge...

  • Update OLX Documentation #999 [sarina]

    Add diataxis tags to OLX pages Add maintenance tables to OLX docs Add failures to maintenance tables Remove OLX docs...

    • sarina: Addresses #948

    • sarina: @pdpinch care to take a look at this PR?

    • pdpinch: Looks good. Just one question.

    • sarina: @pdpinch what's your question?

    • pdpinch: do we need a **Maintenance chart** line here?

      • sarina: I added one because I wanted to call out that the edx-insider course is very old and not in the...

      • pdpinch: Sorry, I wasn't clear. I definitely agree with adding the chart, but I think it's missing the...

      • sarina: Ha! Yes, I'll add that in. Suggested change ...

  • Various Hackathon docs fixes #1003 [sarina]

    Add LTI Enablement instructions to Course Live Add docs on Course Data Researcher Role Update Export Course...

    • sarina: Also resolves #939 but I apparently can't link more than 10 :)

  • Hackathon Docs Fixes 2 #1004 [sarina]

    Addresses #957

• openedx/openedx-learning repo issues, pull requests, and releases [back to top]

  • fix: make database function compatible with postgresql and mysql both #260 [qasimgulzar] open-source-contribution

    Summary This pull request includes updates to the StringAgg class in openedx_tagging/core/tagging/models/utils.py to...

    • ormsbee: @pomegranited, @navinkarkera, @bradenmacdonald: Do any of you folks want to take a look at this...

    • pomegranited: Thank you for this PR @qasimgulzar ! But I don't quite understand the approach -- why not...

      • qasimgulzar: I had thought that way but by adding a subclass it will require me to update api.py and...

      • pomegranited: @qasimgulzar Ah, I see why we can't subclass contrib.postgres.aggregates.StringAgg here -- that...

    • pomegranited: I'm not going to block merging this, so please consider my comments as suggestions :)

      • pomegranited: If delimiter cannot be customized for MySQL or SQLite, then I don't think it should be exposed...

        • pomegranited: And is it possible to avoid overriding as_sql() below by providing the same output_field that...

        • qasimgulzar: I think that's a great idea

        • qasimgulzar: @pomegranited I just updated the PR as per your recommendations, thank you for the great suggestion

    • pomegranited: 👍 Oh good, I'm glad my suggestions were helpful @qasimgulzar . Thank you for doing this, and all...

    • bradenmacdonald

      • bradenmacdonald: Suggested change ';', # Used...

      • bradenmacdonald: Is this used, or is it always , ? Need to either update line 55 below or remove this and update...

  • feat: api for updating children in container [FC-0083] #292 [navinkarkera] open-source-contribution FC

    Description: Updates create_next_container_version api to handle addition or removal of children entities. Related...

    • bradenmacdonald: FYI @navinkarkera it looks like this PR missed a migration for a change to ordering - see #298 (comment)

    • navinkarkera: @bradenmacdonald Oops, missed it.

  • Add support for Django 5.2 #298 [irtazaakram]

    Resolves #296

    • bradenmacdonald

      • bradenmacdonald: Does the User model really have no PII ? And why do we have to declare annotations for common...

        • irtazaakram: The auth.User model does have some PII (username, email, and password), so I’ve added the...

      • bradenmacdonald: Where is this migration coming from when there's no changes to the model - is it related to...

        • irtazaakram: This migration isn't related to Django 5.2 It reflects a change that was made to the...

        • navinkarkera: @irtazaakram Thanks!

    • bradenmacdonald: LGTM

  • v0.20.0 v0.20.0 251355c [irtazaakram]

    ## What's Changed * chore: Upgrade Python requirements by @edx-requirements-bot in...

  • Add Django 5.2 support to openedx-learning #296 [feanil] maintenance

    Context Per the Open edX roadmap, this library has been identified as a Django dependency that needs to be updated...

  • Encapsulate child entity + entity versions in Containers API [FC-0083] #300 [pomegranited] open-source-contribution

    This PR addresses #278 (comment) by creating a dataclass that encapsulates a PublishableEntity.pk with an optional...

    • pomegranited: In #278 (comment) @bradenmacdonald said: I'd like to also introduce some sort of verification...

      • bradenmacdonald: I think it needs to be a separate query. Can we do something like...

      • pomegranited: Oh of course! Thanks @bradenmacdonald , see 8422317.

    • rpenido: LGTM 👍 Thank you for your work, @pomegranited! I tested this using the instructions from the...

      • rpenido: Does it make sense to use last_entities.filter(...) here instead of iterating with a conditional?

        • pomegranited: Yes it does! cf 84dc6f7

    • ChrisChV: Looks good! I will merge it tomorrow morning. CC @ormsbee

  • Subsections in Learning Core #301 [andrii-hantkovskyi] open-source-contribution

    This PR implements the new subsections functionality in Learning Core, which allows for organizing units within...

    • andrii-hantkovskyi: Hi @kdmccormick and @ormsbee, could you please review this PR when you have a chance?

    • kdmccormick: container.versioning.draft should always return an object with an entity_list. Are you finding...

      • andrii-hantkovskyi: Yeah, test_soft_deleting_unit is failing when calling authoring_api.contains_unpublished_changes...

  • Support for grouped changesets of draft modifications #290 [ormsbee]

    New Draft change-tracking models This introduces DraftChangeLog and DraftChangeLogRecord, which are mostly draft...

    • ormsbee: @kdmccormick, @bradenmacdonald: This should be in a fully reviewable state now. Two high level...

    • ormsbee: The edx-platform counterpart to this PR is: openedx/edx-platform#36513

    • ormsbee: Rebasing this now (need to account for the new migration added to the publishing app).

  • feat: add collections support for containers [FC-0083] #299 [rpenido] open-source-contribution

    Description This PR adds support for adding Containers to Collections. We moved the set_collections method from...

    • pomegranited: 👍 This is working great, thank you @rpenido ! I have suggested a couple other places where we...

      • pomegranited: nit: we can use the publishable_entity.learning_package_id, and then we don't have to also...

      • pomegranited: nit: we can make this query a little smaller by fetching a little less data here -- the...

      • pomegranited: It's quicker (and we only have to process a single m2m_changes signal) if we do this using the...

• Tutor PRs search pull requests [back to top]

  • fix: Added edx-platform patch related to instructor dashboard #135 [rehmansheikh222]    overhangio/tutor-indigo

    This PR adds edx-platform patch in the indigo plugin, currently the dark theme toggle does not work in the...

  • docs: Update edx.rtd.io links to docs.openedx.org #5 [sarina]    overhangio/mu

    With the move to docs.openedx.org, update outdated reference links. Ref: openedx/docs.openedx.org#988

    • regisb

  • docs: Update edx.rtd links to docs.openedx.org #7 [sarina]    overhangio/jupyter-xblock

    With the move to docs.openedx.org, update outdated reference links. Ref: openedx/docs.openedx.org#988

    • regisb

  • feat: add gitlab sync #8 [mlabeeb03]    overhangio/tutor-deck

    Sync the main and release branch of this repository to the main and release of gitlab. Fix indentation in all .yml...

    • DawoudSheraz: let's remove the first two as these are build specific changes meant for development and deployment flow.

    • DawoudSheraz: FYI: this is broken until the repository is public.

    • DawoudSheraz

  • fix: Switch to inverted Studio MFE waffle flags #247 [kdmccormick]    overhangio/tutor-mfe

    Part of: openedx/edx-platform#36275 Depends...

  • Update README #250 [sarina]    overhangio/tutor-mfe

    Update outdated edx.rtd.io link Update tCRIL ref Ref openedx/docs.openedx.org#988

    • DawoudSheraz

  • Update edx.rtd.io links to docs.openedx.org #1217 [sarina]    overhangio/tutor

    With the move to docs.openedx.org, update outdated reference links. Ref: openedx/docs.openedx.org#988

    • DawoudSheraz

  • v19.0.3 #62 [Danyal-Faheem]    overhangio/tutor-cairn

    Commit to release v19.0.3 to integrate changes related to: Fixed filters on the default course overview...

    • DawoudSheraz

  • feat: upgrade to teak #26 [Abdul-Muqadim-Arbisoft]    overhangio/tutor-webui

    v20.0.0 teak release Epic: overhangio/tutor#1214 Todos before merge Ensure changelog-entry exists for teak...

    • DawoudSheraz: the tutor core PR link is incorrect due to scriv added at the end.

      • Abdul-Muqadim-Arbisoft: Corrected

  • docs: make upgrade instructions compliant with tutor #65 [regisb]    overhangio/tutor-forum

    Some of the upgrade instructions were inconsistent, in the sense that they did not include tutor local run ......

  • fix: breaking docs due to incompatible table borders #64 [Danyal-Faheem]    overhangio/tutor-cairn

    The docs build was breaking because of incompatible number of borders against the column title letters. We also add...

    • DawoudSheraz: Other than fixing breaking docs, the change is also adding aspects in the table. The commit...

    • DawoudSheraz: I removed the bug label because this is a build issue and not a cairn plugin issue. Normally,...

  • docs: Update edx.rtd links to docs.openedx.org #63 [sarina]    overhangio/tutor-cairn

    With the move to docs.openedx.org, update outdated reference links. Ref: openedx/docs.openedx.org#988

    • Danyal-Faheem

    • Danyal-Faheem: Hi Sarina, thank you for the contribution. As a note it'd be great to see Aspects in this...

    • Danyal-Faheem: Hi @sarina, just wanted to let you know that we've added aspects in the table as well as part of #64.

  • fix: migrate to native filters on default course overview dashboard from filter box charts #53 [Danyal-Faheem]    overhangio/tutor-cairn

    Filter box charts have been completely deprecated as of superset 4.0.0. For this purpose, we move towards native...

    • regisb: This change is breaking deployment of the Open edX sandbox: Traceback (most recent call last): ...

  • test: verify python package distribution build when running make test #65 [Danyal-Faheem]    overhangio/tutor-cairn

    This is a recurring issue that we usually face whenever a new release is created and didn't show up in the existing CI.

    • DawoudSheraz: Let's add a make command as build-pythonpackage from tutor core (python -m build --sdist) and...

      • Danyal-Faheem: I have updated it to reflect the same way we are using it in tutor-core i.e, using make commands...

    • DawoudSheraz: To be consistent with gitlab flow, lets use similar pattern as twine check dist/$(echo...

      • Danyal-Faheem: I have updated it to reflect the same way we are using in tutor-core.

    • Danyal-Faheem: @DawoudSheraz The ci here is failing because the twine version needs to be >6 for this to work...

    • regisb: My 2 cents: if merging in release is a headache, I think it's fine to add this check to the main...

    • DawoudSheraz: @DawoudSheraz The ci here is failing because the twine version needs to be >6 for this to work...

  • fix: remove deprecated filter_scopes from import_dashboard script #66 [Danyal-Faheem]    overhangio/tutor-cairn

    Reference issue: https://github.com/overhangio/tutor-cairn/pull/53/files\#r2034772248 Filter_scopes have been...

    • DawoudSheraz

  • v19.0.4 #67 [Danyal-Faheem]    overhangio/tutor-cairn

    New release to fix the isue with dashboard import failing

    • DawoudSheraz

  • fix: correct language switch URL in header.html #139 [Yagnesh1998]    overhangio/tutor-indigo

    Problem The language switch dropdown in the Tutor Indigo theme was generating incorrect URLs due to the use of...

    • ahmed-arb

  • Add section on deploying MFE changes to production #251 [sarina]    overhangio/tutor-mfe

    Per discussion in https://discuss.openedx.org/t/operation-of-the-mfe/15396/5

    • DawoudSheraz: I think this section can be removed. The next section has the appropriate details (the codebase...

      • sarina: @DawoudSheraz Per discussion in https://discuss.openedx.org/t/operation-of-the-mfe/15396/5...

      • arbrandes: I believe this adds relevant information with more visibility, in a section with an anchor so we...

      • DawoudSheraz: Oh, I did not mean removing the entire section. I meant the first paragraph (🤦🏽 ). The second...

      • sarina: You're suggesting to remove this sentence (which provides context to people who already know...

    • DawoudSheraz

  • [DRAFT] feat! replace webpack by rsbuild for faster build #252 [regisb]    overhangio/tutor-mfe

    This is the result of a long investigation to accelerate the building of MFE apps in Open edX. Many of the...

    • bradenmacdonald: Is there anything you found that doesn't work / isn't supported by rsbuild? If not, we should...

    • regisb: Is there anything you found that doesn't work / isn't supported by rsbuild? No, I haven't....

    • hinakhadim: Awesome! I tested this PR on my system, and it significantly reduces the build time—from 2007...

    • bradenmacdonald: Instead, I now think that we should remove all of the build code from all MFE. There shouldn't...

    • Danyal-Faheem: I tested this out on an Arm64 machine (M1 Macbook) and it seems that the rsbuild step breaks on...

    • regisb: Thanks for testing @Danyal-Faheem! Would you mind trying out debugging with RUST_BACKTRACE, as...

  • chore: change app run command #9 [mlabeeb03]    overhangio/tutor-deck

    Change the run command to runserver.

    • regisb: This requires changing the Makefile and README. Basically, any interface change should have a...

created at 2025-04-14 07:11 by dinghy 1.4.1.