Commit Graph

66305 Commits

Author SHA1 Message Date
Brian Mesick
899703fc95 Merge branch 'master' into bmtcril/dockerhub_login 2025-01-09 16:04:09 -05:00
Brian Mesick
042c0441b0 fix: Use correct secret name for dockerhub password 2025-01-09 15:41:21 -05:00
Brian Mesick
18cf33864d build: Log in to dockerhub for unit test CI
We're being rate limited by dockerhub for pulling too many images in CI.
There isn't a greate solution for caching images for GH actions, but in
theory we should have less strict limits as an authenticated user.
2025-01-09 15:36:03 -05:00
github-actions[bot]
4a7d991074 feat: Upgrade Python dependency lti-consumer-xblock (#36088)
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>
2025-01-09 14:26:56 -05:00
Felipe Montoya
9acdfe6fd2 feat: show exception message from filters in the user interface (#35407) 2025-01-09 18:22:56 +01:00
Alison Langston
d15e520acc Revert "feat: Upgrade Python dependency lti-consumer-xblock (#36084)" (#36086)
This reverts commit 71965ea799.
2025-01-09 11:22:53 -05:00
Kyle D. McCormick
3dc28abcf7 build: Fix type annotations for new mypy version
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.
2025-01-09 10:31:01 -05:00
Kyle D. McCormick
9262c9aa42 build: Upgrade django-stubs even further, to fix mypy
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 https://github.com/python/mypy/issues/17958

Also includes an unrelated common_constraints update.
2025-01-09 10:31:01 -05:00
Kyle D. McCormick
f7ccd7cc3a build: Unpin mypy and upgrade it to latest
This is possible now that django-stubs and djangorestframework-stubs
are unpinned:
a5b773ce7b

which became possible once we upgraded to django 4.2.

Closes: https://github.com/openedx/edx-platform/issues/35667
2025-01-09 10:31:01 -05:00
github-actions[bot]
71965ea799 feat: Upgrade Python dependency lti-consumer-xblock (#36084)
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>
2025-01-09 10:08:30 -05:00
github-actions[bot]
03430adeb1 feat: Upgrade Python dependency edxval (#36082)
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>
2025-01-08 16:42:31 -05:00
David Ormsbee
8409a7c023 refactor: remove course-specific logging code in grading.
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.
2025-01-08 12:02:34 -05:00
edX requirements bot
f452798565 chore: Upgrade Python requirements (#36076)
* chore: Upgrade Python requirements

* chore: Upgrade Python requirements (#36079)

* feat: Upgrade Python dependency urllib3 (#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>
2025-01-08 16:45:31 +05:00
Ahtisham Shahid
85ecad1aab chore: added verification logs in edx.ace.message_sent event (#36074) 2025-01-07 00:36:56 +05:00
Feanil Patel
1d913bee8b Merge pull request #36063 from openedx/feanil/geoip2-bot-update-country-database-953140a
Update GeoLite Database
2025-01-06 11:22:17 -05:00
feanil
320e3bbc22 chore: geoip2: update maxmind geolite country database 2025-01-06 10:48:53 -05:00
Roman Edirisinghe
b510ceff93 fix: remove duplicate (unclosed) tag (#35682)
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.
2025-01-02 19:43:16 +00:00
Kyle D. McCormick
d36560d8c6 refactor: UsageKeyV2Serializer should be a BaseSerializer
...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.
2025-01-02 14:42:09 -05:00
Kyle D. McCormick
50944e9d75 build: Very explicitly annotate a model field so that it passes mypy
This clunky yet type-safe workaround to this django-stubs issue, which
arose when we upgraded django-stubs in the previous commits:
https://github.com/typeddjango/django-stubs/issues/1802
2025-01-02 14:42:09 -05:00
Kyle D. McCormick
a5b773ce7b build: upgrade django-stubs and djangorestframework-stubs
...now that we're on django>=4.2
2025-01-02 14:42:09 -05:00
Kyle D. McCormick
88b8da3e49 docs: Remove the last couple mentions of Paver from the codebase
Part of: https://github.com/openedx/edx-platform/issues/34467
2025-01-02 14:05:15 -05:00
kdmccormick
8a720c1b7b build!: Recompile Python dependencies, without Paver
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: https://github.com/openedx/edx-platform/issues/34467
2025-01-02 14:05:15 -05:00
Kyle D. McCormick
75e3107863 fix: Move implicit dependencies from paver.in to kernel.in
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: https://github.com/openedx/edx-platform/issues/34467
2025-01-02 14:05:15 -05:00
Kyle D. McCormick
2d5543a9ae feat!: Remove Paver
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: https://github.com/openedx/edx-platform/issues/34467
2025-01-02 14:05:15 -05:00
salmannawaz
38dc4eab5d chore: Ensure the return code is handled correctly to account for any unexpected behavior. (#36068)
* chore: Ensure the return code is handled correctly to account for any unexpected behavior.
2025-01-02 23:48:21 +05:00
Peter Pinch
e934c07aed Merge pull request #36054 from mitodl/anas/remove-md4
refactor!: remove md4 and related ENABLE_BLAKE2B_HASHING feature flag
2025-01-02 12:09:21 -05:00
Eemaan Amir
645e8985c2 chore: added a new info log to course update email (#36067)
* chore: added a new info log to course update email

* chore: added a new info log to course update email
2025-01-01 17:05:56 +05:00
Ahtisham Shahid
ab6947fb77 fix: resolved key error in notification pref apis (#36064) 2025-01-01 15:45:26 +05:00
Muhammad Adeel Tajamul
b369345dc1 fix: fixed notification generated event for grouped notifications (#36065) 2025-01-01 14:17:44 +05:00
Ahtisham Shahid
953140aa1b fix: resolved edx.course.goal.email.filtered parsing error (#36059) 2024-12-26 14:34:46 +05:00
Daniel Valenzuela
85f412864f fix: render library v2 assets with whitespace (#35974)
Assets that contain whitespace fail to be rendered when uploaded to library v2
2024-12-24 13:51:03 +00:00
Muhammad Anas
0de7569b68 refactor!: remove md4 and related ENABLE_BLAKE2B_HASHING feature flag 2024-12-24 13:12:18 +05:00
Muhammad Adeel Tajamul
81f825ff15 fix: new discussion post should only be grouped if old notification is not seen (#36032) 2024-12-24 07:47:38 +05:00
Abdur Rahman Asad
bfcaa138ff fix: log correct time and duration for video subsections (#36019)
* 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
2024-12-23 12:51:00 +05:30
Kyle McCormick
644e7e786a build!: requirements/edx-sandbox/py38.txt (#36049)
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.
2024-12-20 20:02:33 +00:00
jawad khan
76cceaa298 feat: Populate notification context with post, comment and response ids (#36008)
* feat: Populate notification context with post, comment and response ids
2024-12-20 15:23:47 +05:00
Kyle McCormick
edbcadae8e revert: build: unpin django-stubs now that we're on django >=4.2 (#36051)
This reverts commit b391fb5400,
which was unintentionally pushed to master instead of a PR.
2024-12-19 20:20:53 +00:00
Kyle D. McCormick
b391fb5400 build: unpin django-stubs now that we're on django >=4.2 2024-12-19 14:16:57 -05:00
Ahtisham Shahid
85a5890dd1 feat: added api to update all notification preferences for user (#35795) 2024-12-19 19:04:16 +05:00
Muhammad Adeel Tajamul
930989e5e6 fix: fixed notification generated event for grouped notifications (#36048) 2024-12-19 16:57:27 +05:00
Kyle McCormick
54250632d8 docs: ADR for Fixing Quality and JS Checks (#35741)
This is a record of the various existing CI issues we faced and the trade-offs
we made to move forward while replacing the Paver CI commands as part of:
https://github.com/openedx/edx-platform/issues/34467
2024-12-18 19:09:43 +00:00
Marlon Keating
ab765ce8f9 Merge pull request #36047 from openedx/marlonkeating/upgrade-edx-enterprise-e2a4b9e
feat: Upgrade Python dependency edx-enterprise
2024-12-18 10:30:32 -08:00
Marlon Keating
e1fa16166e Merge branch 'master' into marlonkeating/upgrade-edx-enterprise-e2a4b9e 2024-12-18 10:03:31 -08:00
Kyle D. McCormick
29d4c69e57 build!: Rejigger "npm run test" to be more intuitive
This is technically a breaking change, but these commands were added a
week ago and were not yet documented, so I'm not worried about breaking
anyone's workflow with this commit.
2024-12-18 11:26:49 -05:00
Kyle D. McCormick
41d28f3ce3 docs: Turn old static asset plan into a retroactive ADR 2024-12-18 11:26:49 -05:00
Kyle D. McCormick
4c0d94e367 docs: Update testing guide 2024-12-18 11:26:49 -05:00
marlonkeating
bdb1ae5315 feat: Upgrade Python dependency edx-enterprise
This change adds support for the page_size query parameter on the enterprise-customer-members endpoint.

Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master`
2024-12-18 16:19:29 +00:00
Ahtisham Shahid
e2a4b9e524 fix: updated goal reminder email time logger (#36045) 2024-12-18 15:38:05 +05:00
github-actions[bot]
98214a5416 feat: Upgrade Python dependency edx-enterprise (#36042)
Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master`

Co-authored-by: bcitro <67378070+bcitro@users.noreply.github.com>
2024-12-17 15:22:01 -05:00
github-actions[bot]
882475dd5e feat: Upgrade Python dependency edx-enterprise (#36041)
Bump version of edx-enterprise to v5.5.0.

Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master`

Co-authored-by: adamstankiewicz <2828721+adamstankiewicz@users.noreply.github.com>
2024-12-17 17:02:36 +00:00