-
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
Add endpoint to update the funding status #4738
Add endpoint to update the funding status #4738
Conversation
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.
looking good a few minor comments
a6ecaac
to
0bb5bbb
Compare
a740f46
to
09501bb
Compare
70d0806
to
cf1aa89
Compare
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.
thanks pablo for addressing all comments, a few notes from testing:
- we shouldn’t allow setting a value to nil on funded applications
- it currently returns a 500 rather than 422 if application is not accepted
- We'll discuss in person when we're allowed to change which should only be on cohorts that allow capping
a32c553
to
081baef
Compare
Addressed in 081baef
Addressed in 35d11e3
Addressed in b41195a |
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.
thanks @pmanrubia!
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.
🙌
6bc9cea
to
9c30630
Compare
893dfb8
to
11e9d35
Compare
Co-authored-by: Abeer Salameh <cwrw@users.noreply.github.com>
The validations are raising errors otherwise
Co-authored-by: Abeer Salameh <cwrw@users.noreply.github.com>
Co-authored-by: Abeer Salameh <cwrw@users.noreply.github.com>
250a754
to
a910a41
Compare
Review app deployed to https://cpd-ecf-review-4738-web.test.teacherservices.cloud |
- If there are no statements do not attempt a query for contracts - Fix specs to point to correct cpd lead provider
Ticket
https://dfedigital.atlassian.net/browse/CPDLP-2949
Context
Adds endpoint to change the
funded_place
attribute for an application. It validatesthat the request can be executed within the current application status.