Add new fields for wordpress filters EDLY-6765 #4350
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adds invite_only field in course run
* Updating Python Requirements * Updating RCM OAuth Client. This is being undertaken as part of a fix for receiving a connection reset error in the ecommerce data loader. The old client would not reauthenticate a request if the old token expired. The new client will which should fix this issue. * WEBSITE-360 add program hooks to admin (openedx#2321) Add program hooks to django admin * Use Mode to determine ecom attributes For seats, we used to set is_id_verified and certificate_type ecom attributes by string comparision on seat type. But now we have Mode, which holds those flags. So now we send those along now based on Mode. The only behavior difference is that we won't send any information for seats that aren't supposed to be in the that run's CourseRunType. * Wait for the official course to be set on the official course run When setting official state for a course run, there is an intermediate time after creating the official run, but before the official course is set that would cause a duplicate external key to be flagged. This now waits until the official course is set before checking. * Add history to Organization (openedx#2323) * Add history to Organization * Fix * Upgrade empty course & run types in RCM After loading seats from ecommerce, try filling in any empty course or course run types. This should only happen if we import runs from Studio, in practice. If a run has seats but the course or course run does not have a type and we fail to find a matching type to use, we will fail the ecom loader (but will let it finish first). DISCO-1427 * fix(i18n): update translations * Stop unpublishing all runs in a course We accidentally were unpublishing all inactive runs in a course, even if that left us with no published runs. But we are supposed to always leave at least one published run so that a course is still marketable. This fixes that logic and adds new code to the unpublish_inactive_runs management command to re-publish old runs that we accidentally unpublished. * WEBSITE-361 set up mgmt command for bulk adding program hooks (openedx#2328) WEB-361 mgmt command to backfill existing program hooks * Added another argument to create_or_update_partner command (openedx#2334) * Drop temporary republish code * Unpublish a few extra course runs In our zeal to republish some courses, we cast the net too wide. Here's a commit that will cause the unpublish_inactive_runs job to unpublish the extra. It's a hardcoded list, which is gross. But it will be deleted soon. * Updating Python Requirements * DISCO-1399: Deprecating pre-type workflow This also includes some deprecation work that used to rely on masters seats. * DISCO-1414 org source of truth 1 (openedx#2330) -add certificate_name to org -populate certificate_name in org data loader with name from lms * DISCO-1530: pop off entitlements until FAP stops sending * DISCO-1499: Update OFAC choices text * DISCO-1509: Adds beautifulsoup to clean_html to preserve HTML entities * Drop temporary unpublish fix code * Pop entitlements again, now that Publisher isn't sending it DISCO-1399 * Adds in lxml to base.in * DISCO-1502: Remove migration commands * DISCO-1507 whitelist blacklist cleanup (openedx#2342) -remove usages which read from org config vars * Fix a few Django admin bugs (openedx#2349) - Fixed the canonical course run of a course from showing a value even if it didn't have one. Just made it a raw id, since this field isn't often changed and doesn't need to be pretty. - Fixed the 'courses' field of a Program from including drafts and from not showing the correct ordering. DISCO-1535 * Return 403 on non-publisher user writes If a non-publisher user (a user without a group associated) tries to use publisher API, we now return a more forceful status code (rather than simply returning an empty set of courses or runs). DISCO-1532 * Updating logs in RCM In an effort to reduce the total amount of logging in RCM, we are updating the logs to remove any logs in the analytics api and only log out information when an error happens in the Ecommerce data loader. * Updating Python Requirements * DISCO-1414 org source of truth (openedx#2343) - add image fields to organization model for logo, certificate_logo, and banner_image - push organization to studio on save - remove OrganizationsApiDataLoader, SchoolMarketingSiteDataLoader, SponsorMarketingSiteDataLoader - add management coommand for downloading images * Always filter with partner. Add in additional log * Updating Python Requirements * DISCO-1414 org source of truth 3 (openedx#2355) - remove download org image command - remove usages in org model to image url fields * add brackets * add fake translation step on extract translations * fix extract translations * fix(i18n): update translation strings * Adding Missing progam type * DISCO-1414 org source of truth 4 (openedx#2358) -add migration for removal of url fields on organization -also add migration from missed migration of previous change which makes blank=True for an org's certificate_name * DISCO-1542 remove check for up-to-date translations (openedx#2357) update docs * Revert "Adding Missing progam type" This reverts commit 96e17d2. * fix(i18n): update translations * Update the publish logic for marketing to publish MicroBachelors Programs * Fix aggregate search query overload (openedx#2366) * Narrow the range of an assertNumQueries to only cover the search under test, rather than computing the expected results * Measure a few more query counts * Use read_queryset to prefetch related seat_types, rather than fetching them one-by-one in the serializer * Remove unused function * Revert "Update the publish logic for marketing to publish MicroBachelors Programs" This reverts commit 8ed422b. * Updating Python Requirements * Drop publisher views This is part of removing the old publisher djangoapp in favor of the new frontend-app-publisher frontend. This drops views, non-admin forms, and email-sending code. DISCO-1465 * Drop publisher commands and dataloaders This is part of removing the old publisher djangoapp in favor of the new frontend-app-publisher frontend. DISCO-1469 DISCO-1470 * Drop unused Publisher APIs & code This is part of removing the old publisher djangoapp in favor of the new frontend-app-publisher frontend. DISCO-1471 * EDUCATOR-4890 make sure empty strings are not considered as duplicated external_key values * WS-543 add redirects from new course run slugs (openedx#2369) WS-543 add redirects from new course run slugs * WS-453 add management command config to disco admin (openedx#2374) * DISCO-1562 remove subject data loader (openedx#2373) - remove data loader, usage in rcm, and all related tests. * fix(i18n): update translation strings * fix(i18n): update translations * update redirect file with latest from Drupal (openedx#2379) * allow spans with lang attribute to be saved to backend for course metadata (openedx#2375) * DISCO-1572 course type org whitelist (openedx#2384) - make org whitelist for course types in django admin panel use a checklist instead of traditional many-to-many list * MICROBA-84 | Add 'credit_value' to Program model (openedx#2385) - Add 'credit_value' field to Program model. Represents number of credits a learner will earn upon successful completion of the program - Add 'credit_value' to ProgramAdmin - Add 'credit_value' to ProgramSerializer * fix(i18n): update translation strings * WS-542 remove redirect data loader (openedx#2383) * Make draft versions of Studio-made runs If a course already exists in Publisher as a draft world, when we notice new runs made in Studio, let's set up a draft version of the run as well as the official. * Drop some unused publisher tables DISCO-1472 * Fix recent drop table migration The unique togethers for a certain publisher table are different in stage and prod for some reason? So this commit just stops trying to alter the unique-together before deleting the model. * Revert "Fix recent drop table migration" This reverts commit c92d100. * Upgrade edx-django-release-util * Adding the Program Type Slug for the Course Serializer [MICROBA-151] This type slug can be used when comparing the program type in api clients. * Upgrade django-release-util version (openedx#2396) * Move devstack discovery.yml to configuration repo (openedx#2394) * Update package lock * fix(i18n): update translation strings * fix(i18n): update translations * Fix runs resetting status accidentally When submitting a run via Publisher, we would mark any reviewed courses as unreviewed, if the course had changed in a meaningful way. But we had a bug in that 'meaningful' check that meant we'd always reset the runs if the course had an entitlement. DISCO-1575 * replaced jsonfield with jsonfield2 (openedx#2398) * removed the constraint for django-autocomplete-light (openedx#2399) * Revert "removed the constraint for django-autocomplete-light (openedx#2399)" This reverts commit dfd035c. * Revert "replaced jsonfield with jsonfield2 (openedx#2398)" This reverts commit 6ca016d. * Added estimated_hours field (openedx#2411) * replaced the jsonfield with jsonfield2 * WS-549 make sure url_slug in discovery matches active url_slug, make read-only (openedx#2412) * upgrade django-autocomplete-light (openedx#2410) * Updated test data for course runs (openedx#2414) * Avoid salesforce signals preventing UpdateMastersSandbox job (openedx#2408) Jenkins builds were being blocked b/c fixtures are sometimes loaded in an odd order when we are loading them in various non-production environments, but we needn't block these use cases on this (fairly production-specific) salesforce integration not having what it would need to operate properly. JIRA:EDUCATOR-4865 * removed constraint of django-debug-toolbar (openedx#2395) * Fix the push_translations job. There was a critical failure because the original formatting did not match with the translation * Updating Python Requirements * Manually compiling messages to fix pull_translations job Hopefully * Adding string change to update in transifex * fix(i18n): update translation strings * fix(i18n): update translations * Add is_marketable to ES index Add is_marketable (based on CourseRunType's field of same name) to the CourseRunIndex. This way we can also query based on that and exclude non-marketable runs from searches. But that will be a separate PR. DISCO-1589 * fix(i18n): update translations * BOM-1234 -Added django-dynamic-filenames dependency to replace UploadToAutoSlug with FilePattern -Modified code to use FilePattern instead of deprecated method UploadToAutoSlug of stdimage -Removed django-stdimage from pins -Updated Requirements -Updated Migrations -Created new Migrations of ProgramType Model -Updated Custom Class according to new lib * BOM-1232 Updated social auth app django for Django2.0 support (openedx#2427) * Adding some additional logging to RCM for debugging DISCO-1541 * Updating Python Requirements * Filter unmarketable runs from searches Don't have course runs that are listed as non-marketable (like Masters or Empty runs) show up in search indices that fuel the marketing site. DISCO-1589 * Made a mistake about response variable type * Catching the correct error * small PR to add the type slug to the Program response * Trying to catch the correct error * Removing unused auth classes (openedx#2434) * Removing unused auth classes BearerAuthentication and BasicAuthentication classes are not used by our system. Additionally, BearerAuthentication is deprecated and will soon be removed from edx-drf-extensions. * WS-681 use course run key in course run url slug (openedx#2437) * Log actual response attributes * fix(i18n): update translations * WS-685 add logging around course update (openedx#2440) * Missing migration for PR#2437 (openedx#2443) * Added the ProgramTypeAttrs to the serializer and model [MICROBA-150] Updated the ProgramType to have a UUID and a Coaching support boolean. Then serialized this data as a type_attrs for type attributes in the API reponse for Programs. * DISCO-1599 fix img size reset (openedx#2447) - when an img tag with size attributes was saved in an htmlfield, then that record was saved a second time, the size attributes were lost. We now explicitly set a config on the HTML2Text class inorder to properly allow processing of size attributes on img's. * Revert "Removing unused auth classes (openedx#2434)" This reverts commit ad0000b. * WS-680 ensure that authoring orgs persist a configurable order. Since order determines order of logos on about page. (openedx#2451) * writing notes about how to test... * wrote some tests... * putting space back * i love lint. i love lint. * Add management command to dedupe course metadata Course and CourseRun metadata history tables largely contain duplicate entries, where the only difference between subsequent records is the `modified` column. This breaks the clean_duplicate_history management command provided by django-simple-history, so this PR provides a slightly tweaked version of that management command to fix that issue for our purposes. DENG-20 * Change repo ownership to group * Make courserun.hidden readonly in admin and add help text * WS-655 index content for search (openedx#2450) * WS-655 wrap Algolia reindex management command (openedx#2457) * WS-655 don't index data with missing required fields * Python Requirements Update (openedx#2455) * Updating Python Requirements * Updated query counts Co-authored-by: usamasadiq <[email protected]> Co-authored-by: Jeremy Bowman <[email protected]> * Adding dockerfile (openedx#2426) * Adding dockerfile * Iterating... * Iterating... Co-authored-by: Fred Smith <[email protected]> * fix(i18n): update translations * BOM-1235 Added tox for multiple Django versions (openedx#2431) Added tox file to run pytest from tox on different Django versions. * fix(i18n): update translation strings * Fix null reference in missing PC log message * availability boosting logic for search (openedx#2463) * WS-743 do not index courses with no active course runs (openedx#2466) * Add recent enrollment count to AG index (openedx#2467) * Fix travis builds * Use system firefox in travis Also requires a bump in geckodriver * Add ADR proposal about changes to program-curricula relationship. * BOM-1045 -edx-drf-ext version bump -make upgrade -Settings update -removed djangorestframework-jwt -pinned edx-rest-api-client version * fix(i18n): update translations * Removing unused bearerAuthentication class (openedx#2458) * Removing unused auth classes BearerAuthentication are not used by our system. Additionally, BearerAuthentication is deprecated and will soon be removed from edx-drf-extensions. * WS-760 combine course and program indices (openedx#2475) * Update dedupe script to use the `everything` manager Several models in this repo inherit the DraftModelMixin which overrides the `objects` attribute to only return non-draft objects, but also provides an `everything` model manager which returns all objects. This change to the deduplicate_course_metadata_history management command makes use of the `everything` model manager so that draft instances of a model are not skipped while deduplicating history. * WS-760 part II, exclude index from coverage, add missing fields (openedx#2477) * Remove non-Active programs and use ProgramType name instead of slug (openedx#2481) * Fix field name subjects -> subject to make facet work (openedx#2482) * Add program type to facets (openedx#2484) * changed javascript_catalog to JavascriptCatalog (openedx#2476) * WS-746 demote professional certificate results temporarily (openedx#2487) * Adding tests to make sure permissions exists in db for catalogs. * DISCO-1335: Upgrades JS packages * Add renovate.json (openedx#2192) Co-authored-by: Renovate Bot <[email protected]> Co-authored-by: Jason Myatt <[email protected]> Co-authored-by: Jeff LaJoie <[email protected]> * Pin dependencies * DISCO-1607: Add stockrecord with sku into ecom request We use the sku in order to get the seat we want to update * add reserved keyword checker to CI flow (openedx#2479) * Update elasticsearch Docker tag to v1.7.6 (openedx#2493) Co-authored-by: Renovate Bot <[email protected]> Co-authored-by: Jeff LaJoie <[email protected]> * Update dependency imports-loader to v0.8.0 (openedx#2497) Co-authored-by: Renovate Bot <[email protected]> * run linter on all systems (openedx#2503) * Update memcached Docker tag to v1.6.2 (openedx#2494) Co-authored-by: Renovate Bot <[email protected]> Co-authored-by: Jeff LaJoie <[email protected]> * Update dependency webpack to v4.42.1 (openedx#2499) Co-authored-by: Renovate Bot <[email protected]> * Update dependency style-loader to v1 (openedx#2501) Co-authored-by: Renovate Bot <[email protected]> Co-authored-by: Jeff LaJoie <[email protected]> * Update dependency sass-loader to v8 (openedx#2500) * Update dependency sass-loader to v8 * Update dependency style-loader to v1 * Updates webpack config Co-authored-by: Renovate Bot <[email protected]> Co-authored-by: Jeff LaJoie <[email protected]> Co-authored-by: Jeff LaJoie <[email protected]> * Adding in status code to RCM error log * Create Django 2.x travis workers * Creating more travis workers and refactored how tests are run in travis/tox Co-authored-by: Zulqarnain <[email protected]> Co-authored-by: Diana Huang <[email protected]> * Fixes duplicate sass-loader in package.json * BOM-1411 Adding default permissions except view to run tests on django2. * Remove Harvard Medical School Global Academy from RCM * Update memcached Docker tag to v1.6.3 * Removing edX specific business logic We updated our specific usage of this to call this command passing in a partner_code. * Fixing RCM RCM has been failing because it has not had access to all of the courses because the service user was being treated as an AnonymousUser instead of a staff user. By passing in the username, it will be treated as staff and get all courses. We are also fixing our backoff code that started failing when we updated which client we use. * Allow the backoff code to run for gateway timeout * Catalog courses query improvement (openedx#2523) * Optimize the performance of the CourseQuerySet The CourseQuerySet.available() function is used frequently, and does a distinct to only return one copy of each Course. However, because of the query construction, it needed to do a SELECT DISTINCT over all fields of the Course (and associated Course Runs). The new version just fetches distinct Course ids, and then gets all course metadata for those courses. This improves query times by about a factor of 3 on the read-replica. * Prefetch a few more attributes of Courses that are used by the CatalogCourseSerializer * Fix serializers in prep for DRF upgrade Newer versions of DRF don't like our hacky "slug on write, full serialization on read" implementation. So rewrite it to be a little more clean. This makes no actual change to serializer behavior. * Update dependency css-loader to v3.5.1 (openedx#2521) Co-authored-by: Renovate Bot <[email protected]> * Avoid re-querying the database for data that has already been prefetc… (openedx#2382) * Avoid re-querying the database for data that has already been prefetched in the read_queryset * Add a test demonstrating the query explosion caused by exclude_expired_course_run * Update dependency url-loader to v4.1.0 (openedx#2526) Co-authored-by: Renovate Bot <[email protected]> * Default to ordering Person by id, because that has an index, unlike created (openedx#2527) * Added test URL for support tools * Accessing the Discovery Service Super Admin Creating/Accessing the Discovery Service Super Admin to create the catalogues and many more * updated the Django version to 2.2 for production and removed constraint on drf-haystack * Update dependency css-loader to v3.5.2 (openedx#2531) Co-authored-by: Renovate Bot <[email protected]> * Update dependency jquery to v3.5.0 (openedx#2532) Co-authored-by: Renovate Bot <[email protected]> * fix(i18n): update translations * Only update courseruns when they change (openedx#2537) * Increasing RCM backoff to 60/180/300 seconds * WS-875 don't index internal/hidden courses to algolia (openedx#2539) * temporarily lock courserun table during ghost migration (openedx#2540) * drop foreign keys on course_metadata.historicalcourserun (openedx#2530) * remove estute from codeowners (openedx#2545) * readd foreign key constraints to historicalcourserun (openedx#2544) * removed get_cache_key utility and using it from edx-django-utils (openedx#2543) * Django2.2 upgrade These migrations are coming due to django2.2 last_name character length increased to 150. * BOM-1519 Removing django<2.2 versions from tox/travis. Adding python3.8 testing support. * BOM-1535 Upgrading django-guardian. * refactored imports from edx-django-utils (openedx#2550) * Updating Python Requirements * Update dependency webpack to v4.43.0 * Update devstack settings JWT_AUTH to enable inter-service communication during local development. * Log instances where the size of data being put into memcache is too large (openedx#2554) * Update dependency node-sass to v4.14.0 * We need to delete old entitlements if the Course Type changed We saw a scenario where a draft Course started as Verified and Audit and had a draft verified Entitlement as a result. The draft Course was then switched to Audit Only, but we failed to remove the Verified entitlement causing it to be created in Ecommerce. This will prevent that scenario and ensure proper cleanup. Note: This does not effect the behavior of published Courses and Course runs as they are controlled by different logic to not allow changing of entitlements * Update dependency css-loader to v3.5.3 * DISCO-1617 admin autocomplete (openedx#2559) - replace autocomplete.ModelSelect2 with new autocomplete_fields property newly available in django2. * Install pywatchman as a local/development requirement. * fix autocomplete fields for curriculum tables (openedx#2562) JIRA:EDUCATOR-5036 * WS-925 make program type translatable: add new model/field (openedx#2563) * WS-926 copy programType.name to programTypeTranslatable.name_t (openedx#2564) * Added a Program Affiliate window endpoint [MICROBA-258] Added a Program Affiliate window endpoint to help the Marketing team with data for affiliates. * Update dependency style-loader to v1.2.1 * Updating Python Requirements * Test fix to go with upcoming changes to the ProgramType table * Update memcached Docker tag to v1.6.5 * Removing a change that is causing a migration * Fix Program admin reordering A combination of django 2.2 and django-autocomplete-light 3.5 broke our admin page's custom autocomplete on the Program edit page. As a quick fix, limit DAL to 3.4 and remove a use of jquery. DISCO-1624 * WS-940 add LevelTypeTranslation and name_t (openedx#2568) * WS-927 remove references to programType.name (openedx#2565) * WS-941 copy level type name to translated field (openedx#2572) * Log cache results for ProgramViewSet pages * WS-947 program and level type endpoints (openedx#2574) * Encode bytestring to concat with cached object in log output * fix(i18n): update translation strings * Log data used to generate ProgramListView keys * WS-942 replace level type name (openedx#2575) * Revert "WS-942 replace level type name (openedx#2575)" (openedx#2582) This reverts commit 4956cf9. * Enterprise API: Include additional detail-level fields in Enterprise Catalog "Courses" list endpoint (openedx#2571) * WS-873 create Spanish index (openedx#2580) * WS-871: Adding Availability Filter for Search (openedx#2578) * model logic and tests * fixed tests * some refactoring * updated tests * lint fun * adding ugettext * PR comments * delete utility functions in model * add translatable text * revert merged weirdness * trying that again * lint * WS-873 create Spanish index (openedx#2580) * code review fixes * putting back .date() Co-authored-by: Rebecca Graber <[email protected]> * WS-792 use correct field name (openedx#2586) * fix(i18n): update translation strings * Revert "Revert "WS-942 replace level type name (openedx#2575)" (openedx#2582)" (openedx#2591) This reverts commit 725fdcb. * add is_enrollable field to search/all endpoint for course and courseruns (openedx#2590) * add is_enrollable field to search/all endpoint for course and courseruns * update tests * Make language tags translatable (openedx#2592) * WS-965 use untranslated name for search (openedx#2587) * Allow pages to be opted out of page-level caching using a waffle flag * clean up translations and fix locale in AG index (openedx#2594) * fix(i18n): update translations * Log a message when a view skips the page cache due to a flag * Fix availability translation by using ugettext instead of ugettext_lazy (openedx#2598) * fix(i18n): update translation strings * BOM-1625 Fixing deprecated allow-tags attribute in admin.py * fix(deps): update dependency node-sass to v4.14.1 * fix(deps): update dependency jquery to v3.5.1 * WS-977: Re-add masters programs to algolia index * WS-976: Updating Availability Facet logic so one product can have multiple statuses (openedx#2607) * updated availability logic and tests * returning list not set * WS-977: Re-add masters programs to algolia index * delete redundant test caused by merge * fix flakey test Co-authored-by: Rick Reilly <credential.helper> * WS-978 stop suppressing prof cert programs (openedx#2606) * WS-980 expand Chinese language options (openedx#2608) * WS-1012 boost Spanish language results in spanish_product index (openedx#2611) * WS-1012 boost Spanish language results in spanish_product index * WS-1022 pin promoted courses/programs in presearch state (openedx#2614) * fix(i18n): update translations * Upgrade Django to 2.2.13 * DISCO-1643: Add in Honor Course Types to be created by default This will create the Course Types of Honor Only, Verified and Honor, and Credit with Honor (Credit, Verified, Honor). Although Honor is not used internally to edX, it is widely used in the Open edX community and this work is to continue to support their use cases. (cherry picked from commit cec6343) * WS-1025 make default search results configurable in admin (openedx#2621) * WS-1025 make pinned search results configurable in admin (cherry picked from commit bbd46dc) * Upgrade Django to 2.2.15 * Upgrade Django to 2.2.16 * Handle JSON decoder error is api returns 404 * Replace status code check with raise_for_status * Fix Mirgrations * Fix tests * Fix Programs Tests Co-authored-by: edX requirements bot <[email protected]> Co-authored-by: Dillon Dumesnil <[email protected]> Co-authored-by: Rebecca Graber <[email protected]> Co-authored-by: Michael Terry <[email protected]> Co-authored-by: Abdul Hannan <[email protected]> Co-authored-by: edX Transifex Bot <[email protected]> Co-authored-by: Manjinder Singh <[email protected]> Co-authored-by: Nick <[email protected]> Co-authored-by: Jeff LaJoie <[email protected]> Co-authored-by: Karen Gu <[email protected]> Co-authored-by: Albert (AJ) St. Aubin <[email protected]> Co-authored-by: karengu <[email protected]> Co-authored-by: Calen Pennington <[email protected]> Co-authored-by: Simon Chen <[email protected]> Co-authored-by: Justin Hynes <[email protected]> Co-authored-by: Michael Terry <[email protected]> Co-authored-by: Hassan Javeed <[email protected]> Co-authored-by: syedimranhassan <[email protected]> Co-authored-by: Aarif <[email protected]> Co-authored-by: Jason Myatt <[email protected]> Co-authored-by: irfanuddinahmad <[email protected]> Co-authored-by: Matt Hughes <[email protected]> Co-authored-by: Muhammad Soban Javed <[email protected]> Co-authored-by: Zulqarnain <[email protected]> Co-authored-by: Usama Sadiq <[email protected]> Co-authored-by: srwang <[email protected]> Co-authored-by: Troy Sankey <[email protected]> Co-authored-by: edX requirements bot <[email protected]> Co-authored-by: Jeremy Bowman <[email protected]> Co-authored-by: Cory Lee <[email protected]> Co-authored-by: Fred Smith <[email protected]> Co-authored-by: Alex Dusenbery <[email protected]> Co-authored-by: Awais Qureshi <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Renovate Bot <[email protected]> Co-authored-by: Jason Myatt <[email protected]> Co-authored-by: Jeff LaJoie <[email protected]> Co-authored-by: Stu Young <[email protected]> Co-authored-by: Diana Huang <[email protected]> Co-authored-by: chinmaybhatk <[email protected]> Co-authored-by: Hammad Ahmad Waqas <[email protected]> Co-authored-by: Zaman Afzal <[email protected]> Co-authored-by: Adam Stankiewicz <[email protected]> Co-authored-by: Rick Reilly <credential.helper> Co-authored-by: Ned Batchelder <[email protected]> Co-authored-by: pkulkark <[email protected]>
* Removing edX specific business logic We updated our specific usage of this to call this command passing in a partner_code. * Fixing RCM RCM has been failing because it has not had access to all of the courses because the service user was being treated as an AnonymousUser instead of a staff user. By passing in the username, it will be treated as staff and get all courses. We are also fixing our backoff code that started failing when we updated which client we use. * Allow the backoff code to run for gateway timeout * Catalog courses query improvement (openedx#2523) * Optimize the performance of the CourseQuerySet The CourseQuerySet.available() function is used frequently, and does a distinct to only return one copy of each Course. However, because of the query construction, it needed to do a SELECT DISTINCT over all fields of the Course (and associated Course Runs). The new version just fetches distinct Course ids, and then gets all course metadata for those courses. This improves query times by about a factor of 3 on the read-replica. * Prefetch a few more attributes of Courses that are used by the CatalogCourseSerializer * Fix serializers in prep for DRF upgrade Newer versions of DRF don't like our hacky "slug on write, full serialization on read" implementation. So rewrite it to be a little more clean. This makes no actual change to serializer behavior. * Update dependency css-loader to v3.5.1 (openedx#2521) Co-authored-by: Renovate Bot <[email protected]> * Avoid re-querying the database for data that has already been prefetc… (openedx#2382) * Avoid re-querying the database for data that has already been prefetched in the read_queryset * Add a test demonstrating the query explosion caused by exclude_expired_course_run * Update dependency url-loader to v4.1.0 (openedx#2526) Co-authored-by: Renovate Bot <[email protected]> * Default to ordering Person by id, because that has an index, unlike created (openedx#2527) * Added test URL for support tools * Accessing the Discovery Service Super Admin Creating/Accessing the Discovery Service Super Admin to create the catalogues and many more * updated the Django version to 2.2 for production and removed constraint on drf-haystack * Update dependency css-loader to v3.5.2 (openedx#2531) Co-authored-by: Renovate Bot <[email protected]> * Update dependency jquery to v3.5.0 (openedx#2532) Co-authored-by: Renovate Bot <[email protected]> * fix(i18n): update translations * Only update courseruns when they change (openedx#2537) * Increasing RCM backoff to 60/180/300 seconds * WS-875 don't index internal/hidden courses to algolia (openedx#2539) * temporarily lock courserun table during ghost migration (openedx#2540) * drop foreign keys on course_metadata.historicalcourserun (openedx#2530) * remove estute from codeowners (openedx#2545) * readd foreign key constraints to historicalcourserun (openedx#2544) * removed get_cache_key utility and using it from edx-django-utils (openedx#2543) * Django2.2 upgrade These migrations are coming due to django2.2 last_name character length increased to 150. * BOM-1519 Removing django<2.2 versions from tox/travis. Adding python3.8 testing support. * BOM-1535 Upgrading django-guardian. * refactored imports from edx-django-utils (openedx#2550) * Updating Python Requirements * Update dependency webpack to v4.43.0 * Update devstack settings JWT_AUTH to enable inter-service communication during local development. * Log instances where the size of data being put into memcache is too large (openedx#2554) * Update dependency node-sass to v4.14.0 * We need to delete old entitlements if the Course Type changed We saw a scenario where a draft Course started as Verified and Audit and had a draft verified Entitlement as a result. The draft Course was then switched to Audit Only, but we failed to remove the Verified entitlement causing it to be created in Ecommerce. This will prevent that scenario and ensure proper cleanup. Note: This does not effect the behavior of published Courses and Course runs as they are controlled by different logic to not allow changing of entitlements * Update dependency css-loader to v3.5.3 * DISCO-1617 admin autocomplete (openedx#2559) - replace autocomplete.ModelSelect2 with new autocomplete_fields property newly available in django2. * Install pywatchman as a local/development requirement. * fix autocomplete fields for curriculum tables (openedx#2562) JIRA:EDUCATOR-5036 * WS-925 make program type translatable: add new model/field (openedx#2563) * WS-926 copy programType.name to programTypeTranslatable.name_t (openedx#2564) * Added a Program Affiliate window endpoint [MICROBA-258] Added a Program Affiliate window endpoint to help the Marketing team with data for affiliates. * Update dependency style-loader to v1.2.1 * Updating Python Requirements * Test fix to go with upcoming changes to the ProgramType table * Update memcached Docker tag to v1.6.5 * Removing a change that is causing a migration * Fix Program admin reordering A combination of django 2.2 and django-autocomplete-light 3.5 broke our admin page's custom autocomplete on the Program edit page. As a quick fix, limit DAL to 3.4 and remove a use of jquery. DISCO-1624 * WS-940 add LevelTypeTranslation and name_t (openedx#2568) * WS-927 remove references to programType.name (openedx#2565) * WS-941 copy level type name to translated field (openedx#2572) * Log cache results for ProgramViewSet pages * WS-947 program and level type endpoints (openedx#2574) * Encode bytestring to concat with cached object in log output * fix(i18n): update translation strings * Log data used to generate ProgramListView keys * WS-942 replace level type name (openedx#2575) * Revert "WS-942 replace level type name (openedx#2575)" (openedx#2582) This reverts commit 4956cf9. * Enterprise API: Include additional detail-level fields in Enterprise Catalog "Courses" list endpoint (openedx#2571) * WS-873 create Spanish index (openedx#2580) * WS-871: Adding Availability Filter for Search (openedx#2578) * model logic and tests * fixed tests * some refactoring * updated tests * lint fun * adding ugettext * PR comments * delete utility functions in model * add translatable text * revert merged weirdness * trying that again * lint * WS-873 create Spanish index (openedx#2580) * code review fixes * putting back .date() Co-authored-by: Rebecca Graber <[email protected]> * WS-792 use correct field name (openedx#2586) * fix(i18n): update translation strings * Revert "Revert "WS-942 replace level type name (openedx#2575)" (openedx#2582)" (openedx#2591) This reverts commit 725fdcb. * add is_enrollable field to search/all endpoint for course and courseruns (openedx#2590) * add is_enrollable field to search/all endpoint for course and courseruns * update tests * Make language tags translatable (openedx#2592) * WS-965 use untranslated name for search (openedx#2587) * Allow pages to be opted out of page-level caching using a waffle flag * clean up translations and fix locale in AG index (openedx#2594) * fix(i18n): update translations * Log a message when a view skips the page cache due to a flag * Fix availability translation by using ugettext instead of ugettext_lazy (openedx#2598) * fix(i18n): update translation strings * BOM-1625 Fixing deprecated allow-tags attribute in admin.py * fix(deps): update dependency node-sass to v4.14.1 * fix(deps): update dependency jquery to v3.5.1 * WS-977: Re-add masters programs to algolia index * WS-976: Updating Availability Facet logic so one product can have multiple statuses (openedx#2607) * updated availability logic and tests * returning list not set * WS-977: Re-add masters programs to algolia index * delete redundant test caused by merge * fix flakey test Co-authored-by: Rick Reilly <credential.helper> * WS-978 stop suppressing prof cert programs (openedx#2606) * WS-980 expand Chinese language options (openedx#2608) * WS-1012 boost Spanish language results in spanish_product index (openedx#2611) * WS-1012 boost Spanish language results in spanish_product index * WS-1022 pin promoted courses/programs in presearch state (openedx#2614) * fix(i18n): update translations * fix(i18n): update translations * WS-1025 make default search results configurable in admin (openedx#2621) * WS-1025 make pinned search results configurable in admin * Updating Python Requirements * DISCO-1640: sort search results This adds a new ordering param to some search endpoints like /api/v1/search/all. You can now specify ordering=aggregation_key just to get a sensible reliable ordering. * fix(i18n): update translations * Updating Python Requirements * Run test with python 3.8 * Update memcached Docker tag to v1.6.6 * Update dependency datatables.net-bs to v1.10.21 * Adds in frontend-app-learner-portal-enterprise to cors whitelist * Update dependency datatables.net to v1.10.21 * WS-1088 fix advertised course run for null start date (openedx#2632) * Update dependency css-loader to v3.6.0 * removed deprecated future imports (openedx#2634) * DISCO-1643: Add in Honor Course Types to be created by default This will create the Course Types of Honor Only, Verified and Honor, and Credit with Honor (Credit, Verified, Honor). Although Honor is not used internally to edX, it is widely used in the Open edX community and this work is to continue to support their use cases. * Fix LMS read timeout on running migrations When running the install_es_indexes command, the discovery app connects with the LMS. The default read timeout of 1s is sometimes too low, and the command crashes. Here, we make use of the `timeout=` keyword argument of the OAuthAPIClient constructor to manually set the timeout. We set it in the django settings to the default value of the class constructor, such that existing platforms are not affected, but this setting is customizable across environments. See: https://discuss.overhang.io/t/error-during-new-setup-no-module-named-ecommerce-settings-tutor/681/10 openedx/edx-rest-api-client#62 (comment) * WS-1115 hide hidden programs from search (openedx#2639) * Updating Python Requirements * Add boto to production requirements (openedx#2641) It's used by stage/prod Discovery, but was only installed incidentally as a transitive dependency. When it was no longer transitively required, the boto requirement was removed, breaking stage Discovery. This adds boto as a direct production requirement. * WS-1116 update person endpoint to accept slug as well as uuid (openedx#2645) * WS-1151 add staff to AG index (openedx#2646) * WS-1116 fixed. Remove new endpoint and open up old endpoint to anonymous access for GET requests (openedx#2649) * WS-728: Make default search results Contentful-driven (openedx#2647) * deleting search default results model * delete model part II * code review changes * remove generate empty query rule * delete language migrations + lint * lint * WS-1165 add org logo to position serializer (openedx#2651) * WS-1143 adding degree.micromasters_path (openedx#2650) This helps out Prospectus developers reduce code striping out the protocol and hostnames. 1. Added micromastersPath to degree in serializer.js 2. update tests to pass with new field micromastersPath * Update dependency sass-loader to v9 * WS-1143: adding degree.micromasters_path to discovery (openedx#2654) * checking degree micromasters_url is a string before calling regex for micromasters_path * fixing ValidationError: Invalid options object. * WS-728: Undo moving default products to contentful (openedx#2658) * putting everything back * quality * ... * typo * rawr * i suck monumentally * Added github action to push docker image (openedx#2655) * WS-1102 add collaborator model and hook up to course model (openedx#2656) Co-authored-by: Sharon Wang <[email protected]> * Updating Python Requirements * Fixes for requirement updates * Update pytest.ini to set django settings as an option There is a priority system for setting the django settings variable (see https://pytest-django.readthedocs.io/en/latest/configuring_django.html#order-of-choosing-settings). In short: command line option --ds, environment var DJANGO_SETTINGS_MODULE, then DJANGO_SETTINGS_MODULE option listed in the configuration file (pytest.ini, tox.ini, etc.). This change updates it so when you run pytest when shelled into the container, you appropriately use the test settings file when you run pytest directly. Normally when shelling in, we source the discovery_env file (/edx/app/discovery/discovery_env) and that file sets the DJANGO_SETTINGS_MODULE environment variable to use the devstack settings file. * fix(i18n): update translation strings * Unpin drf-jwt (openedx#2644) Co-authored-by: root <[email protected]> * fix(i18n): update translations * WS-843: Boosting partner in search (openedx#2666) * boosting on partner * updated to not break facet * fix(deps): update dependency webpack to v4.44.0 * fix(deps): update dependency webpack-cli to v3.3.12 * WS-1180: Collaborator endpoints (openedx#2662) WS-1180: Collaborator API endpoints Added endpoints to POST, PATCH Collaborators Modified the Course POST, PATCH endpoints to take a list of collaborator.uuids added unit tests for collaborator added unit tests in course to cover list of collaborator.uuids Co-authored-by: root <[email protected]> * Updating Python Requirements * Update dependency webpack to v4.44.1 * Update dependency geckodriver to v1.20.0 * Updating migrations that were missed during a requirements update * MST-151 Add the Curriculum unique constraints on memberships * Update dependency sass-loader to v9.0.3 * MST-151 Add the error messages handling while saving the curriculum model so it shows clear error messages * Updating Python Requirements * Add Semgrep Github Action. (openedx#2671) * Updating Python Requirements * WS-1258 update image size for collaborators (openedx#2702) * fix(i18n): update translation strings * WS-1249: logging to debug why course slugs sporadically update (openedx#2704) * wip * added logging in courseviewset * logging course run view set * lint * logger -> log * PR comments * was that it? * fixed another test * oh * lint * WS-1292 Explicitly allow prospectus worker to get all orgs (openedx#2708) * Update CODEOWNERS * rm hack (openedx#2710) * If we have a UserThrottle row for staff, still throttle them Before this change, even if we added a custom throttle for a staff user, it wouldn't take effect. * Update Docker repo name. (openedx#2712) * Retry courses_api data loading on any valid requests * Use the full-jitter algorithm to reduce contention when retrying multiple parallel data loads * Decentralized Devstack: Fix Dockerfile; add YAML for decentralized devstack settings (openedx#2667) * Add devstack.yml for settings * dockerfile clean-up * Reorder layers * Update ports for distributed devstack ARCHBOM-1451 (openedx#2717) Consistently use the production gunicorn ports for decentralized devstack images and settings. * Serve static files in decentralized devstack ARCHBOM-1473 (openedx#2718) Explicitly add static file serving to the URL configuration in decentralized devstack, since we aren't using runserver (which does this automatically). Also: * Move all devstack-specific steps out of the base image creation * Build the static assets just for the devstack image * Turn on `DEBUG` for the devstack image (since otherwise no static or media files will be served) * Build, tag, and upload the new devstack image Co-authored-by: kdmccormick Co-authored-by: jinder1s <[email protected]> Co-authored-by: Jeremy Bowman <[email protected]> Co-authored-by: Manjinder Singh <[email protected]> * Updating Python Requirements * Fixes to support package updates * Updating Python Requirements * Update test matrix (openedx#2725) * Update memcached Docker tag to v1.6.7 * Make sure fixture-created course runs get program type info * WS-1293: Update collaborator endpoint logging to include the POST/PATCH body. * make image required in collaborator serializer * fix tests and add new post test * fix course tests * Fix error management failure on empty response Sometimes, the response object passed to the _fatal_code callback is None. This causes an AttributeError which prevents users from debugging the underlying error. * Updating Python Requirements * Remove constraint with pytest-xdist * Remove boto (openedx#2732) * Update some js dependencies file-loader from 6.0.0 to 6.1.0 sass-loader from 9.0.3 to 10.0.2 Drop imports-loader - it was only used to disable AMD importing for datatables, but that doesn't seem necessary anymore. * Bring back imports-loader My testing was bad, we do still need it. Couldn't make 1.1.0 work quickly, so I'm sticking with 0.8.0 for now. * [BD-20][RACCOON GANG][NBorovenskiy] Haystack replacement to es-dsl. Upgrade to ElasticSearch 7 (openedx#2724) Upgrade to elasticsearch 7. Work by Racoon Gang. * Revert "[BD-20][RACCOON GANG][NBorovenskiy] Haystack replacement to es-dsl. Upgrade to ElasticSearch 7 (openedx#2724)" (openedx#2740) This reverts commit bd6ea0e. * Unpin django-storages and upgrade packages * ws-1216: Only include instructors for a program's courses' advertised course runs * ws-1216: Only include instructors for a program's courses' advertised course runs * Retry any data loader that fails up to 3 times * Retry specific requests to the ecommerce apis up to 5 times * Allow RCM to manage full-ingest retries * Use concurrent.futures.as_completed to process results from ecommerce as they are available * Use Future.add_done_callback to remove custom code for sequencing request and processing of course_runs, entitlements, and enrollment_codes * Delete useless pylint supressions * Python 3.8 code cleanup with pyupgrade (openedx#2697) Run isort to sort unnitest.mock * [BD-20][RACCOON GANG][#2][NBorovenskiy] Haystack replacement to es-dsl. Upgrade to ElasticSearch 7 (openedx#2748) Upgrade to es7 from es1. * Revert "[BD-20][RACCOON GANG][#2][NBorovenskiy] Haystack replacement to es-dsl. Upgrade to ElasticSearch 7 (openedx#2748)" (openedx#2750) This reverts commit 607886d. * Updating Python Requirements * Update Dockerfile to use python 3.8 (openedx#2749) * Update Dockerfile to use python 3.8 * Removed pip-tools constraint * improved admin page for Catalogs * Added prefetching to the Program Affiliate endpoint to help with performance * Remove journals app after Juniper deprecation * Updates to improve performance of the Program and affiliate apis * create a new Django admin form for CurriculumCourseMembership in orde… (openedx#2757) * create a new Django admin form for CurriculumCourseMembership in order to use the autocomplete.ModelSelect2 widget * added space Co-authored-by: Alie Langston <[email protected]> Co-authored-by: alangsto <[email protected]> * fix(i18n): update translations * Improved program affiliate performance * Improve the prefetch of the program detail serializer to improve performance * added status in CourseSearchSerializer to get status on search/all endpoint. (openedx#2758) this status will be used to check availability of the course. * Updating Python Requirements * DISCO-1675: Use mysql 5.7 in docker/travis (openedx#2760) * WS-1358: remove the ability to add video thumbnail to disco admin * Switch to Ubuntu 20.04 focal (openedx#2773) * WS-1357: hide Program card image url in disco admin (openedx#2774) * hide card_image_url * updated test * lint * lint * Fix docker push CI step (openedx#2776) The tzdata package was prompting the user to enter a timezone. This PR tells apt/dpkg to knock off trying to ask us anything. (In this particular case, that means sticking with the default timezone of UTC, which seems fine.) * fix(deps): update dependency style-loader to v2 * Install worked without this flag. * fix(deps): update dependency sass-loader to v10.0.3 * fix(deps): update dependency file-loader to v6.1.1 * fix(deps): update dependency url-loader to v4.1.1 * fix(deps): update dependency datatables.net-bs to v1.10.22 * fix(deps): update dependency datatables.net to v1.10.22 * fix(deps): update dependency webpack to v4.44.2 * fix(i18n): update translations * Updating Python Requirements (openedx#2784) * Python 3.8 code cleanup with pyupgrade - II (openedx#2699) * Taxonomy app added and configured for course discovery. * Python 3.8 code cleanup with pyupgrade (openedx#2700) * fix(i18n): update translations * Updating Python Requirements (openedx#2789) * WS-1414 upload program card images to discovery (openedx#2791) * WS-1414 followup: update algolia index with new card image (openedx#2792) * WS-1414 followup: update algolia index with new card image * [BD-20][RG] Upgrade to elasticsearch 7 (openedx#2797) Switch from elasticsearch 1 and haystack to es7 and elasticsearch_dsl. Co-authored-by: NikolayBorovenskiy <[email protected]> * fix(i18n): update translation strings * Revert "[BD-20][RG] Upgrade to elasticsearch 7 (openedx#2797)" (openedx#2802) This reverts commit 7a08e83. * Add exec ed type * Don't allow course key to be edited in django admin * Updating Python Requirements * Add missing migration file for Mode and Program (openedx#2809) The new migration file reflects two changes: 1. Deprecation of Program.card_image_url in favor of Program.card_image 2. Addition of the executive-education choice for Mode.certificate_type * [BD-20][RACCOON GANG] Haystack replacement to es-dsl. Upgrade to ElasticSearch 7 (openedx#2808) Switch from elasticsearch 1 and haystack to es7 and elasticsearch_dsl. * Revert "[BD-20][RACCOON GANG] Haystack replacement to es-dsl. Upgrade to ElasticSearch 7 (openedx#2808)" (openedx#2814) This reverts commit 02a8aa0. * Run pyupgrade (openedx#2807) * Update dependency file-loader to v6.2.0 * Update dependency sass-loader to v10.0.4 * Update memcached Docker tag to v1.6.8 * fix(i18n): update translations * fix(i18n): update translation strings * Add a seperate requirements.python target. Sometimes you want to just install the python requirements. We had a separate target for JS but not python. * Updating Python Requirements * DISCO-1693: Set the Seats on the Course Run when changed We saw an issue where the official Course Run was not having the seat price properly updated. It had to do with the course run seats being used still had the old price because they weren't explicitly set and the course run was never refreshed from the db. This sets them and resolves the bug * Run pyupgrade (openedx#2815) * Avoid bad string decode call * fix(i18n): update translations * ENT-3534 Taxonomy Connector Version Upgrade (openedx#2825) * Remove temporary debug logging * Disallow editing fields that comprise course run keys * Updating Python Requirements * Switch to es7 (openedx#2830) Convert to elasticsearch 7 and the dsl python library, instead of haystack. * Revert "Switch to es7 (openedx#2830)" (openedx#2833) This reverts commit a348b9c. * Handle JSON decoder error if api fails * chore: upgrade Django to 2.2.18 * chore: upgrade Django to 2.2.19 * chore: upgrade Django to 2.2.20 * chore: upgrade Django to 2.2.21 * chore: upgrade Django to 2.2.22 * chore: upgrade Django to 2.2.23 * chore: upgrade Django to 2.2.24 * translation files recompiled * added missing imports * migrations fixed * LOCAL_DISCOVERY_MEDIA_URL removed from base.py * migration issue fixed * redis requirement fixed --------- Co-authored-by: Dillon Dumesnil <[email protected]> Co-authored-by: Calen Pennington <[email protected]> Co-authored-by: Michael Terry <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Renovate Bot <[email protected]> Co-authored-by: Albert (AJ) St. Aubin <[email protected]> Co-authored-by: chinmaybhatk <[email protected]> Co-authored-by: Aarif <[email protected]> Co-authored-by: edX Transifex Bot <[email protected]> Co-authored-by: Rebecca Graber <[email protected]> Co-authored-by: Stu Young <[email protected]> Co-authored-by: Hammad Ahmad Waqas <[email protected]> Co-authored-by: Awais Qureshi <[email protected]> Co-authored-by: edX requirements bot <[email protected]> Co-authored-by: Alex Dusenbery <[email protected]> Co-authored-by: Nick <[email protected]> Co-authored-by: Matt Hughes <[email protected]> Co-authored-by: Jason Myatt <[email protected]> Co-authored-by: Zaman Afzal <[email protected]> Co-authored-by: srwang <[email protected]> Co-authored-by: Adam Stankiewicz <[email protected]> Co-authored-by: Rick Reilly <credential.helper> Co-authored-by: Luis Moreno <[email protected]> Co-authored-by: Jeff LaJoie <[email protected]> Co-authored-by: Régis Behmo <[email protected]> Co-authored-by: Kyle McCormick <[email protected]> Co-authored-by: Jack Su <[email protected]> Co-authored-by: Syed Imran Hassan <[email protected]> Co-authored-by: Richard I Reilly <[email protected]> Co-authored-by: Muhammad Soban Javed <[email protected]> Co-authored-by: root <[email protected]> Co-authored-by: Simon Chen <[email protected]> Co-authored-by: Adam Blackwell <[email protected]> Co-authored-by: jinder1s <[email protected]> Co-authored-by: Jeremy Bowman <[email protected]> Co-authored-by: Manjinder Singh <[email protected]> Co-authored-by: M. Zulqarnain <[email protected]> Co-authored-by: NikolayBorovenskiy <[email protected]> Co-authored-by: Matt Tuchfarber <[email protected]> Co-authored-by: Michael Roytman <[email protected]> Co-authored-by: Alie Langston <[email protected]> Co-authored-by: alangsto <[email protected]> Co-authored-by: edX requirements bot <[email protected]> Co-authored-by: Saleem Latif <[email protected]> Co-authored-by: Feanil Patel <[email protected]> Co-authored-by: pkulkark <[email protected]> Co-authored-by: Ned Batchelder <[email protected]> Co-authored-by: Ned Batchelder <[email protected]> Co-authored-by: [email protected] <[email protected]>
Fix course_metadata migrations
Fix migrations after merging
fix migrations refactoring
Fix update_index filtering course runs - EDLY-6158
Update marketing_url for course runs - EDLY-6224
Remove unique title for programs
Fix course invite flag issue - EDLY-6132
Add partner disablement option in discovery - EDLY-6648
Fix for Too Many Open Files [6639] Closed api client connections to avoid keeping session files open
Update Dataloader api
mfarhan943
force-pushed
the
farhan/EDLY-6765
branch
3 times, most recently
from
9b0fadb
to
0facc3a
Compare
|
Wrong base branch, A new PR with correct base branch is opened. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
Map Cymanii filters in discovery course runs
Jira
https://edlyio.atlassian.net/browse/EDLY-6765