…w mail task

…-of-veterans-affairs/caseflow into jefftmarks/APPEALS-24997

… changing task types

* feature/APPEALS-21339: (30 commits)
  updated deserialize method to check on rating decision model for attributes in hash
  Revert "Take setup-node for a spin"
  Take setup-node for a spin
  allow slack notifications in uat
  Update open_hearing_tasks_without_active_descendants_checker_spec.rb
  HunJerBAH APPEALS-26881: Reject Unknown Attributes in RatingDecisions Hash (#19072)
  hid mst pact special issues banner behind feature toggles
  updated back button routing and special issues showing on decisions for ama appeals
  routed judge/attorney decisions to special issues pending on mst pact toggle
  added toggle for mst on special issues list
  hid blue water and burn pit behind mst/pact toggle
  added 24 hr caching and commented out toggle for bgs call
  adding logging for BGS contention calls
  updated feature toggles to include current user
  added caching to BGS contention call
  hid pact and mst check behind toggles
  added feature toggle guard clause for bgs calls for ratings
  optimized participant contentions method to return contentions to cut down on BGS calls
  APPEALS-17497: MST & PACT Special issues identification
  added 24 hr caching and commented out toggle for bgs call
  ...

* Got the modal to allow the form to be submitted without a decision_date for vha benifit type

* Got optional label to appear next to the Decision date label for vha benifit type

* cleaned up some lint

* Cleaned up the logic of the submit button

* Updated the storebook file for the modal to inculde an example for the vha benefit type

* Got first new test working

* Added a new condition to a test for non vha benefit type to check that the optional label is not visible

* added some testing for the submit button logic

* extracted shared logic to describe block

* Cleaned up the repeated code and logic

* fixed the code climate issue of repeated code

* made the first recommended change

* Made the rest of the recommended changes

---------

Co-authored-by: Alec Spottswood <ajspottswood@gmail.com>

…19063)

* Remove rerouting for VHA issues w/o decision dates

Prior to this change there would be a reroute when trying to edit a
VHA issue that did not have a decision date.

We no longer want to reroute when a VHA issue has a missing decision
date.

* Add NoDecisionBanner to IssueList component

From the jira Epic:
Value Statement: As a VHA Intake user, I need the ability to bypass
the required Decision Date entry for a HLR/SC if the information is
not provided, so that I may proceed with the intake of the claim.

As a result of allowing the user to save issues with no decision date,
we want to alert the user to the outcome of saving without a decision
date.

* Extract issueSectionRow to its own file

In an attempt to make the codebase more maintainable and testable we
are extracting this logic that can go in its own file.

The purpose of this is to better follow Separations of Concerns
principles. The `issueSectionRow` logic is distinct enough that it
justifies being in its own section.

Now when working on this section in the future a developer can focus
on just `issueSectionRow.jsx` instead of the whole `addIssues` file.

The main reason I did this however was to more easily test this
functionality in storybook.

* Create `issueSectionRow` stories

The stories are testing:
- The basic issueSectionRow functionality
- issueSectionRow when the issue has no decision date

* Move `addIssues.jsx` into the `addIssues` folder

There were several new files created related to `addIssues` and it
felt like time to make a new directory to house these files.

One thing that could be done in the future is to use more absolute
paths instead of relative paths so that in the future when files and
folders get moved around we don't have to edit file paths.

* Add messageStyling to` IssueList` and `Alert`

We want to override the existing css of the `Alert` component's
message body. To do this we are adding a new prop called
`messageStyling` which will contain any additional css.

This is used in the `IssueList` component for the NoDecisionDate
banner. We want the font size and color to match the mockups provided
to us.

I don't love using the `!important` tag here but I think for now it is
fine and can be potentially improved in the future.

* Minor lint fix

* Add margin to No Decision Date Banner

Adjusted the spacing on the no decision date banner so it does not
cover up the dividing line between issues.

* Add tests for NoDecisionDateBanner in `IssueList`

---------

Co-authored-by: Alec Spottswood <ajspottswood@gmail.com>

…-of-veterans-affairs/caseflow into jefftmarks/APPEALS-24997

