Commit Graph

61816 Commits

Author SHA1 Message Date
michaelroytman
8fd2c9849f feat: decouple LTI handlers from XBlock and improve LTI error handling for unauthenticated users
This commit updates the version of the lti-consumer-xblock from 4.3.3 to 4.5.0. This installs the newest version of the lti-consumer-xblock library. This version includes the following changes, from versions 4.4.0 and 4.5.0 of the library.

* Move LTI XBlock Handlers to Django Plugin (4.4.0)

1. The functionality of LTI 1.3 Launch Handler is moved from the XBlock to the Django plugin.
2. The functionality of the Access Token endpoint is moved from XBlock to the Django plugin.
3. A new URL format using the LtiConfiguration ID is introduced for the Access Token endpoint and is used when a LTI Consumer is configured without a location allowing LTI integrations to be created without the XBlock context.
4. A new URL format using the LtiConfiguration ID is introduced for the Keyset Endpoint and is used with the location of the XBlock is not available in the configuration.

* Handle LtiError Error During LTI 1.1 Launch for Unauthenticated User (4.5.0)

1. Improve error handling for LTI errors raised during LTI 1.1 launch when user is unauthenticated by returning a 400 response instead of falling through to the 500 error.
2. Change the error template to be generic to both LTI 1.1 and LTI 1.3 launches.
3. Revert logging amendments that were made to investigate the nature of the LTI errors.
2022-08-22 15:11:33 -04:00
bszabo
2669da94aa Merge pull request #30881 from openedx/bszabo/TNL-10051-instrument-blockstore-api
feat: TNL-10051 blockstore API perf instrumentation
2022-08-22 13:47:24 -04:00
Bernard Szabo
d58607ea2d feat: TNL-10051 reorder import statements
import logging must come near top
2022-08-22 13:19:53 -04:00
Bernard Szabo
e4d5de09da feat: TNL-10051 fix whitespace violation
Need two blank lines, not one
2022-08-22 12:55:50 -04:00
Bernard Szabo
2a9ce209f1 feat: TNL-10051 blockstore API perf instrumentation
Log before & after making blockstore (Django) API calls to get performance from timestamps
2022-08-22 12:22:20 -04:00
Justin Hynes
e02fabbda1 Merge pull request #30851 from openedx/jhynes/fix_devstack_credentials_port_numbers
fix: Override Credentials service URLs in Studio devstack.py files
2022-08-22 08:33:26 -04:00
Shafqat Farhan
b485674815 Merge pull request #30878 from openedx/shafqat/VAN-1051
feat: VAN-1051 - Integrated Optimizely fullstack client and experiment
2022-08-22 15:12:55 +05:00
Shafqat Farhan
a791759202 feat: VAN-1051 - Integrated Optimizely fullstack Client and experiment 2022-08-22 14:33:25 +05:00
Jenkins
061492606e chore(i18n): update translations 2022-08-21 16:57:24 -04:00
Jansen Kantor
29f41c8a81 chore: bump submissions to get json encoding fix (#30876)
* chore: bump submissions to get json encoding fix

* fixup! chore: bump submissions to get json encoding fix

* chore: version bump again
2022-08-19 10:07:11 -04:00
Muhammad Bilal Tahir
cae28b9315 Merge pull request #30864 from openedx/bilal/ENT-6109
chore: added a whitelist urls in devstack
2022-08-19 15:14:33 +05:00
Muhammad Bilal Tahir
10509bc27b chore: added whitelist urls in devstack as reesult of ENT-6109 2022-08-19 01:02:19 +05:00
Jenkins
a13b9cf975 chore: geoip2: update maxmind geolite country database 2022-08-17 16:52:16 -04:00
John Nagro
0c4e59f389 feat: release edx-enterprise 3.56.5 (#30873) 2022-08-17 16:45:17 -04:00
Nathan Sprenkle
e0199b2199 feat: learner home contract updates (#30871)
* feat: add startDate to course run serializer

* fix: remove unused certificate serializer fields

Co-authored-by: nsprenkle <nsprenkle@2u.com>
2022-08-17 16:40:51 -04:00
connorhaugh
dd1367823d Revert "feat: implement V2 libraries usage for library content block (#30615)" (#30872)
This reverts commit dcf1cc14b6.
2022-08-17 14:43:31 -04:00
Diana Huang
3946503960 Revert "feat: Update the cache when active enterprise is changed. (#30866)" (#30870)
This reverts commit 354d2be845.
2022-08-17 12:18:04 -04:00
Peter Kabiri
41ee16a6b7 fix: Removed hardcoded social media and mobile store images. (#30630) 2022-08-17 17:51:06 +05:00
Zaman Afzal
354d2be845 feat: Update the cache when active enterprise is changed. (#30866) 2022-08-17 17:29:21 +05:00
Ahtisham Shahid
69efe48e09 fix: Made user search function case in-sensitive (#30867) 2022-08-17 13:39:22 +05:00
Adam Blackwell
f03f744863 fix: Fixed link to pull request template 2022-08-16 20:56:25 -04:00
Adam Blackwell
0046628fe2 chore: Add link to important doc (#30834) 2022-08-16 20:55:33 -04:00
Nathan Sprenkle
ac07f23e25 feat: update learner home mock data (#30865)
* feat: add startDate to mock data

* feat: remove certDownloadUrl

Co-authored-by: nsprenkle <nsprenkle@2u.com>
2022-08-16 15:43:26 -04:00
Nathan Sprenkle
48e03e7457 feat: learner home mock bff (#30858) 2022-08-16 12:59:32 -04:00
Kyle McCormick
c253ec4181 feat!: change /api/v1/mfe_config to /api/mfe_config/v1
* This changes the API's path. The reasoning is that this is Version 1 of
  the mfe_config API, not Version 1 of the LMS's entire API, so the v1
  should come after mfe_config.
* Why does this matter? Firstly, consistency. Secondly, it affects our
  generated API documentation. If you visited
  https://courses.edx.org/api-docs, you could see that the API was
  listed under "v1" instead of "mfe_config".
2022-08-16 11:38:53 -04:00
Kyle McCormick
1b52ad58a5 docs: add more detail to MFE Config API documentation
No functional changes here. This just uses the edx_api_doc_tools package
to add some additional documentation to the new API. The documentation
can be read from the code, or viewed by visiting
http://<LMS_ROOT>/api-docs and searching for "mfe_config".
2022-08-16 11:38:53 -04:00
Kyle McCormick
8edefe74ff feat!: change names of dynamic MFE config Django settings
Formerly, the settings were:

* `MFE_CONFIG` for common config.
* `MFE_CONFIG_<APP_ID>` for app-specific overrides,
  with each app getting its own Django setting.

This commit changes it to:

* `MFE_CONFIG` for common config (unchanged)
* `MFE_CONFIG_OVERRIDES` for app-specific overrides,
  where each app gets a top-level key in the dictionary.

Why the change?

* We want common.py to have a complete list of overridable settings, as
  it helps operators reason about configuration and allows us to generate
  config documentation using toggle annotations. Dynamically generating
  setting names based on arbitrary APP_IDs makes this impossible.
* getattr(...) generally makes code more complicated bug prone. Tools
  like pylint and mypy cannot effectively analyze any code that uses
  dynamic attribute access.
2022-08-16 11:38:53 -04:00
Sagirov Evgeniy
0ff6d50734 [BD-14] feat: Take the Library Content xblocks out the Advanced section to the top default level (#30803)
* feat: Take the Library Content xblocks out the Advanced section to the top default level

* feat: remove library component from library category
2022-08-16 08:25:46 -04:00
Eugene Dyudyunov
dcf1cc14b6 feat: implement V2 libraries usage for library content block (#30615)
YT: https://youtrack.raccoongang.com/issue/EDX_BLND_CLI-87

- V2 libraries are available for selection in the Random Block edit modal;
- selected V2 library blocks are copied to the modulestore and saved as children of the Random Block;
- V2 library version validation works the same as for the V1 libraries (with possibility to update block with the latest version);
- filtering by problem type can't be done for V2 the same as for V1 because the v2 library problems are not divided by types;
- the problem type field is hidden for v2 libraries in the edit mode;
- unit tests added/updated.
2022-08-16 08:25:26 -04:00
Zainab Amir
f6e886b027 feat: remove unused management commands (#30861)
Removed two commands:
- populate_is_marketable_user_attribute
- populate_user_data_on_braze

VAN-971
2022-08-16 15:10:20 +05:00
Jenkins
7313803e39 chore: geoip2: update maxmind geolite country database 2022-08-15 13:14:55 -04:00
Robert Raposa
30ba3c355d fix: add mapping for learner-pathway-progress (#30857)
Adds code owner mapping for learner-pathway-progress.
2022-08-15 12:21:43 -04:00
Kristin Aoki
a210ffae38 feat: update template text for raw html editor 2022-08-15 11:12:53 -04:00
edx-semantic-release
c6ebd1ca01 chore(i18n): update translations 2022-08-14 17:02:53 -04:00
Jansen Kantor
eec0e4a562 temp: add additional logging to debug problem score issue (#30850)
* temp: add additional logging to debug problem score issue

* fixup! temp: add additional logging to debug problem score issue
2022-08-12 13:46:30 -04:00
John Nagro
dd88a4201e feat: release edx-enterprise 3.56.2 (#30852) 2022-08-12 12:41:31 -04:00
Navin Karkera
b0c39758fb fix: mark a text for i18n (#30840) 2022-08-12 10:58:55 -04:00
Justin Hynes
45d7182d12 fix: Override Credentials service URLs in Studio devstack.py files
In the `cms/envs/common.py` file the `CREDENTIALS_INTERNAL_SERVICE_URL` and `CREDENTIALS_PUBLIC_SERVICE_URL` settings are set to
`http://localhost:8005` which is incorrect for devstack. In devstack the Credentials IDA runs on port `18150`. This causes
issues with Studio being able to communicate with the Credentials IDA.

I've overriden the service URL settings in devstack.py to point to the correct port.
2022-08-12 14:18:07 +00:00
Muneeb Ur Rehman
fd06c5b399 chore: bump the version of learner-pathway-progress (#30849) 2022-08-12 17:54:33 +05:00
Ahtisham Shahid
dcb01d107f fix: Temporary fix for learners stats api performance issue (#30847)
* fix: Temporary fix for learners' stats API performance issue

* fix: resolved linter errors

* fix: learners stats API response is now null

* fix: changed waffle dates

* fix: resolved unit test issue
2022-08-12 17:24:22 +05:00
Saad Yousaf
067ebd79ce fix: fix inIframe query param for discussion MFE link (#30846)
Co-authored-by: SaadYousaf <saadyousaf@A006-00314.local>
2022-08-12 15:13:06 +05:00
Leangseu Kim
719ddff238 fix: section.due datetime version issue 2022-08-11 13:53:11 -04:00
Nathan Sprenkle
e025463872 feat: Learner Dashboard get enrollments (#30808)
* feat: fill out enrollment serializer

* test: add basic integration tests for enrollments

* feat: get info for user account activation

* test: test integrating user account activation

* feat: get course enrollments

* feat: get course email settings

* feat: add ecommerce info

* feat: add resume urls

* refactor: move learner home to separate app

* refactor: remove course limit

Co-authored-by: nsprenkle <nsprenkle@2u.com>
2022-08-11 11:05:07 -04:00
Attiya Ishaque
dafb9f7170 fix: Update to the new error messages and some fixes (#30826) 2022-08-11 15:39:49 +05:00
Muhammad Adeel Tajamul
f24af41b98 feat: redirect method for learners to new MFE (#30769)
Co-authored-by: adeel.tajamul <adeel.tajamul@arbisoft.com>
2022-08-11 11:49:45 +05:00
connorhaugh
6b6935b8f9 Revert "feat: add new template text for raw html editor" (#30843)
This reverts commit 95a13bff5d.
2022-08-10 15:17:20 -04:00
Alexander J Sheehan
58199ee670 Merge pull request #30842 from openedx/asheehan-edx/bumping-enterprise-3-56-1
chore: bumping enterprise requirements to 3.56.1
2022-08-10 13:16:36 -04:00
Alexander Sheehan
a5992fa080 chore: bumping enterprise requirements to 3.56.1 2022-08-10 12:36:20 -04:00
John Nagro
691a866b87 feat: release edx-enterprise 3.56.0 (#30841) 2022-08-10 11:16:18 -04:00
Zainab Amir
4f2aadb74e feat: reposition code to set is_marketable (#30839) 2022-08-10 18:47:50 +05:00