search

openedx / edx-platform

The Open edX LMS & Studio, powering education sites around the world!
https://openedx.org
GNU Affero General Public License v3.0
7.1k stars 3.79k forks source link

fix: remove broken management command (and celery task) logic #34972

Closed justinhynes closed 3 weeks ago

justinhynes commented 3 weeks ago

Description

[APER-3385]

This PR fixes an existing management command that now has incorrect logic. We have recently done a lot of work to improve certificate-related date business logic to fix data inconsistencies between systems. Instead of maintaining separate and duplicated logic for sending date data to Credentials, instead we can use an existing (and tested) Celery task that will determine and send the correct date to the Credentials IDA.

Additionally, the original version of this management command skipped self-paced courses completely. This is no longer the case as we know that there are self-paced courses that have been associated with a certificate available date because of bugs in the product. This management command will serve as a means to do a mass data fixup for data stored by the Credentials IDA.

edx-pipeline-bot commented 3 weeks ago

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

edx-pipeline-bot commented 3 weeks ago

2U Release Notice: This PR has been deployed to the edX production environment.