- Mar 16, 2021
-
-
edX cache uploader bot authored
-
Muhammad Soban Javed authored
* Added a pylint plugin to set django settings module according to the module on which pylint is ran * Fix quality Co-authored-by:
Awais Qureshi <awais.qureshi@arbisoft.com>
-
Awais Qureshi authored
BOM-2453
-
- Mar 15, 2021
-
-
Carla Duarte authored
AA-212: add masquerade test for progress tab
-
Kyle McCormick authored
The render_xblock view, which powers the Learning MFE (among other things) returned a 404 when un- enrolled course staff users tried to load it while masquerading as learners. This was because we checked course access after enabling the masquerading context, which triggered a redirect- to-enrollment exception. The fix is simply to enable the masquerading context after checking course access. Content-level behavior and access is still calculated within the masquerading context, as intended. TNL-7989
-
Carla Duarte authored
-
Awais Qureshi authored
tests are failings and complaining related objects doest not exists in User table. Create object in test setup to fix it. In another fix article id was giving integrity error. Fixing task test.
-
Michael Terry authored
fix: don't duplicate text in forum notification email
-
Michael Terry authored
-
Awais Qureshi authored
tests are failings and complaining related objects doest not exists in User table. Create object in test setup to fix it. In another fix article id was giving integrity error. Fixing task test.
-
Ben Holt authored
Exposed the Date header so clients can accurately compute times relative to the dates returned by the API; browser time is notoriously unreliable for this, especially for a Learner-facing countdown call-to-action based on the access expiration date. (REV-2126) Using the Date header for this allows the client to make use of information that is already sent, does not require additional calls nor modifying the API, and could be generalized to more or all our APIs without modifying them.
-
Justin Hynes authored
MICROBA-1011 | Reduce use of certs models directly in instructor dashboard app
-
Michael Terry authored
fix: bump edx-ace to 1.0.1 to fix braze email styling
-
Justin Hynes authored
MICROBA-1011 | Reduce use of Certificates models accessed directly in Instructor Dashboard djangoapp [MICROBA-1011] - When creating bulk allowlist entries make the tool use the `is_on_allowlist` certificate API function over accessing the CertificateWhitelist model methods directly - When creating bulk allowlist entries make the tool use the `create_or_update_allowlist_entry` certificate API function over using CertificateWhitelist model/Django ORM directly
-
edX cache uploader bot authored
-
Matthew Piatetsky authored
[AA-461] Add management command to export course metadata for all courses
-
Michael Terry authored
-
Alex Dusenbery authored
Allowing it to do so left us prone to error when an explicit enterprise_customer_uuid arg is provided. We should modify this command in the future to perform deletions of open assignments as its only action, and it should only be invoked this way after we have verified that all backfilled enterprise_customer fields on the assignments have been set correctly. enterprise version 3.18.4
-
David Ormsbee authored
Improved error handling in Course Outline generation.
-
M. Zulqarnain authored
-
M. Zulqarnain authored
-
Justin Hynes authored
MICROBA-1011 | Remove duplicate allowlist entries and add constraint preventing future occurrences
-
Usama Sadiq authored
-
Kyle McCormick authored
A "section" tag in an OLX upload used to map to the SemanticSectionDescriptor, which translated it into a Sequence ("sequential" tag). This is both obscure and confusing, since it uses language that predates Studio. Back in the LMS prototype days, "section" was inconsistently used to be interchangeable with "sequence" and "sequential", and what Studio today calls a "section" was called a "chapter". Bits of this legacy terminology are still around in the courseware rendering code. The upshot is that if you make an OLX tag "section" before this commit, it would not map to what we call a "Section" in all our documentation, but to a "Subsection"; furthermore, if that <section> only had one child element, the node would be replaced with its singular child, removing the <section> node from the course tree entirely. The fact that you can make a "section" OLX tag at all is nowhere in our documentation because courses haven't bee...
-
Arslan Ashraf authored
-
Azan Bin Zahid authored
disable xsslint
-
Azan Bin Zahid authored
-
edX requirements bot authored
-
David Ormsbee authored
* Introduces the idea of content errors into the learning_sequences public API, accessible using get_content_errors(). * Makes course outline generation much more resilient to unusual structures (e.g. Section -> Unit with no Sequence in between), with the understanding that anything that doesn't conform to the standard structure will simply be skipped. * Improves the Django Admin for learning_sequences to display content errors and improve sequence data browsing within a course. * Switches the main table viewed in the Django admin from LearningContext to CourseContext, which is appropriate since only course runs generate outlines. This was done as part of TNL-8057, with the end goal of making course outline generation resilient enough to switch over apps to using the learning_sequences outline API. The types of course structure errors that this PR addresses cause display issues even in the current Outline Page experience, but would break the outline generation for learning_sequences altogether. The approach for error messages here is very generic, to keep modulestore concepts from seeping into learning_sequences (which is not aware of the modulestore/contentstore). We may need to address this later, with a more normalized content error data model. While the Django admin page is backwards compatible with the old versions of the models, we should run the backfill_course_outlines management command after deploying this change, to get the full benefits.
-
David Ormsbee authored
Adding new functionality is going to require running this command on all courses, not just ones that have it missing.
-
- Mar 12, 2021
-
-
Manjinder Singh authored
* fix: six was being used without being imported Removed usage of the python library six from module. This is fixing an error where the python library six is used in a module without being imported. Multiple PRs merging around same time caused this conflict.
-
Kyle McCormick authored
replace six with str
-
Carla Duarte authored
-
Justin Hynes authored
MICROBA-1011 | Remove duplicate allowlist entries and add constraint to model to prevent future occurrences [MICROBA-1011] - Add data migration to remove the duplicate certificate allowlist entries before adding the new constraint to the CertificateWhitelist model. - Add `unique_together` constraint to the CertificateWhitelist model. A learner should only have a single allowlist entry per course-run.
-
Waheed Ahmed authored
Added custom for SSO auth entries to monitor which auth entries are still in use.
-
Carla Duarte authored
AA-213: create progress tab mfe waffle flag
-
Matthew Piatetsky authored
-
Kyle McCormick authored
The Learning MFE had a bug where instructors selecting 'View As: Audit' would find themselves redirected to the 'Enroll Now' page. This was because the Courseware API, which powers the MFE, checked courseware access *after* setting up masquerading--so, instead of checking whether the instructor had access to view the course, we checked whether some arbitrary un-enrolled learner would have access to the course (the answer is generally No). We fix this by doing what the Legacy courseware index does: calculate courseware access *before* setting up masquerading. TNL-7989
-
Michael Terry authored
feat: update to edx-ace 1.0.0
-
Michael Terry authored
The new version switches how you specify recipients, to use lms_user_id instead of usernames. AA-489
-