Kaustav/test master pr2 #749
Open
+111,099
−1,398,238
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
This was a temporary backcompat alias to requirements/edx-sandbox/quince.txt. Developers relying on this file to run a python3.8 sandbox should: * first, switch to requirements/edx-sandbox/quince.txt, which works with * python3.8 and then, upgrade their codejail sandbox to a newer `<release>.txt` * since python3.8 is EOL. See the readme in this directory for more details.
* fix: log correct time and duration for video subsections If a subsection of a video is added to a course using advanced settings, show and log correct start and end times to the users. * test: add unit tests
Assets that contain whitespace fail to be rendered when uploaded to library v2
* chore: added a new info log to course update email * chore: added a new info log to course update email
refactor!: remove md4 and related ENABLE_BLAKE2B_HASHING feature flag
… unexpected behavior. (openedx#36068) * chore: Ensure the return code is handled correctly to account for any unexpected behavior.
BREAKING CHANGE: Removes all remaining Paver commands including `pavelib/prereqs.py:*` and `pavelib/assets.py:*`. BREAKING CHANGE: Removes `./manage.py [lms|cms] compile_sass`, which was just a wrapper around Paver commands. BREAKING CHANGE: Removes paver.txt. Operators should install testing.txt instead. Part of: openedx#34467
There are three packages which edx-platform needs in order to run which were being installed transitively via paver.in. Since we are removing paver.in, these dependencies need to be transferred into kernel.in: * psutil * pymemcache * wrapt Part of: openedx#34467
BREAKING CHANGE: Removes libsass from requirements/edx/base.txt. Operators will need to install requirements/edx/assets.txt in order to compile Sass. Commit generated by workflow `kdmccormick/edx-platform/.github/workflows/compile-python-requirements.yml@refs/heads/master` Part of: openedx#34467
...now that we're on django>=4.2
This clunky yet type-safe workaround to this django-stubs issue, which arose when we upgraded django-stubs in the previous commits: typeddjango/django-stubs#1802
...not a Serializer, as it overrides to_representation to a str rather than a dictionary. This type error arose during the djangorestframework-stubs upgrade in the previous commits.
This PR removes some bad (duplicated) markup in the static tab file. The current behavior leads to an unclosed HTML tag, which can be confirmed by viewing the document source.
…e-country-database-953140a Update GeoLite Database
* chore: Upgrade Python requirements * chore: Upgrade Python requirements (openedx#36079) * feat: Upgrade Python dependency urllib3 (openedx#36081) upgrade: upgrade urllib3 version Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master` Co-authored-by: mumarkhan999 <42294172+mumarkhan999@users.noreply.github.com> --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: mumarkhan999 <42294172+mumarkhan999@users.noreply.github.com>
This removes special logging for course course-v1:UQx+BUSLEAD5x+2T2019 as part of an edX investigation (EDUCATOR-4568 in their internal JIRA). That issue was closed on 2021-01-08 by Mat Carter with the comment: > Aged out, closing. If this issue continues, please create a new > ticket optionally referencing this one. (Note that Mat has since left edX/2U.) The EDUCATOR-4602 ticket was created to track the cleanup of this logging code.
Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master` Co-authored-by: jansenk <1639231+jansenk@users.noreply.github.com>
The latest version replaces pyjwkset with the pyjwt package Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master` Co-authored-by: alangsto <46360176+alangsto@users.noreply.github.com>
This is possible now that django-stubs and djangorestframework-stubs are unpinned: openedx@a5b773c which became possible once we upgraded to django 4.2. Closes: openedx#35667
Rather than constraining django-stubs' major version to our django major version (4.x.x), we are going to go one ahead (5.x.x), as recommended by python/mypy#17958 Also includes an unrelated common_constraints update.
Includes some new Request type annotations in openedx.core.types.http, plus a new meta-utility @type_annotation_only to ensure that we don't accidentally start instantiating those new classes.
This file is very out of date so just buliding it from scratch to get the latest versions of 2nd order dependencies that are still compatible with our explicit dependencies.
build: Regenerate package-lock.json
…#36325) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
This is a pure refactoring of cms/envs/production.py, removing several redundant statements that have accrued over the years as the platform moved from python-only, to python+json, to python+json+yaml, to today's python+yaml setup. This is the CMS version of: * a81493c * (originally 1593923) Also included: * Add some more explicit structure to the both LMS's and CMS's production.py using big comments. * In both LMS and CMS settings, alphabetize the production overrides, and remove the extraneous comments. Separate out the handful of settings which have useful comments. The rest of the settings' comments were not helpful--they were either just stating the obvious, or they were duplicative of what's documented in common.py. Co-Authored-By: Feanil Patel <feanil@axim.org> Part of: openedx#36215
This package polyfills the Fetch api but that API is now widely available so I don't think we need this package anymore.
build: Drop the `whatwg-fetch` polyfill.
LIBRARY_ENABLED_BLOCKS added to verify enabled blocks in get_allowed_block_types
Adds a new field `user_fk` to EnterpriseCustomerUser table. Part 1 of multiple releases to add a user foreign key. Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master`
This adds a toggle for running codejail in both remote and local configurations for testing purposes. edx/edx-arch-experiments#895
… detailed docstring - Updated `generate_username_suggestions` function to include validation for non-ASCII characters. - Improved function documentation to clarify arguments, return types, and username generation logic. - Added type hints for better code clarity and maintainability.
- Introduced `remove_special_characters_from_name` and `generate_username_suggestions` functions to enhance username handling. - Added comprehensive test cases for username generation, including ASCII validation and uniqueness checks. - Implemented tests for special character removal and suggestion generation based on various input scenarios, including edge cases.
- Removed unnecessary blank lines in the add-remove-label-on-comment.yml and check_python_dependencies.yml files to improve readability and maintainability.
…penedx#36265) Fixes errors thrown when previewing/editing (built-in) LTI blocks in a Libraries v2 context. These fixes allow Library Authors to store LTI blocks in their content libraries.
…36333) Replace image with its alt text while converting html to plain text for indexing.
fix(deps): update dependency uglify-js to v3
…Course Optimizer (openedx#36321) * fix: ignore drag and drop in course optimizer scanning links step
* feat: only fetch published pathways
…ent toggles working
Sets up tracing in code jail calls to track remote and local execution, and adds custom attributes to track error messages from the calls. edx/edx-arch-experiments#895
|
Thank you for your pull request! Congratulations on completing the Open edX tutorial! A team member will be by to take a look shortly. |
|
Sandbox deployment successful 🚀 |
|
Sandbox deployment successful 🚀 |
|
Sandbox deployment successful 🚀 |
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.
Master PR for tests