-
Notifications
You must be signed in to change notification settings - Fork 19
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
Allow SCM team members to move AMA cases #13786
Conversation
@@ -84,6 +93,6 @@ def cancel_parent_judge_review | |||
end | |||
|
|||
def open_judge_assign_task | |||
JudgeAssignTask.create!(appeal: appeal, parent: appeal.root_task, assigned_to: assigned_by) | |||
JudgeAssignTask.create!(appeal: appeal, parent: appeal.root_task, assigned_to: parent.assigned_to) |
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.
Ensures the JudgeAssignTask
does not get assigned to the assigning SCM user if an attorney task is cancelled.
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.
I'm not sure the Judge on the JDRTask is who should always be the assigning judge in the case of cancelling? Pondering this.
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.
How does this affect AVLJ cases?
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.
I'm trying to think of a normal scenario that this would affect, but cannot come up with one.
Judge assigns a case to Attorney. They are both the assigner of the attorney task and the assignee of the attorney tasks parent. The parent cannot be reassigned until the attorney task is completed. If cancelled, it will go back to the Judge either way.
I don't know of a scenario that would affect AVLJ cases either.
Code Climate has analyzed commit 68ef04b and detected 0 issues on this pull request. View more on Code Climate. |
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.
Couple questions and thoughts!
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.
GTG once the test go green! 🎉
Bumps #12424
Description
This PR addresses all of the AMA AC from #12424 except for Request Cases on behalf of the Judge which is addressed by #13727. Allows SCM users to perform all assigning and reassigning judges can on AMA cases.
Acceptance Criteria
Testing Plan
AMA at judge assign
AMA at judge decision review