Skip to content
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

Correspondence Intake Processor cleanup #21796

Merged

Conversation

Rnmarshall93
Copy link
Contributor

Resolves Correspondence Intake Processor cleanup

Description

This change refactors a few different models so that they're cleaner and inherit correctly.

  • associated_with_claims_folder_mail_task.rb now inherits from CorrespondenceMailTask
  • sent_to_roj_mail_task.rb now inherits from CorrespondenceMailTask
  • vacols_updated_mail_task.rb now inherits from CorrespondenceMalTask
  • In CorrespondenceIntakeProcessor, the tasks Returned or undeliverable mail, Sent to ROJ have been removed as they are deprecated.
  • In CorrespondenceIntakeProcessor, the methods class_for_data and task_class_for_type have been refactored to task_class_for_task_related and task_class_for_task_unrelated additionally, two new methods have been added to replace their functionality. task_class_for_task_related and task_class_for_task_unrelated
  • In CorrespondenceIntakeProcessor, the large list of tasks has been refactored into two distinct lists. TASKS_NOT_RELATED_TO_APPEAL_TASK_TYPES and TASKS_RELATED_TO_APPEAL_TASK_TYPES

Acceptance Criteria

  • Code compiles correctly

HunJerBAH
HunJerBAH previously approved these changes Jun 4, 2024
@HunJerBAH HunJerBAH merged commit 15cf10e into feature/APPEALS-36284-S4 Jun 4, 2024
11 of 17 checks passed
@HunJerBAH HunJerBAH deleted the ryan/APPEALS-44425-new_feature_fix branch June 4, 2024 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants