Commit Graph

5329 Commits

Author SHA1 Message Date
Justin Hynes
9e83bf8f94 feat: Add configuration option to route View Records button to the Learner Record MFE
[APER-1922]

We are converting the legacy UI of the `records` app in the Credentials IDA (credentials.edx.org/records/, credentials.edx.org/records/programs/{uuid}, etc.) to a new MFE. Today, the Program Dashboard and the legacy (non-MFE) profile page have buttons that route learners to the Credentials IDA pages. We need to (optionally) introduce a way to route learner's to the new MFE instead.

- Introduces a new configuration setting called `LEARNER_RECORD_MICROFRONTEND_URL` (defaulting to None). This will be used by the LMS to store the base URL of the new MFE (e.g. records.stage.edx.org).
- Introduces a new waffle switch named `USE_LEARNER_RECORD_MFE`. This will be used to control whether routing learner's to the new MFE is enabled from the LMS's side.
- Updates the existing `get_credentials_records_url` function to add additional logic that will determine if we need to build a link to the legacy FE or the new MFE
- Adds tests for new and existing behavior. There were no existing unit tests for the utility function that I updated.
2022-09-07 13:44:05 -04:00
Kshitij Sobti
3cd3c60b59 feat: enable unit-level control over discussions by default (#30903)
This PR changes the default behaviour of the discussions experience by making the previous "unit-level
visibility" the default mechanism for configuring discussions.

Prior to this PR, under the new discussions configuration experience, all units would automatically get
assigned a discussion topic and have discussions enabled for them (other than units in graded or exam
subsections). However, if authors wanted they could enabled a custom visibility mode which would allow
toggling discussions on or off on a per-unit level.

This PR makes this custom visibility mode the standard behaviour (and eventually, only behaviour)
and enables discussion for all units by default. This replicates the behaviour that already existed,
however, now gives authors control over disabling discussions for individual units by default.

It also removes the ability to disable discussions for all units (while still keeping course-wide
discussions) enabled.
2022-09-07 16:41:17 +05:00
Ahtisham Shahid
4541750eec feat: added tracking event on learner search (#30945) 2022-09-07 14:50:00 +05:00
Ahtisham Shahid
80b654676e fix: update free tier validation rule (#30932) 2022-09-06 16:59:07 +05:00
Shafqat Farhan
34a4cbf715 feat: VAN-1075 - Added country field validation on registration endpoint 2022-09-06 15:19:14 +05:00
Zainab Amir
c962583304 fix: remove legacy error messages (#30937) 2022-09-06 14:26:27 +05:00
AsadAzam
21e9490c66 fix: hide new provider if flag disabled (#30925) 2022-08-31 18:29:53 +05:00
jawad khan
dc256139f3 feat: Exchange jwt token with session cookies (#29963)
Exchange jwt token with session cookies so that it can work on mobile.
Mobile platform is migrating to jwt and for accessing  xblocks we need session cookies in exchnage of jwt token.
For additional details, see
https://github.com/openedx/edx-platform/blob/master/openedx/core/djangoapps/oauth_dispatch/docs/decisions/0013-mobile-migration-to-jwt.rst

LEARNER-8518
2022-08-31 13:29:52 +05:00
Mubbshar Anwar
2980265764 Revert "feat: Pass segment properties (#30742)" (#30918)
This reverts commit b76d5c2a66.
2022-08-30 12:11:25 +05:00
Mubbshar Anwar
b76d5c2a66 feat: Pass segment properties (#30742)
- For enrollment email pass extra segment event properties.
VAN-999
2022-08-29 15:36:24 +05:00
Mubbshar Anwar
1b509f3a81 perf: Performance improvement (#30907)
Course recommendation API performance improvement.

VAN-1061
2022-08-29 14:44:37 +05:00
Attiya Ishaque
614054bf10 fix: [VAN-1038] city field value is saved for optional fields. (#30897) 2022-08-25 16:58:00 +05:00
Kshitij Sobti
c2326f0399 fix: change conditions for showing new and legacy discussion providers (#30818)
The current logic for showing discussion providers makes it hard to switch
from the legacy to the new provider. This commit changes the conditions in
which different providers are shown, and which provider is used as default.

Before this commit, the new provider would be hidden if the legacy provider
was in use and vice-versa. So both would only be shown if neither legacy
nor the new provider were in use (i.e. an LTI provider was in use).

Now, all providers are always displayed to global staff. If the waffle
flag for the new provider is set
(`discussions.enable_new_structure_discussions`), then new provider is
always displayed, and the legacy provider is hidden unless it's currently
in use.

If flag is not set, then the new provider is always hidden unless it is
used by a course.

Finally, the default provider now depends on the flag above. If it is
set globally, then the default provider is the new provider, otherwise
the legacy provider remains the default provider.
2022-08-25 04:00:18 -07:00
Syed Sajjad Hussain Shah
a868646e98 fix: number of usersuggestions sometimes less than 3 [VAN-1023] (#30875) 2022-08-24 12:21:32 +05:00
Raymond Zhou
aafac8f69b Revert "feat: TNL-10051 blockstore API perf instrumentation" (#30885) 2022-08-22 16:51:53 -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
Peter Kabiri
41ee16a6b7 fix: Removed hardcoded social media and mobile store images. (#30630) 2022-08-17 17:51:06 +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
Attiya Ishaque
dafb9f7170 fix: Update to the new error messages and some fixes (#30826) 2022-08-11 15:39:49 +05:00
Zainab Amir
4f2aadb74e feat: reposition code to set is_marketable (#30839) 2022-08-10 18:47:50 +05:00
Zainab Amir
e73139d60f feat: add error handling to email task (#30820) 2022-08-04 13:03:03 +05:00
SaadYousaf
a948f535c1 feat: add flag to pass inIframe query_param for discussions MFE. 2022-08-03 15:39:02 +05:00
Marcin
86bd82c4fb feat: added flag for exams ida (#30815) 2022-08-02 13:52:38 -04:00
Robert Raposa
2af65d1b77 feat: add monitoring for users across requests (#30795)
We already add the user id (imperfectly) to many requests.
However, when a user starts off unauthenticated, it is not
possible to correlate to those requests. Adding the raw
IP chain provides that possibility. See new custom attribute
ip_chain.raw.
2022-07-28 13:45:34 -04:00
Muhammad Ammar
24dbf9653c Merge pull request #30792 from openedx/bilaltahir21/ENT-5938-fix
fix: revert api client for UNENROLL_DONE signal
2022-07-27 17:31:32 +05:00
Muhammad Bilal Tahir
a35176e09b fix: revert api client for UNENROLL_DONE signal 2022-07-27 16:35:59 +05:00
Muhammad Soban Javed
d053bba952 Revert "Revert "refactor: move common/lib/capa/capa to xmodule/capa" (#30762)"
This reverts commit 4463ee751d.
2022-07-27 15:36:08 +05:00
Muhammad Soban Javed
4463ee751d Revert "refactor: move common/lib/capa/capa to xmodule/capa" (#30762) 2022-07-21 18:22:15 +05:00
Muhammad Soban Javed
06064b237b Merge pull request #30403 from openedx/iamsobanjaved/BOM-2582-move-common-lib-capa
refactor: move common/lib/capa/capa to xmodule/capa
2022-07-21 16:27:47 +05:00
Robert Raposa
49dcb68a5e docs: fix restricted app adr status (#30677)
The decision to provide Restricted Applications expired
JWTs was superseded by another ADR. This commit simply
adds clarity around that change.

Co-authored-by: Phillip Shiu <pshiu@users.noreply.github.com>
2022-07-20 17:02:33 -04:00
connorhaugh
45c2858d61 Revert "feat: FC-0001 Remove Persistent Course Grades Flags (#30540)" (#30744)
This reverts commit 4a9243ef9f.
2022-07-19 13:45:31 -04:00
Sagirov Evgeniy
4a9243ef9f feat: FC-0001 Remove Persistent Course Grades Flags (#30540)
* feat: revert Fix certificate generation without persistent grades

* feat: Remove Persistent Course Grades Flags

* test: update tests
2022-07-19 11:30:37 -04:00
Soban Javed
9eba9f983a refactor!: move common/lib/capa/capa to xmodule/capa
As part of dissolving our sub-projects in edx-platform, we are moving this package under the xmodule directory.
We have fixed all the occurences of import of this package and also fixed all documents related references.
This might break your platform if you have any reference of `import capa` or `from capa import` in your codebase or in any Xblock.

Ref: https://openedx.atlassian.net/browse/BOM-2582
2022-07-19 12:20:04 +05:00
John Nagro
efa979a13d temp: guard against resume button failures in dropdown template (#30703) 2022-07-06 13:05:14 -04:00
Azan Bin Zahid
a3f9a27ed9 temp: add broad exception and log err for resume block (#30701) 2022-07-06 18:54:51 +05:00
Attiya Ishaque
9d1f31603e feat: [VAN-986] Add amplitude API (#30675)
* feat: [VAN-986] Add amplitude API

* feat: add suggestion

* feat: add spinner

Co-authored-by: Zainab Amir <zainab.amir@arbisoft.com>
2022-07-05 16:23:38 +05:00
Ahtisham Shahid
6f71e4cc0d fix: Made lti_config optional field (#30680)
Co-authored-by: AhtishamShahid <ahtishamshahid@A006-00850.local>
2022-06-30 19:09:14 +05:00
jawad khan
2dc7990653 feat: Added token grant_type in JWT payload (#30519)
To exchange jwt with session cookies we need to determine JWT grant type in
AccessTokenExchangeView. JWT only having password grant type will be allowed to exchange session.
Added ADR for mobile migration to JWT authentication.

LEARNER-8886
2022-06-30 16:49:11 +05:00
Waheed Ahmad
fb604e4345 fix: login endpoint username param rate limiting (#30673)
The username was allowed in the login endpoint alongside the email address
but rate-limiting logic was not updated to rate limit on the new POST
param `email_or_username`.

VAN-1003
2022-06-30 15:24:19 +05:00
Braden MacDonald
5d93f883f1 docs: Clarify a potentially confusing deprecation message 2022-06-29 12:13:39 -07:00
Ahtisham Shahid
8e8bb57b3b feat: Added flag for big blue button provider (#30645)
* feat: added flag for big blue button provider

* feat: Updated a comment

* fix: updated tests

* fix: updated serializer

* fix: resolved failed tests

Co-authored-by: AhtishamShahid <ahtishamshahid@A006-00850.local>
2022-06-28 16:34:10 +05:00
Mohammad Ahtasham ul Hassan
44fa09eba5 refactor: refactor discussions_xblock (#30636)
JIRA: https://openedx.atlassian.net/browse/BOM-2580
This PR aims at refactoring the discussion xblock sub project and moving it within the xmodule directory effectively removing its position as a sub project within edx-platform
2022-06-27 17:11:56 +05:00
Ahtisham Shahid
73c2b1be7e Added Big Blue button as live provider (#30613)
* feat: added new live provider and fixed tests

* feat: added free_tier compatiblity

* fix: resolved linter issues and other refactors

* fix: ran makemigration to generate migrations

* fix: Implemeted key retrival for tabs

Co-authored-by: AhtishamShahid <ahtishamshahid@A006-00850.local>
2022-06-23 11:48:44 +05:00
jawad khan
fe6b666f5b feat: exchange third party auth token with jwt token (#30283)
Exchange third party auth token wiith jwt token
since mobile platform is moving to jwt token we
need jwt token instead of access token now.

LEARNER-8517

Co-authored-by: Robert Raposa <rraposa@edx.org>
2022-06-21 14:58:39 -04:00
Muhammad Umar Khan
a389a9ff10 Revert "Revert "refactor: move xmodule folder to root"" 2022-06-20 18:20:06 +05:00
Muhammad Umar Khan
d890f06507 Revert "refactor: move xmodule folder to root" 2022-06-20 16:03:48 +05:00
M Umar Khan
a91df0c40f refactor: move xmodule folder to root
- Moving xmodule folder to root as we're dissolving sub-projects of common folder in edx-platform
    - More info: https://openedx.atlassian.net/browse/BOM-2579
- -e common/lib/xmodule has been removed from the requirements as xmodule has itself become the part of edx-platform and not being installed through requirements
- The test files common/lib/xmodule/test_files/ have been removed as they are not being used anymore
2022-06-20 14:33:45 +05:00