Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add reference to new generic verification model in platform #223

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

Zacharis278
Copy link
Member

Description:

Adds a reference to the new generic verification model introduced into edx-platform as part of openedx/platform-roadmap#367.

This only introduces the model and database migration. Existing features will be updated to utilize this in followup PRs

JIRA:

COSMO-492

Pre-Merge Checklist:

  • [N/A] Updated the version number in edx_name_affirmation/__init__.py if these changes are to be released. See OEP-47: Semantic Versioning.
  • Described your changes in CHANGELOG.rst.
  • Confirmed Github reports all automated tests/checks are passing.
  • Approved by at least one additional reviewer.

Post-Merge:

  • [N/A] Create a tag matching the new version number.

Copy link

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  edx_name_affirmation
  models.py
  edx_name_affirmation/tests
  test_models.py
Project Total  

This report was generated by python-coverage-comment-action

@Zacharis278 Zacharis278 force-pushed the zhancock/add-platform-verification-ref branch from 1d23e7e to b7ce9ca Compare September 19, 2024 17:48
Copy link
Member

@schenedx schenedx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Zacharis278 Zacharis278 merged commit d919950 into main Sep 20, 2024
6 checks passed
@Zacharis278 Zacharis278 deleted the zhancock/add-platform-verification-ref branch September 20, 2024 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants