Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity
60,618 Commits 1 Branch 0 Tags
0ad9959667ffe5ba081a3fb6d8b1455e4e204d65
Commit Graph

60618 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
ChrisChV
0ad9959667 feat: Wrap the choice label with a <div> on choicegroup.html 2021-12-21 14:01:01 -05:00
Michael Roytman
492b17899b Merge pull request #29626 from edx/mroytman/MST-1254-change-_does_name_change_require_verification-to-use-passing-cert-statuses 
Do Not Prevent Learner From Changing Name if Learner Is Not Enrolled in Verified Mode or Learner Has Non-Passable Certificate
 2021-12-17 11:04:35 -05:00
Zainab Amir
bc156a6d04 feat: send perf-lang custom attribute (#29632) 2021-12-17 14:12:48 +05:00
Long Lin
dee42c6498 chore: bump edx-enterprise version 2021-12-16 15:47:04 -05:00
michaelroytman
ea3dc9cad6 fix: Do Not Prevent Learner From Changing Name With Non-Verified Enrollment/Non-Passable Certificate 
The _does_name_change_require_verification(user_profile, old_name, new_name) method of the accounts user_api determines whether a learner can change their name from old_name to new_name. Originally, it delegated solely to the NameChangeValidator class of the edx-name-affirmation API, which ran a set of checks against the name change. One of said checks was asserting that learners with one or more certificates could not change their name without completing IDV.

This pull request changes this behavior.

Learners may have certificates that are not in a passable status (e.g. "unverified"). We only want to require IDV for name changes for learners that have passing statuses. The existing code prevented learners from changing their name if they had any certificates at all, irrespective of the certificate status. This change only considers certificates in a passable status.

Additionally, learners may have certificates and also not be enrolled in any "verified" seats. For example, despite edX no longer offering "honor" seats, learners may have enrollments in "honor" modes, which grant certificates but are not considered "verified" enrollment modes. IDV requires that a learner be enrolled in a "verified" seat in order to complete IDV. Prior to this change, learners that were navigated to IDV to validate a name change were unable to complete IDV. This change introduce a check that a learner is in a "verified" mode in addition to using the NameChangeValidator. This prevents the account MFE from navigating an IDV-ineligible learner to IDV.

MST-1254: https://openedx.atlassian.net/browse/MST-1254
 2021-12-16 14:26:07 -05:00
Waheed Ahmed
c1e4109396 fix: org image url in save for later event props (#29625) 2021-12-16 22:55:07 +05:00
Bianca Severino
93e5d6ce50 Merge pull request #29544 from edx/bseverino/remove-idv-upgrade 
[MST-1195] Hide course upgrade on dashboard if there is no SKU
 2021-12-16 12:02:53 -05:00
Julia Eskew
05ff94a993 feat: Change the text of the Studio button to add an HTML component 
to read "Text" and display a text icon instead of a "</>" icon.
 2021-12-16 11:59:10 -05:00
Bianca Severino
ae8e9672a1 feat: hide course upgrade on dashboard if there is no SKU 2021-12-16 11:19:31 -05:00
Zainab Amir
85c6cea965 fix: use settings instead of config (#29622) 2021-12-16 18:00:31 +05:00
Zainab Amir
063d11a866 fix: update save_for_later api endpoint (#29620) 
fix: update save_for_later api endpoint
 2021-12-16 14:47:52 +05:00
Syed Muhammad Dawoud Sheraz Ali
74cce08c48 Merge pull request #29445 from regisb/regisb/fix-language-cookie-samesite 
fix: language cookie "samesite" attribute
 2021-12-16 14:42:04 +05:00
Jawayria
0d2eb75f09 Merge pull request #29602 from edx/jawayria/fix-ugettext-lms7 
fix: replace 'ugettext' with 'gettext' in lms
 2021-12-16 14:12:39 +05:00
Muhammad Soban Javed
ac7a42d196 Merge pull request #29520 from edx/iamsobanjaved/pytest-warnings-gha 
test: save pytest warning reports to GHA artifacts
 2021-12-16 14:10:15 +05:00
Jawayria
da009d6bdd Merge pull request #29600 from edx/jawayria/fix-ugettext-lms5 
fix: replace 'ugettext' with 'gettext' in lms
 2021-12-16 13:59:18 +05:00
Jawayria
1bcb9adfe1 Merge pull request #29601 from edx/jawayria/fix-ugettext-lms6 
fix: replace 'ugettext' with 'gettext' in lms
 2021-12-16 13:59:06 +05:00
Jawayria
d96d3a2187 Merge pull request #29603 from edx/jawayria/fix-ugettext-lms4 
fix: replace 'ugettext' with 'gettext' in lms
 2021-12-16 13:58:53 +05:00
Jawayria
e8be2e7923 Merge pull request #29604 from edx/jawayria/fix-ugettext-lms3 
fix: replace 'ugettext' with 'gettext' in lms
 2021-12-16 13:58:42 +05:00
Jawayria
750ed8c837 Merge pull request #29605 from edx/jawayria/fix-ugettext-lms1 
fix: replace 'ugettext' with 'gettext' in lms
 2021-12-16 13:58:31 +05:00
Jawayria
df5345afe6 fix: replace 'ugettext' with 'gettext' in lms 2021-12-16 13:23:36 +05:00
Jawayria
b2113696c9 fix: replace 'ugettext' with 'gettext' in lms 2021-12-16 13:22:40 +05:00
Jawayria
8e93280ec4 fix: replace 'ugettext' with 'gettext' in lms 2021-12-16 13:20:29 +05:00
Jawayria
0c7e37bc38 fix: replace 'ugettext' with 'gettext' in lms 2021-12-16 13:19:40 +05:00
Jawayria
3dfbb9f44e fix: replace 'ugettext' with 'gettext' in lms 2021-12-16 13:18:28 +05:00
Jawayria
f72a6e3a79 fix: replace 'ugettext' with 'gettext' in lms 2021-12-16 13:15:27 +05:00
Régis Behmo
53b8a80668 fix: language cookie "samesite" attribute 
The language cookie "samesite" attribute was always set to "None", even in
non-secure environments, such as the devstack. This was causing client-side
warnings in non-https environments, and the language cookie was not properly
set.
 2021-12-16 09:04:26 +01:00
edX Transifex Bot
b401ecb3e7 geoip2: update maxmind geolite country database 2021-12-15 16:19:32 -05:00
Ned Batchelder
e39f9c503a build: suppress Django deprecation warnings (#29528) 
Suppress them both in tests (via setup.py and pytest.ini)
and in management command & application runs
(via logsettings.py).

Developers aren't looking at these warnings; they'll be dealt with in a
formal process for upgrading Django.  Suppress them for now so that
important information isn't lost in the noise.
 2021-12-15 10:45:12 -05:00
Zainab Amir
dd610118a7 feat: create braze alias and event (#29612) 2021-12-15 19:07:25 +05:00
Soban Javed
b4ac8d284a test: save pytest warning reports to GHA artifacts 
- add pytest_hooks import back in common/lib/conftest.py. This import was removed during refactoring work
 2021-12-15 17:06:40 +05:00
edX requirements bot
cf64f0e198 chore: Updating Python Requirements (#29610) 2021-12-15 16:15:53 +05:00
Jawayria
833ce7d586 Merge pull request #29570 from edx/jawayria/fix-url-common 
fix: fixed RemovedInDjango40 warnings in common
 2021-12-15 13:55:37 +05:00
Jawayria
293edae5e6 Merge pull request #29573 from edx/jawayria/fix-url-cms 
fix: replace 'django.conf.url()' with 'django.urls.re_path()'
 2021-12-15 13:55:13 +05:00
Jawayria
13d502353a Merge pull request #29574 from edx/jawayria/fix-ugettext-cms1 
fix: replace 'ugettext' with 'gettext' in cms
 2021-12-15 13:54:57 +05:00
Jawayria
c1276cd122 Merge pull request #29577 from edx/jawayria/fix-ugettext-cms2 
fix: replace 'ugettext' with 'gettext' in cms
 2021-12-15 13:54:42 +05:00
Jawayria
323dd7184b Merge pull request #29578 from edx/jawayria/fix-ugettext-cms3 
fix: replace 'ugettext' with 'gettext' in cms
 2021-12-15 13:54:27 +05:00
Jawayria
ccfe993cb8 Merge pull request #29579 from edx/jawayria/fix-ugettext-cms4 
fix: replace 'ugettext' with 'gettext' in cms
 2021-12-15 13:54:13 +05:00
Michael Terry
34138c1516 Merge pull request #29606 from edx/mikix/401-for-anon 
fix: have learning sequence give 401 for anon users instead of 404
 2021-12-14 12:00:39 -05:00
Michael Terry
a630747695 fix: have learning sequence give 401 for anon users instead of 404 
This will avoid leaking whether a course exists or not to anonymous
users and also avoid some false-positive error rates when web
crawlers hit bad URLs.
 2021-12-14 11:32:50 -05:00
Jawayria
bd5e00b725 fix: fixed RemovedInDjango40 warnings in common 2021-12-14 19:55:01 +05:00
Jawayria
c320d5cd1b fix: replace 'django.conf.url()' with 'django.urls.re_path()' 2021-12-14 19:53:24 +05:00
Jawayria
a1ff38819a fix: replace 'ugettext' with 'gettext' in cms 2021-12-14 19:51:13 +05:00
Jawayria
d1cd6e8b60 fix: replace 'ugettext' with 'gettext' in cms 2021-12-14 19:49:04 +05:00
edX requirements bot
66436c2e79 chore: Updating Python Requirements (#29587) 2021-12-14 16:13:05 +05:00
Matt Hughes
a92f372a3e fix: redirect behavior to learner-specific progress pages 
JIRA:AU-375
 2021-12-13 15:52:00 -05:00
John Nagro
3873ff7de9 feat: avoid certain strings in the suffix appended to edX usernames (#29524) 
ENT-2824
 2021-12-13 15:32:51 -05:00
John Nagro
2d3e774db1 feat: Show OAuth Auth link for Blackboard Admin (#29584) 
- edx-enterprise version bump 3.36.6

ENT-4389
 2021-12-13 14:01:20 -05:00
Dillon Dumesnil
fb6b2610fc Merge pull request #29581 from edx/ddumesnil/update-backpopulate-user-tours-command 
fix: Only evaluate queryset once
 2021-12-13 09:52:17 -08:00
Jawayria
4b69973701 Merge pull request #29367 from edx/jawayria/bom-2606-1 
chore: applied lint-amnesty on common/djangoapps/{edxmako, split_modulestore_django, static_replace, third_party_auth, track}
 2021-12-13 21:58:43 +05:00
Jawayria
e7c62ff61b Merge pull request #29368 from edx/jawayria/bom-2606-2 
chore: Applied lint-amnesty on common/djangoapps/student
 2021-12-13 21:58:31 +05:00