feat: [FC-0006] add Verifiable Credentials optional feature

[wowkalucky]

Note: for the sake of review effort simplification, it is decided to split this large PR onto 2 parts:

• initial part - introduces a feature flag with a new application modules structure
• main part - adds all the implementation logic

This is the part 1/2 of the Verifiable Credentials feature.

Related PRs:

• 2nd part
• documentation
• configuration

---

Run JavaScript tests locally with Karma

There is work being done on a fix to get Karma to run in CI. Until then, however, contributors are required to run these tests locally.

• Make sure you are inside the devstack container
• Run make test-karma
• All tests pass

---

• Relates to feat: [FC-0006] add Verifiable Credentials optional feature frontend-app-learner-record#151

TODO

• - add related Open edX Credentials PR link.
• - update test coverage

[openedx-webhooks]

Thanks for the pull request, @wowkalucky! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[e0d]

@wowkalucky confirming this should still be in draft?

[wowkalucky]

@e0d coverage is almost there. Moved for review.
The Verifiable Credentials documentation is a matter of a separate PR.

[wowkalucky]

@e0d we increased patch coverage now. Should we proceed to reach the overall project's target value in this PR?

[mphilbrick211]

Hi @AliAdnanSohail - would you mind taking a look at this and merging if all looks good? Thanks!

[hurtstotouchfire]

This is in our current sprint so it should be reviewed and hopefully merged within 2 weeks.

[mphilbrick211]

Hi @AliAdnanSohail - would you mind taking a look at this and merging if all looks good? Thanks!

Hi @AliAdnanSohail - I tagged you by mistake, so please disregard - no need to review this PR.

[justinhynes]

As I understand it, the __init__.py of a Django app should have a default_app_config defined and point to the Config class defined in the apps.py (in this case the VerifiableCredentialsConfig class).

Not sure if this was missed or just being planned for later, but figured I'd add it here in case it was missed.

(I also have a half-baked assumption that since the ready function isn't defined as part of the VerifiableCredentialsConfig class, we might not be able to add a declaration for the default_app_config yet)

[justinhynes]

(I also see that there is a larger part 2 OSPR I wasn't immediately aware of, we could probably address that there)

[openedx-webhooks]

@wowkalucky 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.