-
Notifications
You must be signed in to change notification settings - Fork 66
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
98361: Update DR engine controllers to use engine jobs #19920
Conversation
- also contestable issues endpoints -v1 and v2
- evidence - notification callbacks
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.
LGTM!
Generated by 🚫 Danger |
Hi, I must ask if there's a way to break this PR up. It seems to be an unnecessary amount of functionality to add in a single PR. While I understand that some changes require everything to go in at once, this seems like something that can go in piece by piece. We have the 500 LOC rule for a reason - please try to break down. |
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.
Please break down into smaller PRs
Sorry about that - this PR was waiting for #19789 to be merged, so most of the commits were duplicated off of those. I'll merge and update so it'll make the PR smaller. |
…ts-api into 98361-dr-engine-job-versions
PR is ready for review. |
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.
LGTM*
* G = Gooder
* Duplicate main form endpoints - also contestable issues endpoints -v1 and v2 * Fix linting errors * Duplicate final endpoints - evidence - notification callbacks * Fix shared example constant collision * 98361: Update DR engine controllers to use engine jobs --------- Co-authored-by: Molly Trombley-McCann <[email protected]>
Summary
This PR adds two feature flags to switch to the engine versions of the
SubmitUpload
andForm4142Submit
jobs when using the Decision Review engine controllers.decision_review_new_engine_submit_upload_job
decision_review_new_engine_4142_job
This is owned by Benefits Decision Reviews.
Related issue(s)
department-of-veterans-affairs/va.gov-team#98361
Testing done
Tested locally and via spec tests.
The new jobs are called from the new controllers instead of the original jobs when the respective feature flag is enabled.
What areas of the site does it impact?
Controllers for the decision reviews functionality which are duplicated in engine context.
Acceptance criteria