-
Notifications
You must be signed in to change notification settings - Fork 8
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
Restore funded_place if modified after acceptance #4790
Restore funded_place if modified after acceptance #4790
Conversation
Review app deployed to https://cpd-ecf-review-4790-web.test.teacherservices.cloud |
@@ -16,6 +16,9 @@ def call | |||
return self unless valid? | |||
|
|||
npq_application.update!(lead_provider_approval_status: "rejected") | |||
if FeatureFlag.active?(:npq_capping) && npq_application.funded_place? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah this is the wrong service, can we please only do this on the ChangeToPending
service and set this to nil
please as per the ticket
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed in 7ad22c1
d17199d
to
7ad22c1
Compare
7ad22c1
to
ab34683
Compare
@@ -33,6 +33,9 @@ def call | |||
profile.destroy! | |||
end | |||
npq_application.update!(lead_provider_approval_status: "pending") | |||
if FeatureFlag.active?(:npq_capping) && !npq_application.funded_place.nil? | |||
npq_application.update!(funded_place: false) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we please set this to nil
not false?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed in fb5852c
Context
Description
We set a funded place when an application is accepted. If we revert the acceptance we also need to unset the capping change.