chore: add mongo 7 to testing matrix #639
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
replacing non encrypted fields of moodle config model with encrypted ones Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master`
…x-enterprise-3ccd2d1 feat: Upgrade Python dependency edx-enterprise
The `logging.Logger.warn` method has been deprecated since Python 3.3 and is due to be removed all together in Python 3.13. See python/cpython#105377 `logging.Logger.warning` is the preferred and recommended way to log warnings. Fixes openedx/public-engineering#149 Co-authored-by: Lewis M. Kabui <[email protected]>
After we merged this PR: openedx#33920 this error began popping up in logs: Unable to load XBlock 'staffgradedxblock' .... ImportError: cannot import name 'get_course_blocks' from partially initialized module 'lms.djangoapps.course_blocks.api' (most likely due to a circular import) ... The root cause was the new imports of `derived_key` and `BlockKey` into xmodule/library_content_block.py. Those new imports come from xmodule/modulestore/store_utilities.py, which runs `XBlock.load_classes()` at the module level, which fails because we are still in the process of loading xmodule/library_content_block. As a solution, we move both `derived_key` and `BlockKey` to xmodule/util/keys.py. We could potentially move that file to opaque-keys eventually, depending on how well we think that those concepts generalize. Also: * We rename the function from derived_key to derive_key, as functions should be verbs. * We combine the first to parameters of derive_key (a source ContextKey and a source BlockKey) into a single parameter (a source UsageKey). In my opinion, this makes the function call easier to understand.
UsageKeys don't have a usage_id field. This line would have crashed if run.
…cate-in-facebook-improvements-for-edx.org
Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master` Co-authored-by: jansenk <[email protected]>
…penedx#33940) During the process of generatinng report for problem responses, there are two places where N + 1 query problem exist. In both cases, `StudentModule` objects are fetched and looped over where `student.username` field for each object is accessed. This result in a seperate database call to get the username for each student response. This problem is fixed by creating a join to fetch the related table in the original query using `select_related`. In a test conducted on report having 5000 `StudentModule` objects, the number of queries for the request reduced from 8363 to 29. The total time taken for the task reduced from 23764 ms to 7394 ms.
openedx#34033) * refactor: changed survey report message location and added a new informative sent state * chore: added Generate and Send text on button * chore: added comment to new state column and added reference to static jquery. * fix: fixed non Attribute error when a report hasn't been sent yet
still needs mocks for all tests to work FIXES: APER-2851
…cate-in-facebook-improvements-for-edx.org
Added json field in learner transmission audit to record 3 most response status information Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master`
…x-enterprise-00630cf feat: Upgrade Python dependency edx-enterprise
removed unencrypted user credentials data columns Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master`
…cate-in-facebook-improvements-for-edx.org
…x-enterprise-0a0d8d8 feat: Upgrade Python dependency edx-enterprise
Re-factored to make the code more testable, and added some more tests, also improved dry run logging FIXES: APER-2851
…cate-in-facebook-improvements-for-edx.org
not sure why the manual linter didn't complain FIXES: APER-2851
Fixing a linter air FIXES: APER-2851
improving a comment FIXES: APER-2851
includes a very long comment explaining this for future developers
…icate-share-certificate-in-facebook-improvements-for-edx.org feat: adds a certificate template modifier
feat: Remove edX branding on account deletion
edx enterprise version bump 4.12.4 Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master`
…e-edx-enterprise-d95ac60 feat: Upgrade Python dependency edx-enterprise
Co-authored-by: Zia Fazal <[email protected]>
feat: Adding a group membership to the EnterpriseCustomerUserReadOnlySerializer Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master` Co-authored-by: kiram15 <[email protected]> Co-authored-by: Kira Miller <[email protected]>
Makefile paver usage replaced with manage.py. This avoids running the production-unsuitable `pavelib.prereqs.install_prereqs` step during deployments.
feat: add list endpoint for course reset enabled enrollments
Add vanguards to CODEOWNERS VAN-1817
* feat: updated api to get all question type reponses * test: fixed and added new test cases
fix: Proximus learner transmission data failures Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master` Co-authored-by: sameenfatima78 <[email protected]>
Exposes the hide_from_toc xblock attribute so course authors can configure it as a section visibility option in Studio. Before this change, the Hide from TOC functionality was mainly used by OLX components. Hence, it wasn't available for configuration through the Studio UI. Still, its implementation existed in the platform and could be used by setting the attribute: hide_from_toc=true as part of the OLX definition. Ref: https://openedx.atlassian.net/wiki/spaces/OEPM/pages/3853975595/Feature+Enhancement+Proposal+Hide+Sections+from+course+outline
* feat: Upgrade Python dependency edx-enterprise Bump version of edx-enterprise to v4.13.0. Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master` --------- Co-authored-by: adamstankiewicz <[email protected]> Co-authored-by: Adam Stankiewicz <[email protected]>
…checkbox_master fix: course mode added to the metadata
DanielVZ96
force-pushed
the
dvz/mongo-7
branch
3 times, most recently
from
d13feaf
to
0481f9e
Compare
fire segment event for PWNED_PASSWORD on registration page password validation VAN-1830
Description: Add country field error message in api response VAN-1862
feat: fix pull_translations for production deployment
chore: edx-enterprise version bump
|
The upstream PR is merged now, so we can close this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Temporal in order to test openedx#34213