feat: add ability to notify credentials about honor certificates

[DmytroAlipov]

Description

Honor mode is absent by default in the course list for "Program Records" that appears error for "Program Records" for users with permission "Active" after clicking on "View My Records" button:

500 error appeared

Implemented:

• Developed solution for adding course mode "Honor" to the list of modes for "Program Records"
• "Earned" and "completed" status are set up for Honor Courses and Program after finishing the program for staff and particular user

Added a WaffleFlag course_modes.extend_certificate_relevant_modes_with_honor. When enabled - credential will receive data about the HONOR course certificates.

[openedx-webhooks]

Thanks for the pull request, @DmytroAlipov! 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.

[mphilbrick211]

Hi @openedx/2u-aperture! Is this something you could review/merge?

[justinhynes]

I added an internal ticket for tracking and prioritization purposes -- APER-2691.

[mphilbrick211]

@jmakowski1123 - Per Kelly's comment here, would you mind adding this to your product review queue? Thanks!

#32745 is the backport for this (which Kelly moved to a draft until product review is complete on this one)

[mphilbrick211]

Hi @DmytroAlipov! Flagging the new tests that have popped up, per the recent update: https://discuss.openedx.org/t/minor-change-to-edx-platform-check-statuses/11131

[mphilbrick211]

Hi @jmakowski1123! Is there a roadmap ticket for this yet (for Product review)?

[mphilbrick211]

Hi @jmakowski1123! Is there a roadmap ticket for this yet (for Product review)?

@jmakowski1123 - following up on this

[jmakowski1123]

This is approved. I think it should remain behind a flag to allow operators the flexibility to decide how and when credentials obtained in honor mode should be displayed, particularly because use cases for the honor mode are varied.

[mphilbrick211]

This is approved. I think it should remain behind a flag to allow operators the flexibility to decide how and when credentials obtained in honor mode should be displayed, particularly because use cases for the honor mode are varied.

@ihor-romaniuk FYI ^

[mphilbrick211]

@justinhynes @openedx/2u-aperture - this is all set from Product and ready for review.

[jsnwesson]

@mphilbrick211 I'll be bringing this up with the team next Tuesday! Ideally we'll be able to prioritize it for the following sprint.

[mphilbrick211]

Thank you, @jsnwesson!

[justinhynes]

I plan on taking a look at this PR today. Thanks for your patience.

[DmytroAlipov]

Hi @justinhynes
Following your advice, I used is_eligible_for_certificate. Indeed, this also works correctly.
This allowed us not to add a new WaffleFlag.

Regarding the question Was there a reason the existing functionality wasn't sufficient in the notify_credentials work flow?, the service LMS does not notify Credentials about the user receiving Honor's certificates.

Without changes in the code, we get the following result when going to the “Program Record” page:

With changes:

[justinhynes]

Thanks for the update here, this should be prioritized for review shortly.

[justinhynes]

Thanks for this @DmytroAlipov! Looks good, I've approved it but there are a few conflicts before merge. Any chance you can fix those up and then I'll ensure this gets merged?

[mphilbrick211]

@justinhynes - this is a backport to #32745 which has been closed. Can this be closed as well?

[justinhynes]

Hi @mphilbrick211, I'm just a little confused. I am keeping an eye on this PR and once the tests pass, I can merge it. I wasn't planning on closing this PR without merging though.

I did close the backport of this change to Palm, as Palm is quite old now.

[DmytroAlipov]

@justinhynes Finally, all tests passed successfully😅

[openedx-webhooks]

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

[edx-pipeline-bot]

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

[edx-pipeline-bot]

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

[edx-pipeline-bot]

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

[edx-pipeline-bot]

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