* feature/APPEALS-21339:
  exclude db/scripts in codeclimate (#19103)
  Craig/test yarn cache (#19096)
  test update to yarn dependency cache keys
  add exception handling to audit remove script

…-of-veterans-affairs/caseflow into jefftmarks/APPEALS-24997

…view Queue (#19048)

* Initial commit with the incomplete tab for the vha business line.

* Updated the issue type count query logic to work for the new incomplete tab. Added the new appeal unique id alias i.e. the external id to the select statements. Added tabname to the TaskTableTab state in order to account for different behavior for different types of tabs.

* Removed some debugging code.

* Added new constant for the incomplete tab description. Updated a lot of instances of BusinessLine.find_by(url: vha) to use the new VhaBusinessLine.singleton method to hopefully avoid module/class preloading issues.

* Refactored business line querybuilder to work with subtypes of businessline. Added a None tag to the list of filter options if there is not a issue category present on a request issue and fixed the method of counting to work with null issue category column values. Refactored union query joins into an array to reduce code complexity.

* Updated decision review task serializer spec and it's inherited specs for the new fields.

* Refactored the task_filter_details to only include the filter counts for the tabs that the businessline can query for.

* Updated the decision reviews controller in progress method for
 generic non comp buisiness line org to verify that it also includes on hold tasks. Added new tests for the incompe vha businessline for the incomplete tasks method and tab. Also added an in progress tasks test for vha since it should no longer include tasks that have the on_hold status.

* Added tests for generic non comp business line to the business line spec. Also added incomplete_tasks to the vha businessline spec tests.

* Fixed a typo in the controller for the in progress issue types key in the filtering hash.

* Updated reviews spec test and added a test for a non vha business line to make sure on_hold tasks still appear in the in progress tab and that the incomplete tab is not present.

* Storybook file updates for Noncomp.

* Updated the NonCompTabs jest test file.

* Updated NonCompTabs jest test for both vha and generic businesslines.

* Removed unused import.

* Added more expect statements to the reviews spec feature test to check the incomplete tab description and a check to make sure that it is using the correct filter information from the backend for each tab.

* Removed some commented out code. Removed some todos. Fixed some code climate issues.

* Fixed a lot of tests that were failing because the swap to VhaBusinessLine instead of using BusinessLine.create or similar factory bot methods in test setup. Fixed a bug where the completed tab index is now 2 instead of 1 for vha since there are 3 tabs now. It still guesses that it is the last tab or 1. Added a couple of todos related to possibly overriding methods in business line to make sure VhaBusinessLine is created correctly. Added businessline config to show.html.erb for determining the tab and preventing a possible error.

* More testing changes for VhaBusinessLine.

* Made some small updates to the veterans_health_administration seed file to attepmt to fix test setup failures and some other small code climate fixes.

* Changed VhaBusinessLine singleton a to more closely resemble what happens in the has_business_line concern.

* Updated sanitized json file so that the VhaBusinessLine has the correct type on it.

* Altered the wrong json block to the VhaBusinessLine type and fixed it.

* Changed sanitize to sanitize_sql and updated a jest test to use the renamed constant.

* More spec test updates to deal with the VhaBusinessLine subclass and decision review task serializer changes.

* Removed import and todo.

* Another attempt at fixing the brakeman warning.

* Yet more attempts to fix brakeman warning.

* Brakeman really doesn't like string interpolation.

* Fixed a bug where the assigned_to field was set to business_line_id instead of parent in the business line model class.

* Fixed a rare css bug where the user dropdown menu could be blocked by the search bar div in the decision review queue.

* Rewrote some of the issue type count to use arel to avoid brakeman warnings.

* Refactored some repeated code into the issue count helper method.

* Removed a couple of todo statements. Rewrote the issue type count method to use active record instead of an SQL string block.

* Removed another todo block.

* Added a VhaBusinessLine migration.

* Made a couple of small code refactors and moved the require_dependency to the bottom of the business_line.rb file instead of in the decision_reviews_controller.

* Removed commented out code in user.rb model.

…fairs/caseflow into feature/APPEALS-15373

…is action' copy under same variable name across caseflow

…/APPEALS-24996

msteele/APPEALS-24996

…w to 'Provide instructions and context'

…fairs/caseflow into feature/APPEALS-15373

…changing to itself from UI

…-affairs/caseflow into min/APPEALS-25000

* 'min/APPEALS-25000' of github.com:department-of-veterans-affairs/caseflow:
  Revert "APPEALS-17497: MST & PACT Special issues identification"

…-affairs/caseflow into min/APPEALS-25000

* 'min/APPEALS-25000' of github.com:department-of-veterans-affairs/caseflow:
  test adding assertion (#19127)
  APPEALS-24996 Updated color to use existing variable name
  unskip test (#19076)

…ent-of-veterans-affairs/caseflow into jefftmarks/APPEALS-24997

* Appeals-26061: Auto tagging feature flag (#19029)

* Set Rspec mocks for AWS Comprehend Data

---------

Co-authored-by: youfoundmanesh <129548081+youfoundmanesh@users.noreply.github.com>

…ot set

Co-authored-by: Matthew Thornton <99351305+ThorntonMatthew@users.noreply.github.com>

…/APPEALS-25001

msteele96/APPEALS-25001

…EALS-25000

Min/Appeals-25000

…condary to usa-button:

…ent-of-veterans-affairs/caseflow into jefftmarks/APPEALS-24997

…controller

…rks/APPEALS-24997

jefftmarks/APPEALS-24997

* initial commit: fixing POA Refresh button

* fixing css

* removing unnecessary code and adding hasPOA

* fixing few code climate issues

* fixing specs those failed in GHA

* fixing code climate issue

* Spec fixing for POA refresh

* removing commented code

* fixing code climate

* code climate fix for duplicate code

* reverting

* changing code climate issues

* fixing codeclimate

* fixing code climate

* fixing display of POA if error

* increasing threshold and adding seeds file in exclue pattern

* reverting old change and adding excluded patterns

* Revert "changing code climate issues"

This reverts commit 1681179.

* code climate fixes

* Adding code to controller concern

* removing unused code

* removing linter issues

…ppeal

* update info text for POA component for VHA

* update unrecognized poa text for vha

* address linter style issues

* add forgotten period

---------

Co-authored-by: Craig Reese <109101548+craigrva@users.noreply.github.com>