openedx/openedx-platform

cleanup direct model imports from certificates djangoapp

开放

#36,647 创建于 2025年5月1日

 (4 条评论) (0 个反应) (1 位负责人)Python (4,331 个派生)auto 404
good first issuehelp wanted

仓库指标

星标
 (8,150 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

There are a small number of direct imports from the certificates djangoapp's models.py. Wherever possible, we want to use the app's API) or data classes for use outside of the certificates app itself.

As of this writing, this appears to be all imports of the app that are not from api.py, data.py, test factories, the signals, or an exception definition:

the majority of them appear to be in legacy front-ends, unsurprisingly.

Acceptance Criteria

  • create any necessary data classes to get rid of direct imports of lms.djangoapps.certificates.models from outside the django app
  • change the code to reference the appropriate data class or API method
  • if it is difficult, costly, or impossible to make the change, consider adding a comment explaining why
  • verify that all the affected apps still work properly, and that all the tests pass

贡献者指南