Skip to content
Snippets Groups Projects
  1. Sep 09, 2021
  2. Sep 02, 2021
  3. Sep 01, 2021
  4. Aug 31, 2021
  5. Aug 30, 2021
    • Maria Grimaldi's avatar
      refactor: replace User for UserFactory and its methods · 54627e11
      Maria Grimaldi authored
      This change is done so the profile is automatically created for tests users
      54627e11
    • David Ormsbee's avatar
      fix: prefer HTML5 video when youtube support is deprecated. · 8b17afc8
      David Ormsbee authored
      In the case where a server/course has both:
      
      * YouTube deprecated (waffle flag: videos.deprecate_youtube = True)
      * HLS enabled (ConfigModel: HLSPlaybackEnabledFlag, in video_config)
      
      Prior to this commit, we would prefer the HLS source if one of the
      HTML5 video sources specified a .m3u8 file. However it looks like this
      file isn't always guaranteed to be produced.
      
      This fix always prefers HTML5 video sources to YouTube if the server or
      course is configured to deprecate YouTube, as long as at least one
      non-YouTube source is found. TNL-8594.
      
      Note that this may confuse users because we're essentially bypassing
      their Studio-specified preferred primary source with a server-wide
      setting.
    • Usama Sadiq's avatar
      build: Moved user and group management commands and unit tests to edx-django-utils · cde05061
      Usama Sadiq authored
      - Removed manage_user and manage_group commands and their unit tests from edx-platform and added then to edx-django-utils.
      - Modified User.post_save signal to ensure the user profile is created when manage_user management command is run to create a user.
      - Added edx-django-utils to INSTALLED_APPS for LMS and Studio.
      - Moved generate_password from openedx.core.djangoapps.user_authn.utils to edx_django_utils.user along with its unit test.
  6. Aug 27, 2021
  7. Aug 26, 2021
  8. Aug 25, 2021
    • oliviaruizknott's avatar
      refactor: send cert changed signal on_commit · eb2d6062
      oliviaruizknott authored
      Because we have ATOMIC_REQUESTS turned on, the COURSE_CERT_CHANGED
      signal was being sent *before* the date override was committed to the
      database. This means that the date override data sent to credentials on
      CertificateDateOverride save() was always one commit behind.
      
      Django provides an on_commit() function to allow us to perform actions
      only after a transaction is completed successfully. I believe this
      is when we want this signal to be sent.
      
      This is an alternative to 1) disabling atomic transactions for a view
      (honestly, I’m not sure which view we would target anyway), or 2)
      passing more data down through the signal.
      
      Toward MICROBA-1423
      eb2d6062
  9. Aug 24, 2021
  10. Aug 23, 2021
    • Michael Terry's avatar
      feat: allow unsubcribing from a course goal with just a token · 2176dd78
      Michael Terry authored
      * Add unsubscribe_token uuid field to CourseGoal model
      * Add endpoint to unsubcribe from just a token (no login needed)
      * Add admin page for the course_goals djangoapp
      * Add get_course_overview_or_404 utility method
      * Clean up URL handling in course_home_api
      
      AA-907
      2176dd78
  11. Aug 18, 2021
  12. Aug 17, 2021
    • Bianca Severino's avatar
      fix: update proctored exam settings URL · 1255bd85
      Bianca Severino authored
      The proctored exam settings link now directs to a new modal view, rather
      than the old exam settings page.
      1255bd85
    • Andy Shultz's avatar
      feat: update name affirmation to 0.6.1 · fa89221d
      Andy Shultz authored
      Certificate tests required an update to switch from boolean verified
      flag to status. This feature is not deployed yet so only test uses
      needed attention.
      
      MST-969 and friends
      fa89221d
    • oliviaruizknott's avatar
      feat: Send date override to credentials · e9902965
      oliviaruizknott authored
      When sending a GeneratedCertificate to Credentials, send the associated
      CertificateDateOverride (if there is one), or else None. This
      will be triggered after any save of a GeneratedCertificate, and after
      any save or deletion of a single CertificateDateOverride.
      
      Credentials will eventually store its own copy of this date override, or
      edit or remove exiting date overrides.
      e9902965
    • mohtamba's avatar
      Remove waffle flag for bulk allowance · 25dfb9d4
      mohtamba authored
      Remove the waffle flag for bulk allowance, so by default the bulk allowance feature is enabled and is the only option.
      25dfb9d4
  13. Aug 16, 2021
  14. Aug 13, 2021
  15. Aug 12, 2021
  16. Aug 11, 2021
  17. Aug 10, 2021
  18. Aug 09, 2021
  19. Aug 06, 2021
  20. Aug 05, 2021
    • Braden MacDonald's avatar
      refactor: run modulestore tests in common/lib/... using Django · da09bcad
      Braden MacDonald authored
      Does 3 things:
      (1) Use django for modulestore tests
      (2) Use normal LMS settings for modulestore tests instead of openedx/tests/settings.py
      (3) Simplify some TestCase subclasses by converting them to use ModuleStoreTestCase
      
      
      Details and rationale:
      
      (1) Currently parts of the modulestore test suite are designed to run "without django", although there is still a lot of django functionality imported at times, and many of the tests do in fact use django. But for the upcoming PR #27565 (moving split's course indexes from MongoDB to MySQL), we will need to always have Django enabled. So this commit paves the way for that change.
      
      (2) The previous tests that did use Django used a special settings file, openedx/tests/settings.py which made some debugging confusing because those tests had quite different django settings than other tests. This change deletes that file and runs the tests using the LMS test settings.
      
      (3) The test suite also contains many different ways of initializing and testing a modulestore, with significant differences in their configuration, and also a lot of repetition. I find this makes understanding, debugging and writing tests more difficult. So this commit also reduces the number of different "test case using modulestore" base classes:
      
      * Simplifies MixedWithOptionsTestCase and MixedSplitTestCase by making them simple subclasses of ModuleStoreTestCase.
      * Removes PureModulestoreTestCase.
      da09bcad
    • cdeery's avatar
      fix: collapse unit tests by using ddt · b4b461b6
      cdeery authored
      b4b461b6
    • cdeery's avatar
      fix: changes suggested in code review · eafea504
      cdeery authored
      Cleaned up the nits.
      eafea504
Loading