feat: receiver for verified exam event - #33390
Conversation
df7196d to
0ec31dd
Compare
There was a problem hiding this comment.
Added comment to function that is causing circular dependency issue when trying to do a top level import of any functions from api.py. This comment is taken from the PR that introduced this dependency.
There was a problem hiding this comment.
Is this what was discussed in the huddle yesterday?
0ec31dd to
dc0effc
Compare
There was a problem hiding this comment.
Nice! Mocking is easy to read and understand.
There was a problem hiding this comment.
Is this what was discussed in the huddle yesterday?
There was a problem hiding this comment.
And this is also related to the convo yesterday about needing to do the import inline?
There was a problem hiding this comment.
Yes! This is what we chatted about in the huddle - because the python API for this service is meant to be used both externally and internally, we end up with some circular dependencies. To avoid that, we have to use an inline import here
dc0effc to
9574690
Compare
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
5 participants
MST-2109
Description
This PR implements a new consumer for the
EXAM_ATTEMPT_VERIFIEDopenedx event, which is sent via the event bus. The handler triggers an undo of a subsection grade override, if one exists.