Commit Graph

23014 Commits

Author SHA1 Message Date
Michael Terry
789edbe555 Merge pull request #28256 from edx/mikix/course-home-access
feat: send course access error along to course home MFE
2021-07-26 13:34:54 -04:00
Justin Hynes
49bcdfe481 Merge pull request #28245 from edx/jhynes/microba-1227_cleanup
fix: remove disabled v1 course certificate queue code
2021-07-26 10:31:27 -04:00
Michael Terry
29159891c3 feat: send course access error along to course home MFE
We previously only sent it to the courseware MFE. Also unify the
naming of this access error to 'course_access' in courseware and
course home APIs.
2021-07-23 14:01:58 -04:00
Andrew Shultz
27ee8aee4a Merge pull request #28255 from edx/ashultz0/proc-bump-3206
chore: update edx-proctoring to 3.20.6
2021-07-23 09:58:32 -04:00
Andy Shultz
2e0393d098 fix: update platform test that directly creates proctor attempts
Student name field is gone. This test was providing a blank name so nothing
more than delete is needed.

MST-872
2021-07-22 16:44:11 -04:00
Thomas Tracy
abccf1ee26 [feat] Add enrollment mode to outline API (#28234) 2021-07-22 13:33:14 -04:00
Justin Hynes
0069bd724f fix: remove disabled v1 course certificate queue code
[MICROBA-1227]
[DEPR-155]

* remove queue.py (and associated unit tests)
* remove references to `queue.py` from functions in api.py (also update unit tests)
2021-07-22 11:03:49 -04:00
Matt Tuchfarber
325629dd25 Merge pull request #28237 from edx/tuchfarber/move_may_certify_2
Move cert display decisions to certificates app
2021-07-22 10:20:13 -04:00
Dillon Dumesnil
f886ced838 Merge pull request #28233 from edx/ddumesnil/hide-content-after-due-progress-aa-912
fix: AA-912: Hide URL after due date if subsection is marked as hide …
2021-07-22 07:17:35 -07:00
Piotr Surowiec
a5d9dd4fb9 fix: hide unavailable sequence title in course breadcrumbs (#28028)
When a sequence title was unavailable for a user (e.g. before starting a timed
exam), the `None` value was displayed in course breadcrumbs. This hides it in
such cases and ensures that a trailing breadcrumb separator is not displayed.
2021-07-22 09:52:25 -04:00
Farhaan Bukhsh
fa5736285e fix: Passwored reset page throwing page not found error (#27982)
Signed-off-by: Farhaan Bukhsh <farhaan@opencraft.com>
2021-07-22 09:51:02 -04:00
Dillon Dumesnil
7dd4f2db99 fix: AA-912: Hide URL after due date if subsection is marked as hide after due
A bug was reported of a learner accessing content that should have been hidden
due to the hide after due setting in Studio by the course team by clicking on
the Progress tab. This takes into account that value and will now hide the URL
on the Progress tab in that scenario.
2021-07-21 17:53:23 -04:00
Matt Tuchfarber
d16c9a804c refactor: Move cert code to cert app
The `may_certify` function is used to determine whether a learner should
be able to see that their certificate is ready. It is therefore entirely
in the certificates domain. This remove may_certifiy and
may_certify_for_course from the xmodule and course_overview
apps and into the certificates app. The xmodule may_certify was not called
outside of tests prior to this, and the other's were easily moved.

In order to avoid circular imports, this also moved
certificate_info_for_user out of the model code and into an API
2021-07-21 15:56:19 -04:00
connorhaugh
0d26738fce fix: Prevent Transcripts from Failing Export, Edit (#28242)
Instead of having json errors in transcript acquisition and conversion cause errors, have transcription conversion and acquisition simply return an error message in the transcription which can prompt a change from the user.

Although not uploading a transcript is handled, transcripts can often cause errors in edit, export, and other activities due to json errors. These errors block the entire use of these features, so to allow for reupload, etc, we add an error message instead of transcript and log the event.

In response to [TNL-8539](https://openedx.atlassian.net/secure/RapidBoard.jspa?rapidView=580&projectKey=TNL&modal=detail&selectedIssue=TNL-8539)

Testing: Unit tests coverage is included in the PR. Upload, import, and export of courses with transcriptions is also easily hand-testable. Just create a video in studio, add an irrelevant transcript. Then try to import, export, and edit the problem. Expected behavior is success.
2021-07-21 15:52:54 -04:00
Bianca Severino
14c85d9465 feat: pass verified name flag into user api 2021-07-20 16:58:39 -04:00
Zach Hancock
6ba12c71d3 test: test case for duplicate user in multiple providers 2021-07-20 12:03:26 -04:00
Zach Hancock
aa2bf9c063 feat: program_enrollments support for multiple SAML providers 2021-07-20 12:03:26 -04:00
Christie Rice
8b362f11b5 fix: Unify required parameter checks
MICROBA-1373
2021-07-19 14:48:08 -04:00
Justin Hynes
fd578b6636 Merge pull request #28196 from edx/crice/gen
fix: Retrieve grade and enrollment mode only once
2021-07-19 14:47:03 -04:00
Bianca Severino
430f1554b4 Revert "feat: pass verified name flag into account settings"
This reverts commit 85dc8c8d67.
2021-07-19 12:06:33 -04:00
Maari
3d98bff695 Fix retrieving BadgeAssertion data from Badgr (#28039)
* Fix retrieving BadgeAssertion data from Badgr

With edx#27181, the integration with Badgr was updated and fixed to
be working with the Badgr v2 API. However, retrieving the Badge Assertion
data from Badgr still needs to be updated for the new json response
structure so we can save it at our end as well.

* Add BadgeAssertion table to LMS admin
2021-07-19 11:55:00 -04:00
Bianca Severino
85dc8c8d67 feat: pass verified name flag into account settings 2021-07-19 11:03:24 -04:00
Justin Hynes
38b0261761 Merge pull request #28208 from edx/jhynes/microba-1351
feat!: Remove `AUDIT_CERT_CUTOFF_DATE` configuration setting
2021-07-19 09:32:32 -04:00
Zia Fazal
6f6a9e9c0b Merge pull request #27762 from edly-io/ERTE-5
[ERTE-5] feat: Added enterprise uuid in event context for enterprise enrolment events
2021-07-19 10:43:43 +05:00
Justin Hynes
0dd319a089 feat!: Remove AUDIT_CERT_CUTOFF_DATE configuration setting
[MICROBA-1351][DEPR-159]
* Remove `AUDIT_CERT_CUTOFF_DATE`
2021-07-16 15:38:14 -04:00
Thomas Tracy
3ecd233c11 [feat] MB-1192: Add user_has_passing_grade to outline API (#28198)
* [feat] add user_has_passing_grade to outline API

* default to false

* add tests

* linting
2021-07-16 15:03:35 -04:00
Christie Rice
c7d06c6bfa fix: Retrieve grade and enrollment mode only once and pass their values to the generation task, so those values can be saved in the cert. Also protect against missing course grades.
MICROBA-1373
2021-07-16 13:43:07 -04:00
João Cabrita
7261bc42bd feat: Add legacy_link field to TextBook app
This implements the `legacy_link` method to the TextbooksCourseApp
and makes the `legacy_link` URL absolute in the API response.

Related tickets:
* [TNL-8438](https://openedx.atlassian.net/browse/TNL-8438)
* [BB-4416 (OpenCraft Internal)](https://tasks.opencraft.com/browse/BB-4416)
2021-07-16 19:34:13 +05:00
sofiayoon
85c529e099 Merge pull request #28200 from edx/syoon/fix-webpack-loader-setting
fix: webpack loader config to display error
2021-07-16 09:19:18 -04:00
mohtamba
fa872a1c8d Merge branch 'master' into mtamba/bulk_allowance_modal_waffle_flag 2021-07-15 19:09:39 -04:00
Sofia Yoon
60493757e6 fix: webpack loader config to display error 2021-07-15 17:06:01 -04:00
mohtamba
1c53194250 Add Waffle Flag for Bulk Allowances
Add bulk allowance flag to allow for new feature to be enabled.
2021-07-15 13:41:45 -04:00
Christie Rice
a1ff409963 fix: Pass certificates values to generation, if available (#28179)
MICROBA-1373
2021-07-15 10:42:11 -04:00
Bianca Severino
04cec15840 Merge pull request #28187 from edx/bseverino/verified-name-course-flag
[MST-806] Add CourseWaffleFlag for verified name
2021-07-15 10:35:13 -04:00
Albert (AJ) St. Aubin
26678daf1f feat: Add the Request Certificate button to a Course card
[MICROBA-678]

When a certificate is in an unexpected state (i.e. notpassing with a
passing grade) this alert will allow the user to attempt to resolve the
issue on their own. It will run the code that checks the certificates
status. It requires that the course is configured to allow users to
Request Certificates though.
2021-07-15 10:27:18 -04:00
Christie Rice
6d3a4014bb fix: Save parameter values in the certificate, if available (#28178)
MICROBA-1373
2021-07-15 09:51:24 -04:00
Awais Qureshi
88b35ba9a9 Merge pull request #28183 from edx/removing-python_2_unicode_compatible
Django3 has removed python_2_unicode_compatible.
2021-07-15 17:55:45 +05:00
Awais Qureshi
6568689287 Django3 has removed python_2_unicode_compatible.
Execute the codemodes on this file and removed all usage.
2021-07-15 13:26:17 +05:00
Awais Qureshi
fe8a0c6f00 Django3 has removed python_2_unicode_compatible.
Execute the codemodes on this file and removed all usage.
2021-07-15 12:09:50 +05:00
Bianca Severino
7fea6d4692 feat: add CourseWaffleFlag for verified name 2021-07-14 16:59:46 -04:00
Awais Qureshi
cfe81cd834 Django3 has removed python_2_unicode_compatible.
Execute the codemodes on this file and removed all usage.
2021-07-15 00:37:15 +05:00
Awais Qureshi
6dc6777354 Django3 has removed python_2_unicode_compatible.
Execute the codemodes on this file and removed all usage.
2021-07-15 00:28:09 +05:00
Awais Qureshi
0433af537f Django3 has removed python_2_unicode_compatible.
Execute the codemodes on this file and removed all usage.
2021-07-15 00:26:09 +05:00
Christie Rice
f98b8e5be0 refactor: Replace references to V2 course certificates (#28180)
MICROBA-1227
2021-07-14 13:37:07 -04:00
Carla Duarte
d754951cf8 feat: add problem scores to progress tab api (AA-875) 2021-07-14 12:33:23 -04:00
Kshitij Sobti
423ac00809 Makes counting flagged comments optional, instead of automatic in case of moderators 2021-07-14 13:19:12 +05:00
Kshitij Sobti
689d2084f5 Review feedback 2021-07-14 13:19:12 +05:00
Aayush Agrawal
ffe3ee3869 feat: Adds additional data and filters to discussions API
This change adds three new filters to the threads API. They are:
* Filtering only threads that are flagged for abuse
* Filtering by the thread type (discussion or question)
* Filtering by the thread author

In addition it also adds a new ``abuse_flagged_count`` field for threads. It
returns a count of the number of comments in a thread that are flagged for abuse.
This is only visible to users that have moderator privileges or higher.

Finally it also adds a ``abuse_flagged_any_user`` field that is set if any user
has flagged a thread. This field too, is only visible to moderators or above.

Co-authored-by: Kshitij Sobti <kshitij@opencraft.com>
2021-07-14 13:19:12 +05:00
stvn
72df0a3ca6 merge(#28049): arjun/wiki-modal-buttons
commits
=======
- fix: change buttons in wiki modal to anchor tags
2021-07-13 12:46:47 -04:00
Justin Hynes
d786f6d394 Merge pull request #28168 from edx/jhynes/microba-1289_overviews-continued
refactor: Remove use of CourseOverview model directly in the Certificates app
2021-07-13 12:40:59 -04:00