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

10417 Remove getCurrentUser minimum necessary changes #10425

Closed
En-8 opened this issue Jul 17, 2024 · 1 comment
Closed

10417 Remove getCurrentUser minimum necessary changes #10425

En-8 opened this issue Jul 17, 2024 · 1 comment
Assignees

Comments

@En-8
Copy link
Collaborator

En-8 commented Jul 17, 2024

This issue represents the minimum necessary work required to set up the pattern for removing applicationContext.getCurrentUser. The intent with splitting out this work is to ease the transition by making the change backwards compatible so that we don't create undue burden for (or just outright break) in-flight work while the broader set of changes is in test but not staging. There is also an example of what it will look like to move away from get current user on the back end.

This is merely a preparatory refactor, it will not solve the bug identified in 10417

While no functional changes are expected, here are the potentially-impacted areas, should something have gone wrong:

  • Custom Case Report
  • Maintenance mode
  • All "public" endpoints
    • searches (case, order, opinion, practitioner)
    • viewing documents
    • Todays Orders & Todays Opinions
@ttlenard
Copy link
Collaborator

ttlenard commented Jul 23, 2024

Let's be sure that we test the following:

  • Terminal user
  • Sealed case/doc visibility
  • policy date documents (decisions, briefs)
  • Opinions
  • Orders
  • Maintenance mode
  • Custom Case Report

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Closed
Development

No branches or pull requests

6 participants