-
Notifications
You must be signed in to change notification settings - Fork 18
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
Caceves/vineel 42281 #21533
Closed
Closed
Caceves/vineel 42281 #21533
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* APPEALS-29410 Response letter section * create componenet to add letter * fix button * enabled and disabled continue buttom * custom part * selector element * move state variable * slector value * error in selector * Selectors * Selectors 2 * change selector * change selector * remove part * Response windows * Response windows again * updation radio * clean values * Fix option for custom radio * fix reason value * Fix option in the radio buttom * Fix rubocop error * Add spec file * Add spec file * Remove comments for the code * modified options in the response wimdows and refactor de code * remove import in AddCorrespondenceView * confirmation of response letters * fix custom field * adding dispatch * fix response windows * refactor * continue funcionality * contiue button * adding response letter section * fixing lint * fixing table * lint fixes * refactoring * appending days to custom value * fixing lint errors * fixing lint * fixing lint --------- Co-authored-by: Kamala Madamanchi <kamala.madamanchi@va.gov> Co-authored-by: piedram <maite.piedrayera@va.gov>
Co-authored-by: Kevin Ma <magavakevin10@gmail.com>
* APPEALS-29410 Response letter section * create componenet to add letter * fix button * enabled and disabled continue buttom * custom part * corrspondence response letter changes * specs and refactoring * selector element * move state variable * slector value * error in selector * Selectors * Selectors 2 * change selector * change selector * remove part * Response windows * Response windows again * updation radio * clean values * Fix option for custom radio * rubocop fix * fix reason value * Fix option in the radio buttom * Fix rubocop error * Update migration and rubocop fixes * Add spec file * Add spec file * Remove comments for the code * modified options in the response wimdows and refactor de code * remove import in AddCorrespondenceView * confirmation of response letters * fix custom field * adding dispatch * fix response windows * refactor * continue funcionality * contiue button * adding response letter section * fixing lint * fixing table * lint fixes * Add continue botton * Save data in DB * change style * Save values * fix continue buttom * dix submit button --------- Co-authored-by: Kamala Madamanchi <kamala.madamanchi@va.gov> Co-authored-by: piedram <maite.piedrayera@va.gov> Co-authored-by: vinner57 <128258952+vinner57@users.noreply.github.com>
* adding correspondence task default route * adding intake and review package route overrides * adding efolder ulpoad failed logic for routing * adding efolder failed upload task depending on parent task * fixing intake scoping * adding task_url attribute to column serializer * attempting a correspondence root task addition * Moved constant file for access * removing root task method, since all defaults to /under_construction * replacing task urls from a newly created constants file * code cleanup * adding tests * rubocop and linting errors * rubocop fixes * Serialize attributes regardless of values * Make CorrespondenceTask an abstract class * Make package action tasks inherit from CorrespondenceTask * Update to not intatiate abstract CorrespondenceTasks * Refactor queue correspondences seeds * Fix broken pagination on Correspondence Cases team page * Merge with feature 32900 * Randomize va date of receipt * Remove expectation to find instance of CorrespondenceTask - an abstract class * Remove expectation for abstract class to appear in task tree * Assign ReviewPackageTask before creating EfolderUploadFailedTask * Rubocop fixes --------- Co-authored-by: sulak-j <justin.sulak@va.gov> Co-authored-by: Christopher Aceves <christopher.aceves@va.gov>
* Create OrganizationPermissions * Add Column Comments to Migration * Add Indices Using add_safe_index * Create OrganizationUserPermission and Related Files * Change OrganizationUserPermission Inheritance --------- Co-authored-by: youfoundmanesh <129548081+youfoundmanesh@users.noreply.github.com>
* Initial job files created; added RunAsyncable module to controller * Added aria-label for spec fix * Refactor to functional component * Added Auto Assign Correspondences Button, added route, added method to controller * Updated spec for future testing * Added initla business logic for assignment * Updated seed data for VADOR date * Update to include current user for seed data when ran individually * Optimized SQL query by 10% * Updated instruction details to contain css_id * Update reek file for job, fixed rubocop and linting * Updated class name * Removed unused constant
* Refactor Correspondence Auto-Assign Logic - Move logic into new service class called by auto-assign job. - Add specs and other supporting files. * Add Missing Method Name * Fix Data Setup Causing Spec Failure
* Refactor for Auto-Assignment Permission Check - Rework auto-assignment logic and add new check for auto-assign user permission. - Add specs and supporting files. * Fix Linting Issues * Add Seeds for Auto-Assign Permissions
* Refactor for Auto-Assignment Permission Check - Rework auto-assignment logic and add new check for auto-assign user permission. - Add specs and supporting files. * Fix Linting Issues * Add Seeds for Auto-Assign Permissions * add nod check * Update rspec * small fix --------- Co-authored-by: Alexander Smith <alex.smith.99@thunderyard.com>
* Add Auto-Assignment Logging Tables * Add User Association to BatchAutoAssignmentAttempt
* Created auto assign logger for statistics * Added comments for future work * Updated method * Update log * Added current user to initialization * Added current user to @current_user, changed record methods from private to public, added parameters to assignment method * Added base code for rspecs
- Refactor auto assigner class to follow master algorithm and move user finding logic piece into separate class. - Update a few classes with support methods needed and add specs.
* Update MailTeamSupervisor with InboundOpsTeam (#20529) * Add permissions to Organizations (#20543) * Create OrganizationPermissions * Add Column Comments to Migration * Add Indices Using add_safe_index * Create OrganizationUserPermission and Related Files * Change OrganizationUserPermission Inheritance --------- Co-authored-by: youfoundmanesh <129548081+youfoundmanesh@users.noreply.github.com> * Kev ma/appeals 38420 (#20527) * Initial job files created; added RunAsyncable module to controller * Added aria-label for spec fix * Refactor to functional component * Added Auto Assign Correspondences Button, added route, added method to controller * Updated spec for future testing * Added initla business logic for assignment * Updated seed data for VADOR date * Update to include current user for seed data when ran individually * Optimized SQL query by 10% * Updated instruction details to contain css_id * Update reek file for job, fixed rubocop and linting * Updated class name * Removed unused constant * Refactor Correspondence Auto-Assign Logic (#20556) * Refactor Correspondence Auto-Assign Logic - Move logic into new service class called by auto-assign job. - Add specs and other supporting files. * Add Missing Method Name * Fix Data Setup Causing Spec Failure * Create Auto Assignment Permission Check (#20565) * Refactor for Auto-Assignment Permission Check - Rework auto-assignment logic and add new check for auto-assign user permission. - Add specs and supporting files. * Fix Linting Issues * Add Seeds for Auto-Assign Permissions * Deepak/appeals 38547 (#20578) * Refactor for Auto-Assignment Permission Check - Rework auto-assignment logic and add new check for auto-assign user permission. - Add specs and supporting files. * Fix Linting Issues * Add Seeds for Auto-Assign Permissions * add nod check * Update rspec * small fix --------- Co-authored-by: Alexander Smith <alex.smith.99@thunderyard.com> * Add Auto-Assignment Logging Tables (#20608) * Add Auto-Assignment Logging Tables * Add User Association to BatchAutoAssignmentAttempt * Created auto assign logger for statistics (#20610) * Created auto assign logger for statistics * Added comments for future work * Updated method * Update log * Added current user to initialization * Added current user to @current_user, changed record methods from private to public, added parameters to assignment method * Added base code for rspecs * Refactor Auto-Assignment and Add Sensitivity Check - Refactor auto assigner class to follow master algorithm and move user finding logic piece into separate class. - Update a few classes with support methods needed and add specs. * Add Correspondence Auto Assignment Logging - Rework class to log auto-assignment attempts. - Add logging to auto-assigner. - Add specs and rework some constants to follow app guidelines. --------- Co-authored-by: SanthiParakal133 <132940479+SanthiParakal133@users.noreply.github.com> Co-authored-by: youfoundmanesh <129548081+youfoundmanesh@users.noreply.github.com> Co-authored-by: Kevma50287 <104021955+Kevma50287@users.noreply.github.com>
* Create Inbound Ops users * create Mail Team and Mail Team super user * Update name * update sensitivity_level for users and veterans * Revert "update sensitivity_level for users and veterans" This reverts commit 877e941. * small fix
* Create CreateCorrespondenceAutoAssignmentLevers * Add Uniqueness Constraint
* adding correspondence task default route * adding intake and review package route overrides * adding efolder ulpoad failed logic for routing * adding efolder failed upload task depending on parent task * fixing intake scoping * adding task_url attribute to column serializer * attempting a correspondence root task addition * Moved constant file for access * removing root task method, since all defaults to /under_construction * replacing task urls from a newly created constants file * code cleanup * adding tests * rubocop and linting errors * rubocop fixes * Serialize attributes regardless of values * Make CorrespondenceTask an abstract class * Make package action tasks inherit from CorrespondenceTask * Update to not intatiate abstract CorrespondenceTasks * Refactor queue correspondences seeds * Fix broken pagination on Correspondence Cases team page * Merge with feature 32900 * Randomize va date of receipt * Remove expectation to find instance of CorrespondenceTask - an abstract class * Remove expectation for abstract class to appear in task tree * Merge in task url work * Add ability to link veterans details to tasks or modals * Route to under construction if closed * Assign ReviewPackageTask before creating EfolderUploadFailedTask * give ids and labels to column items * Add CorrespondenceTaskHelpers module * Test: correspondene queue task links spec * Move url strings into constants file * Fix extra space --------- Co-authored-by: sulak-j <justin.sulak@va.gov>
* Reset BGS Service * Add New Sensitivity Check Method - Create new method in BGSService for checking participant sensitivity using the BGS gem. - Update auto assign logic to use new method for checking sensitivity for user/veteran pairs. * Fix Lint Error * Update Sensitivity Check Logic - Use new method in BGS gem for checking user sensitivity levels. - Update specs and supporting files accordingly.
* Move Auto Assign Button to Correspondence Queue (#21324) * Move Auto Assign Button to Correspondence Queue * Fix Lint Errors * Reset BGS Service (#21352) * Use New BGS Gem Sensitivity Check Methods (#21356) * Reset BGS Service * Add New Sensitivity Check Method - Create new method in BGSService for checking participant sensitivity using the BGS gem. - Update auto assign logic to use new method for checking sensitivity for user/veteran pairs. * Fix Lint Error * Update Sensitivity Check Logic - Use new method in BGS gem for checking user sensitivity levels. - Update specs and supporting files accordingly. --------- Co-authored-by: cacevesva <109166981+cacevesva@users.noreply.github.com> Co-authored-by: Christopher Aceves <christopher.aceves@va.gov>
* Initial commit for UI/UX changes * Added the existing caseflow search properties * Updated the checkboxes style fix * Changes for spec file with the latest code changes * Modified the seed data to check pagination
* Initial commit for UI/UX changes * Added the existing caseflow search properties * Updated the checkboxes style fix * Changes for spec file with the latest code changes * Modified the seed data to check pagination * Modified the lint issue change for if condition * Modified the lint issue change for if condition
…21371) * create CMP rake task to generate correspondence data in UAT * added vbms doc types to rake task * removed useless method
…-of-veterans-affairs/caseflow into feature/APPEALS-32900
* change eFolderUploadFailedTask Routing * change condicion for efolder --------- Co-authored-by: Jim Foley <james.foley4@va.gov>
* creating review package controller, moving methods, editing routing * Moving review package spec tests to new RP Spec file * cleaning up code from intake controller spec
…1380) Co-authored-by: Jim Foley <james.foley4@va.gov>
- Fix bug causing correspondences not be sorted by VA DOR in ascending (oldest -> newest) order. - Update specs to test this scenario.
* Added a join clause of appeal_type = 'Appeal' to the with_original_appeal_and_judge_task method in the distribution_scopes.rb to prevent a sequential table scan which was causing a database timeout with large datasets in the tasks table in production and prod test. (#21386) Co-authored-by: = <tyler.broyles@va.gov> * Hotfix/appeals 21696 (#21343) * fix(create conference): update test for nil appellant email * fix(create conference): nil appellant email check * fix(conference job): lint * fix(create conference): minor * null * null * null --------- Co-authored-by: Raymond Hughes <131811099+raymond-hughes@users.noreply.github.com> * Hotfix/appeals 23420 v3 (#21347) * Add LegacyAppealSearchSerializer to rendering * Make additional improvements to the Legacy Serializer * Add MST and PACT to both search serializers respectively * schema check-in --------- Co-authored-by: raymond-hughes <raymond.hughes2@va.gov> Co-authored-by: Raymond Hughes <131811099+raymond-hughes@users.noreply.github.com> * hotfix/APPEALS-27522 (#21326) * update erroneous db cleaner syntax and add temp fix to changeset_at_index_for_task method * update changeset_at_index_for_task method to account for lint error * fix lint error and code climate error --------- Co-authored-by: Raymond Hughes <131811099+raymond-hughes@users.noreply.github.com> * Hotfix/appeals 10993 v2 (#21349) * added logic for deciding aod on attorney claimant * added tests * refactored test to return true when motion is granted --------- Co-authored-by: Andrew Hadley <hadley_andrew@bah.com> Co-authored-by: Raymond Hughes <131811099+raymond-hughes@users.noreply.github.com> * Fixed a bug where some of the Specialty Case Team assign queue settings would take priority over judge assign queue settings for judges who were also members of the specialty case team organization. (#21387) Co-authored-by: = <tyler.broyles@va.gov> * Merge hotfix/APPEALS-43956 into release/FY24Q2.7.0 (#21389) * Added additional logic checks to the attorney task cancel to fix a bug where the cancel attorney task would attempt to update a non existing SpecialtyCaseTeamAssignTask when the appeal matches the sct_appeal? criteria of a 'vha' benefit type. * Removed feature toggle from the checks since the cancel task should work properly for appeals that were disributed while the feature toggle used in distribution was turned on. * Added a regression test. --------- Co-authored-by: = <tyler.broyles@va.gov> * Merge hotfix/APPEALS-43990 into release/FY24Q2.7.0 (#21388) * Added a nil value to the inclusion field of the distributed case validation for the sct_appeal boolean that was causing an error when calling the rename_for_redistribution! on a distributed case where the value was not already set to true and false * Added a regression test for the sct_appeal validation. --------- Co-authored-by: = <tyler.broyles@va.gov> * Added a null coalesce to prevent a javascript error caused by a race condition where remaining users could be nil instead of an array if the user is searching for a user at the same time they are adding a user. (#19487) * APPEALS-43712, APPEALS-43835: Open VeteranRecordRequest and QualityReviewTask are causing distributions to fail (#21354) * remove Appeal.can_redistribute_appeal? from JudgeAssignTask creation during normal distribution * update test * re-add tests for validating each bug scenario * disable mail task test that is failing due to how it is written * updating hearing_shift_date to prevent future hearing dates from occuring in local/demo (#21306) Co-authored-by: Craig Reese <109101548+craigrva@users.noreply.github.com> * Lint fixes (#21383) * APPEALS-43423 - AssignedReassignPackageTask remove user limitation (#21400) * Ryandwill/appeals 43393 (#21339) * refactored correspondence_controller_util into correspondence_controller_concern * refactor correspondence_controller * updated method to be class level and in CorrespondenceTask model * Updating the class methods from controller to model * Updated the correspondence_task.rb * Update intake endpoint with custom serialization and move logic to models * update QueueApp with new props * remove controller passed props from redux * update frontend components for correspondence objects new data shape * Update test for controller changes * update test for new controller changes * update test for controller changes * fix typo in error msg and related tests * remove unused argument * remove unneeded prop validation * move task update logic and banner presentation to correspondence tasks controller * move team action to queue controller * rubocop fix * remove review package methods to prepare for feature merge * clean up review package veteran update * fix test to follow assertion * update spec for new pagination requirements * fix typo * refactor reassign/remove package modals for RP page * fix test setup to create RemovePackageTask properly * fix test selector * fix test - quotation marks and hyphen character in string * fix style issues * rspec fixes * update rp controller inbound ops team check * remove pry * removed leftover binding pry * remove pry * remove previously deleted code * fix flaky test * fix tests --------- Co-authored-by: Ryan Marshall <rnmarshall1993@gmail.com> Co-authored-by: divyadasari-va <divya.dasari@va.gov> Co-authored-by: HunJerBAH <Jeremy.Hunton@va.gov> * Move Correspondence ART CSS Styling Into a Single Stylesheet- Queue - APPEALS-43392 (#21384) * WIP * All styling has meen moved to scss * Small fixes * fixed inbound ops team being listed in queuetable dropdown (#21405) * fixed inbound ops team being listed in queuetable dropdown * refactored isMailTeamAffiliated * removed commented out code --------- Co-authored-by: HunJerBAH <Jeremy.Hunton@va.gov> Co-authored-by: HunJerBAH <99915461+HunJerBAH@users.noreply.github.com> * Reassign Correspondence Allows Reassignment over 60 Limit (#21403) * Reassign Correspondence Allows Reassignment over 60 Limit * Reassign Correspondence Allows Reassignment over 60 Limit * added limit check to banner message and resolve linting error with safety operator * remove file app/models/concerns/correspondence_controller_util.rb --------- Co-authored-by: HunJerBAH <Jeremy.Hunton@va.gov> * Kamalam7/appeals 35469 (#21407) * Return to queue from the Intake form * specs for Return to queue from Intake form * Resolved merge conflicts for Return to queue Intake form * Lint fix in Correspondence intake file --------- Co-authored-by: HunJerBAH <99915461+HunJerBAH@users.noreply.github.com> * Wmedders21/appeals 44301 (#21413) * correspondence controller specs live in spec/requests * Remove unused veteran endpoint * Add request tests for user access and feature flag * add tests for review package controller * remove unnecessary line * refactor test for simplecov coverage * remove unneeded test * add intake spec * add controller tests * add erroneously removed method * Fix rubocop style * fix selector in flaky test * add error class to rescue --------- Co-authored-by: HunJerBAH <99915461+HunJerBAH@users.noreply.github.com> * Justin/appeals 43606 (#21399) * lining up misaligned brackets * CorrespondencePdfFooter inline styles removed * removing unused class and styling * CmpDocuments.jsx styles into a stylesheet * CorrespondenceReviewPackage.jsx styles into a stylesheet * ReviewForm.jsx WIP * moving css out of ReviewPackageCaseTitle * ReviewForm.jsx styles into a stylesheet * scss fix * CorrespondencePdfToolBar.jsx styles into a stylesheet * ReviewPackageData file scss * removing excess styling to keep textarea at application default sizing * updated test on intake spec * cleaned up old binding pry --------- Co-authored-by: Dev-KRedd <khalin.redding@va.gov> Co-authored-by: HunJerBAH <99915461+HunJerBAH@users.noreply.github.com> Co-authored-by: HunJerBAH <Jeremy.Hunton@va.gov> * Add unit tests (#21421) * added completed_by_date * APPEALS-44300 - Prior Mail test * created approve unit test on ReassignPackageTask model. * added tests * added remove package tasks approve and reject methods * Reassign Correspondence Allows Reassignment over 60 Limit (#21403) * Reassign Correspondence Allows Reassignment over 60 Limit * Reassign Correspondence Allows Reassignment over 60 Limit * added limit check to banner message and resolve linting error with safety operator * remove file app/models/concerns/correspondence_controller_util.rb --------- Co-authored-by: HunJerBAH <Jeremy.Hunton@va.gov> * Kamalam7/appeals 35469 (#21407) * Return to queue from the Intake form * specs for Return to queue from Intake form * Resolved merge conflicts for Return to queue Intake form * Lint fix in Correspondence intake file --------- Co-authored-by: HunJerBAH <99915461+HunJerBAH@users.noreply.github.com> * Wmedders21/appeals 44301 (#21413) * correspondence controller specs live in spec/requests * Remove unused veteran endpoint * Add request tests for user access and feature flag * add tests for review package controller * remove unnecessary line * refactor test for simplecov coverage * remove unneeded test * add intake spec * add controller tests * add erroneously removed method * Fix rubocop style * fix selector in flaky test * add error class to rescue --------- Co-authored-by: HunJerBAH <99915461+HunJerBAH@users.noreply.github.com> --------- Co-authored-by: KiMauVA <Ki.Mau@va.gov> Co-authored-by: piedram <110848569+piedram@users.noreply.github.com> Co-authored-by: HunJerBAH <Jeremy.Hunton@va.gov> Co-authored-by: Kamala Madamanchi <110078646+kamala-07@users.noreply.github.com> Co-authored-by: HunJerBAH <99915461+HunJerBAH@users.noreply.github.com> Co-authored-by: Will Medders <93014155+wmedders21@users.noreply.github.com> --------- Co-authored-by: Robert Travis Pierce <robert@roberttravispierce.com> Co-authored-by: = <tyler.broyles@va.gov> Co-authored-by: Christian Cain <149622375+christian-cain-bah@users.noreply.github.com> Co-authored-by: Raymond Hughes <131811099+raymond-hughes@users.noreply.github.com> Co-authored-by: Aiman Kayad <Akaya005@fiu.edu> Co-authored-by: raymond-hughes <raymond.hughes2@va.gov> Co-authored-by: Dani Coleman <60626984+dcoleman21@users.noreply.github.com> Co-authored-by: Drew Hadley <50673809+VandelayUtd@users.noreply.github.com> Co-authored-by: Andrew Hadley <hadley_andrew@bah.com> Co-authored-by: Tyler Broyles <109369527+TylerBroyles@users.noreply.github.com> Co-authored-by: Craig Reese <109101548+craigrva@users.noreply.github.com> Co-authored-by: Isaiah Saucedo <irsaucedo5@gmail.com> Co-authored-by: divyadasari-va <135847343+divyadasari-va@users.noreply.github.com> Co-authored-by: Ki Mau <ki.mau@va.gov> Co-authored-by: Will Medders <93014155+wmedders21@users.noreply.github.com> Co-authored-by: Ryan Marshall <rnmarshall1993@gmail.com> Co-authored-by: divyadasari-va <divya.dasari@va.gov> Co-authored-by: Dev-KRedd <134650238+Dev-KRedd@users.noreply.github.com> Co-authored-by: Rnmarshall93 <110805785+Rnmarshall93@users.noreply.github.com> Co-authored-by: piedram <110848569+piedram@users.noreply.github.com> Co-authored-by: Kamala Madamanchi <110078646+kamala-07@users.noreply.github.com> Co-authored-by: sulak-j <95875751+sulak-j@users.noreply.github.com> Co-authored-by: Dev-KRedd <khalin.redding@va.gov>
* consolidated backend checks to use inbound_ops_team_superuser over mail_superuser * updated superuser namespace in the frontend
* cleaned up magic string usage * updated cancel routing for superusers * fixed linting issues * added rerouting for users that cannot access a specific intake * updated Review Package task to be the same user that was assigned the Intake * resolved linting issues * updated inbound ops team superuser check * refactored intake controller test * refactored reassign package task specs and reassignment on cancel * refactored task specs to be more DRY * added intake task fix to intake_form_spec * cleaned up test failures related to intake task within intake. * refactored related correspondence spec tests * updated test to click correct section * updated failing snapshots * fixed typo on prop validations
* APPEALS-41707 - Functionality and some CSS * APPEALS-41707 - Updates * APPEALS-41707 - Unrelated Specs and Correspondence CSS Lint fixes * APPEALS-41707 - Additional Linting * APPEALS-41707 - More Clean up
* added migrations to refactor correspondence intake relationship to task instead of user/correspondence * updated model relations and controller check for an open intake * updated specs to reflect code changes. * removed leftover pry
Code Climate has analyzed commit a02ef40 and detected 23 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
closing just testing code climate. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves Jira Issue Title
Description
Please explain the changes you made here.
Acceptance Criteria
Testing Plan
Frontend
User Facing Changes
Storybook Story
For Frontend (Presentation) Components
MyComponent.stories.js
alongsideMyComponent.jsx
)Backend
Database Changes
Only for Schema Changes
created_at
,updated_at
) for new tablesCaseflow::Migration
, especially when adding indexes (useadd_safe_index
) (see Writing DB migrations)migrate:rollback
works as desired (change
supported functions)make check-fks
; add any missing foreign keys or add toconfig/initializers/immigrant.rb
(see Record associations and Foreign Keys)belongs_to
for associations to enable the schema diagrams to be automatically updatedIntegrations: Adding endpoints for external APIs
Best practices
Code Documentation Updates
Tests
Test Coverage
Did you include any test coverage for your code? Check below:
Code Climate
Your code does not add any new code climate offenses? If so why?
Monitoring, Logging, Auditing, Error, and Exception Handling Checklist
Monitoring
Logging
Auditing
Error Handling
Exception Handling