-
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
Should we create a new appeal stream for motions records in Caseflow? #9993
Comments
cc: @laurjpeterson Our convo earlier. |
I think we should just continue to add tasks to the same appeal in Caseflow. That is, do not create a new record (which I think could mean having a new docket number, etc. or defining a whole new type of record), but add tasks (e.g. Motion for Reconsideration) to the Root task for that appeal. This would be a change, because I think right now we close RootTasks when appeals are dispatched from the Board. (If that's the case, I wonder if we can keep them open, and use just the fact that the BVADispatchTask is complete as an indication that the appeal is complete so we can keep the RootTask open to deal with scenarios like this.) @cmgiven - do you see any pros to Motions being their own type of record in Caseflow that I don't see? |
The VACOLS approach to motions makes it difficult to understand what post-decisional motions have been filed against a particular appeal. The advantage in a VACOLS context is that it allows you to record a second decision on the reconsideration, but we potentially have a lot more flexibility in how we record decisions. I could certainly make an argument for tracking CAVC appeals separately, as it seems analogous to Supplemental Claim/Higher-Level Review/Board Appeal. We would then track CAVC issues using the same RequestIssues/DecisionIssues framework we use for everything else. Arguably a post-decisional motion should be considered the same way. And unlike VACOLS, we're set up to handle the linkage of the motion to the appeal by nature of how we do issue linking. |
@cmgiven to clarify
Are you saying that post-decisional motions should be considered the same way [as tracking CAVC appeals separately]? So they should be their own record instead of tasks added to the previous appeal? |
There are pros and cons both ways. But the argument for making it its own record is that the motion has the potential to have its own set of DecisionIssues, something that becomes more complex if it’s just a task on the original appeal. |
Discussion 3/20:
|
This discussion ended up getting moved back to the original ticket, see the 04/03/2019 decision here. |
Closing this ticket. Decisions on this are being made as part of work in #8547 . |
Bumps [eslint](https://github.com/eslint/eslint) from 4.9.0 to 4.18.2. <details> <summary>Release notes</summary> *Sourced from [eslint's releases](https://github.com/eslint/eslint/releases).* > ## v4.18.2 > * 6b71fd0 Fix: [email protected], because 4.0.3 needs "ajv": "^6.0.1" ([#10022](https://github-redirect.dependabot.com/eslint/eslint/issues/10022)) (Mathieu Seiler) > * 3c697de Chore: fix incorrect comment about linter.verify return value ([#10030](https://github-redirect.dependabot.com/eslint/eslint/issues/10030)) (Teddy Katz) > * 9df8653 Chore: refactor parser-loading out of linter.verify ([#10028](https://github-redirect.dependabot.com/eslint/eslint/issues/10028)) (Teddy Katz) > * f6901d0 Fix: remove catastrophic backtracking vulnerability (fixes [#10002](https://github-redirect.dependabot.com/eslint/eslint/issues/10002)) ([#10019](https://github-redirect.dependabot.com/eslint/eslint/issues/10019)) (Jamie Davis) > * e4f52ce Chore: Simplify dataflow in linter.verify ([#10020](https://github-redirect.dependabot.com/eslint/eslint/issues/10020)) (Teddy Katz) > * 33177cd Chore: make library files non-executable ([#10021](https://github-redirect.dependabot.com/eslint/eslint/issues/10021)) (Teddy Katz) > * 558ccba Chore: refactor directive comment processing ([#10007](https://github-redirect.dependabot.com/eslint/eslint/issues/10007)) (Teddy Katz) > * 18e15d9 Chore: avoid useless catch clauses that just rethrow errors ([#10010](https://github-redirect.dependabot.com/eslint/eslint/issues/10010)) (Teddy Katz) > * a1c3759 Chore: refactor populating configs with defaults in linter ([#10006](https://github-redirect.dependabot.com/eslint/eslint/issues/10006)) (Teddy Katz) > * aea07dc Fix: Make max-len ignoreStrings ignore JSXText (fixes [#9954](https://github-redirect.dependabot.com/eslint/eslint/issues/9954)) ([#9985](https://github-redirect.dependabot.com/eslint/eslint/issues/9985)) (Rachael Sim) > > ## v4.18.1 > * f417506 Fix: ensure no-await-in-loop reports the correct node (fixes [#9992](https://github-redirect.dependabot.com/eslint/eslint/issues/9992)) ([#9993](https://github-redirect.dependabot.com/eslint/eslint/issues/9993)) (Teddy Katz) > * 3e99363 Docs: Fixed typo in key-spacing rule doc ([#9987](https://github-redirect.dependabot.com/eslint/eslint/issues/9987)) (Jaid) > * 7c2cd70 Docs: deprecate experimentalObjectRestSpread ([#9986](https://github-redirect.dependabot.com/eslint/eslint/issues/9986)) (Toru Nagashima) > > ## v4.18.0 > * 70f22f3 Chore: Apply memoization to config creation within glob utils ([#9944](https://github-redirect.dependabot.com/eslint/eslint/issues/9944)) (Kenton Jacobsen) > * 0e4ae22 Update: fix indent bug with binary operators/ignoredNodes (fixes [#9882](https://github-redirect.dependabot.com/eslint/eslint/issues/9882)) ([#9951](https://github-redirect.dependabot.com/eslint/eslint/issues/9951)) (Teddy Katz) > * 47ac478 Update: add named imports and exports for object-curly-newline ([#9876](https://github-redirect.dependabot.com/eslint/eslint/issues/9876)) (Nicholas Chua) > * e8efdd0 Fix: support Rest/Spread Properties (fixes [#9885](https://github-redirect.dependabot.com/eslint/eslint/issues/9885)) ([#9943](https://github-redirect.dependabot.com/eslint/eslint/issues/9943)) (Toru Nagashima) > * f012b8c Fix: support Async iteration (fixes [#9891](https://github-redirect.dependabot.com/eslint/eslint/issues/9891)) ([#9957](https://github-redirect.dependabot.com/eslint/eslint/issues/9957)) (Toru Nagashima) > * 74fa253 Docs: Clarify no-mixed-operators options (fixes [#9962](https://github-redirect.dependabot.com/eslint/eslint/issues/9962)) ([#9964](https://github-redirect.dependabot.com/eslint/eslint/issues/9964)) (Ivan Hayes) > * 426868f Docs: clean up key-spacing docs (fixes [#9900](https://github-redirect.dependabot.com/eslint/eslint/issues/9900)) ([#9963](https://github-redirect.dependabot.com/eslint/eslint/issues/9963)) (Abid Uzair) > * 4a6f22e Update: support eslint-disable-* block comments (fixes [#8781](https://github-redirect.dependabot.com/eslint/eslint/issues/8781)) ([#9745](https://github-redirect.dependabot.com/eslint/eslint/issues/9745)) (Erin) > * 777283b Docs: Propose fix typo for function ([#9965](https://github-redirect.dependabot.com/eslint/eslint/issues/9965)) (John Eismeier) > * bf3d494 Docs: Fix typo in max-len ignorePattern example. ([#9956](https://github-redirect.dependabot.com/eslint/eslint/issues/9956)) (Tim Martin) > * d64fbb4 Docs: fix typo in prefer-destructuring.md example ([#9930](https://github-redirect.dependabot.com/eslint/eslint/issues/9930)) (Vse Mozhet Byt) > * f8d343f Chore: Fix default issue template ([#9946](https://github-redirect.dependabot.com/eslint/eslint/issues/9946)) (Kai Cataldo) > > ## v4.17.0 > * 1da1ada Update: Add "multiline" type to padding-line-between-statements ([#8668](https://github-redirect.dependabot.com/eslint/eslint/issues/8668)) (Matthew Bennett) > * bb213dc Chore: Use messageIds in some of the core rules ([#9648](https://github-redirect.dependabot.com/eslint/eslint/issues/9648)) (Jed Fox) > * 1aa1970 Docs: remove outdated rule naming convention ([#9925](https://github-redirect.dependabot.com/eslint/eslint/issues/9925)) (Teddy Katz) > * 3afaff6 Docs: Add prefer-destructuring variable reassignment example ([#9873](https://github-redirect.dependabot.com/eslint/eslint/issues/9873)) (LePirlouit) > * d20f6b4 Fix: Typo in error message when running npm ([#9866](https://github-redirect.dependabot.com/eslint/eslint/issues/9866)) (Maciej Kasprzyk) > * 51ec6a7 Docs: Use GitHub Multiple PR/Issue templates ([#9911](https://github-redirect.dependabot.com/eslint/eslint/issues/9911)) (Kai Cataldo) > * dc80487 Update: space-unary-ops uses astUtils.canTokensBeAdjacent (fixes [#9907](https://github-redirect.dependabot.com/eslint/eslint/issues/9907)) ([#9906](https://github-redirect.dependabot.com/eslint/eslint/issues/9906)) (Kevin Partington) > * 084351b Docs: Fix the messageId example (fixes [#9889](https://github-redirect.dependabot.com/eslint/eslint/issues/9889)) ([#9892](https://github-redirect.dependabot.com/eslint/eslint/issues/9892)) (Jed Fox) > * 9cbb487 Docs: Mention the `globals` key in the no-undef docs ([#9867](https://github-redirect.dependabot.com/eslint/eslint/issues/9867)) (Dan Dascalescu) > > ## v4.16.0 > * e26a25f Update: allow continue instead of if wrap in guard-for-in (fixes [#7567](https://github-redirect.dependabot.com/eslint/eslint/issues/7567)) ([#9796](https://github-redirect.dependabot.com/eslint/eslint/issues/9796)) (Michael Ficarra) > * af043eb Update: Add NewExpression support to comma-style ([#9591](https://github-redirect.dependabot.com/eslint/eslint/issues/9591)) (Frazer McLean) > * 4f898c7 Build: Fix JSDoc syntax errors ([#9813](https://github-redirect.dependabot.com/eslint/eslint/issues/9813)) (Matija Marohnić) > * 13bcf3c Fix: Removing curly quotes in no-eq-null report message ([#9852](https://github-redirect.dependabot.com/eslint/eslint/issues/9852)) (Kevin Partington) > * b96fb31 Docs: configuration hierarchy for CLIEngine options (fixes [#9526](https://github-redirect.dependabot.com/eslint/eslint/issues/9526)) ([#9855](https://github-redirect.dependabot.com/eslint/eslint/issues/9855)) (PiIsFour) > * 8ccbdda Docs: Clarify that -c configs merge with `.eslintrc.*` (fixes [#9535](https://github-redirect.dependabot.com/eslint/eslint/issues/9535)) ([#9847](https://github-redirect.dependabot.com/eslint/eslint/issues/9847)) (Kevin Partington) > * 978574f Docs: Fix examples for no-useless-escape ([#9853](https://github-redirect.dependabot.com/eslint/eslint/issues/9853)) (Toru Kobayashi) ></tr></table> ... (truncated) </details> <details> <summary>Changelog</summary> *Sourced from [eslint's changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md).* > v4.18.2 - March 2, 2018 > > * 6b71fd0 Fix: [email protected], because 4.0.3 needs "ajv": "^6.0.1" ([#10022](https://github-redirect.dependabot.com/eslint/eslint/issues/10022)) (Mathieu Seiler) > * 3c697de Chore: fix incorrect comment about linter.verify return value ([#10030](https://github-redirect.dependabot.com/eslint/eslint/issues/10030)) (Teddy Katz) > * 9df8653 Chore: refactor parser-loading out of linter.verify ([#10028](https://github-redirect.dependabot.com/eslint/eslint/issues/10028)) (Teddy Katz) > * f6901d0 Fix: remove catastrophic backtracking vulnerability (fixes [#10002](https://github-redirect.dependabot.com/eslint/eslint/issues/10002)) ([#10019](https://github-redirect.dependabot.com/eslint/eslint/issues/10019)) (Jamie Davis) > * e4f52ce Chore: Simplify dataflow in linter.verify ([#10020](https://github-redirect.dependabot.com/eslint/eslint/issues/10020)) (Teddy Katz) > * 33177cd Chore: make library files non-executable ([#10021](https://github-redirect.dependabot.com/eslint/eslint/issues/10021)) (Teddy Katz) > * 558ccba Chore: refactor directive comment processing ([#10007](https://github-redirect.dependabot.com/eslint/eslint/issues/10007)) (Teddy Katz) > * 18e15d9 Chore: avoid useless catch clauses that just rethrow errors ([#10010](https://github-redirect.dependabot.com/eslint/eslint/issues/10010)) (Teddy Katz) > * a1c3759 Chore: refactor populating configs with defaults in linter ([#10006](https://github-redirect.dependabot.com/eslint/eslint/issues/10006)) (Teddy Katz) > * aea07dc Fix: Make max-len ignoreStrings ignore JSXText (fixes [#9954](https://github-redirect.dependabot.com/eslint/eslint/issues/9954)) ([#9985](https://github-redirect.dependabot.com/eslint/eslint/issues/9985)) (Rachael Sim) > > v4.18.1 - February 20, 2018 > > * f417506 Fix: ensure no-await-in-loop reports the correct node (fixes [#9992](https://github-redirect.dependabot.com/eslint/eslint/issues/9992)) ([#9993](https://github-redirect.dependabot.com/eslint/eslint/issues/9993)) (Teddy Katz) > * 3e99363 Docs: Fixed typo in key-spacing rule doc ([#9987](https://github-redirect.dependabot.com/eslint/eslint/issues/9987)) (Jaid) > * 7c2cd70 Docs: deprecate experimentalObjectRestSpread ([#9986](https://github-redirect.dependabot.com/eslint/eslint/issues/9986)) (Toru Nagashima) > > v4.18.0 - February 16, 2018 > > * 70f22f3 Chore: Apply memoization to config creation within glob utils ([#9944](https://github-redirect.dependabot.com/eslint/eslint/issues/9944)) (Kenton Jacobsen) > * 0e4ae22 Update: fix indent bug with binary operators/ignoredNodes (fixes [#9882](https://github-redirect.dependabot.com/eslint/eslint/issues/9882)) ([#9951](https://github-redirect.dependabot.com/eslint/eslint/issues/9951)) (Teddy Katz) > * 47ac478 Update: add named imports and exports for object-curly-newline ([#9876](https://github-redirect.dependabot.com/eslint/eslint/issues/9876)) (Nicholas Chua) > * e8efdd0 Fix: support Rest/Spread Properties (fixes [#9885](https://github-redirect.dependabot.com/eslint/eslint/issues/9885)) ([#9943](https://github-redirect.dependabot.com/eslint/eslint/issues/9943)) (Toru Nagashima) > * f012b8c Fix: support Async iteration (fixes [#9891](https://github-redirect.dependabot.com/eslint/eslint/issues/9891)) ([#9957](https://github-redirect.dependabot.com/eslint/eslint/issues/9957)) (Toru Nagashima) > * 74fa253 Docs: Clarify no-mixed-operators options (fixes [#9962](https://github-redirect.dependabot.com/eslint/eslint/issues/9962)) ([#9964](https://github-redirect.dependabot.com/eslint/eslint/issues/9964)) (Ivan Hayes) > * 426868f Docs: clean up key-spacing docs (fixes [#9900](https://github-redirect.dependabot.com/eslint/eslint/issues/9900)) ([#9963](https://github-redirect.dependabot.com/eslint/eslint/issues/9963)) (Abid Uzair) > * 4a6f22e Update: support eslint-disable-* block comments (fixes [#8781](https://github-redirect.dependabot.com/eslint/eslint/issues/8781)) ([#9745](https://github-redirect.dependabot.com/eslint/eslint/issues/9745)) (Erin) > * 777283b Docs: Propose fix typo for function ([#9965](https://github-redirect.dependabot.com/eslint/eslint/issues/9965)) (John Eismeier) > * bf3d494 Docs: Fix typo in max-len ignorePattern example. ([#9956](https://github-redirect.dependabot.com/eslint/eslint/issues/9956)) (Tim Martin) > * d64fbb4 Docs: fix typo in prefer-destructuring.md example ([#9930](https://github-redirect.dependabot.com/eslint/eslint/issues/9930)) (Vse Mozhet Byt) > * f8d343f Chore: Fix default issue template ([#9946](https://github-redirect.dependabot.com/eslint/eslint/issues/9946)) (Kai Cataldo) > > v4.17.0 - February 2, 2018 > > * 1da1ada Update: Add "multiline" type to padding-line-between-statements ([#8668](https://github-redirect.dependabot.com/eslint/eslint/issues/8668)) (Matthew Bennett) > * bb213dc Chore: Use messageIds in some of the core rules ([#9648](https://github-redirect.dependabot.com/eslint/eslint/issues/9648)) (Jed Fox) > * 1aa1970 Docs: remove outdated rule naming convention ([#9925](https://github-redirect.dependabot.com/eslint/eslint/issues/9925)) (Teddy Katz) > * 3afaff6 Docs: Add prefer-destructuring variable reassignment example ([#9873](https://github-redirect.dependabot.com/eslint/eslint/issues/9873)) (LePirlouit) > * d20f6b4 Fix: Typo in error message when running npm ([#9866](https://github-redirect.dependabot.com/eslint/eslint/issues/9866)) (Maciej Kasprzyk) > * 51ec6a7 Docs: Use GitHub Multiple PR/Issue templates ([#9911](https://github-redirect.dependabot.com/eslint/eslint/issues/9911)) (Kai Cataldo) > * dc80487 Update: space-unary-ops uses astUtils.canTokensBeAdjacent (fixes [#9907](https://github-redirect.dependabot.com/eslint/eslint/issues/9907)) ([#9906](https://github-redirect.dependabot.com/eslint/eslint/issues/9906)) (Kevin Partington) > * 084351b Docs: Fix the messageId example (fixes [#9889](https://github-redirect.dependabot.com/eslint/eslint/issues/9889)) ([#9892](https://github-redirect.dependabot.com/eslint/eslint/issues/9892)) (Jed Fox) > * 9cbb487 Docs: Mention the `globals` key in the no-undef docs ([#9867](https://github-redirect.dependabot.com/eslint/eslint/issues/9867)) (Dan Dascalescu) > > v4.16.0 - January 19, 2018 > > * e26a25f Update: allow continue instead of if wrap in guard-for-in (fixes [#7567](https://github-redirect.dependabot.com/eslint/eslint/issues/7567)) ([#9796](https://github-redirect.dependabot.com/eslint/eslint/issues/9796)) (Michael Ficarra) > * af043eb Update: Add NewExpression support to comma-style ([#9591](https://github-redirect.dependabot.com/eslint/eslint/issues/9591)) (Frazer McLean) ></tr></table> ... (truncated) </details> <details> <summary>Commits</summary> - [`22ff6f3`](eslint/eslint@22ff6f3) 4.18.2 - [`817b84b`](eslint/eslint@817b84b) Build: changelog update for 4.18.2 - [`6b71fd0`](eslint/eslint@6b71fd0) Fix: [email protected], because 4.0.3 needs "ajv": "^6.0.1" ([#10022](https://github-redirect.dependabot.com/eslint/eslint/issues/10022)) - [`3c697de`](eslint/eslint@3c697de) Chore: fix incorrect comment about linter.verify return value ([#10030](https://github-redirect.dependabot.com/eslint/eslint/issues/10030)) - [`9df8653`](eslint/eslint@9df8653) Chore: refactor parser-loading out of linter.verify ([#10028](https://github-redirect.dependabot.com/eslint/eslint/issues/10028)) - [`f6901d0`](eslint/eslint@f6901d0) Fix: remove catastrophic backtracking vulnerability (fixes [#10002](https://github-redirect.dependabot.com/eslint/eslint/issues/10002)) ([#10019](https://github-redirect.dependabot.com/eslint/eslint/issues/10019)) - [`e4f52ce`](eslint/eslint@e4f52ce) Chore: Simplify dataflow in linter.verify ([#10020](https://github-redirect.dependabot.com/eslint/eslint/issues/10020)) - [`33177cd`](eslint/eslint@33177cd) Chore: make library files non-executable ([#10021](https://github-redirect.dependabot.com/eslint/eslint/issues/10021)) - [`558ccba`](eslint/eslint@558ccba) Chore: refactor directive comment processing ([#10007](https://github-redirect.dependabot.com/eslint/eslint/issues/10007)) - [`18e15d9`](eslint/eslint@18e15d9) Chore: avoid useless catch clauses that just rethrow errors ([#10010](https://github-redirect.dependabot.com/eslint/eslint/issues/10010)) - Additional commits viewable in [compare view](eslint/eslint@v4.9.0...v4.18.2) </details> <br /> [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=eslint&package-manager=npm_and_yarn&previous-version=4.9.0&new-version=4.18.2)](https://help.github.com/articles/configuring-automated-security-fixes) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot ignore this [patch|minor|major] version` will close this PR and stop Dependabot creating any more for this minor/major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) - `@dependabot use these labels` will set the current labels as the default for future PRs for this repo and language - `@dependabot use these reviewers` will set the current reviewers as the default for future PRs for this repo and language - `@dependabot use these assignees` will set the current assignees as the default for future PRs for this repo and language - `@dependabot use this milestone` will set the current milestone as the default for future PRs for this repo and language You can disable automated security fix PRs for this repo from the [Security Alerts page](https://github.com/department-of-veterans-affairs/caseflow/network/alerts). </details>
Creating a ticket for this open question so that we can track it and so that we can have our engineering team weigh in on technical complexity.
We learned that Lit Support opens a new record for Motions and CAVC cases in VACOLS and they track them separate from Original cases. In Caseflow search results, Nina also mentioned that motions attorneys would like to see a list of cases that are either original, CAVC, or other motions to quickly understand the story of a claim.
Two potential approaches were discussed for tracking post-decisional motions in Caseflow:
MVP proposed approach:
Higher-effort proposed approach:
Open questions:
The text was updated successfully, but these errors were encountered: