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.

Sign up for GitHub

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

Jump to bottom

10417 remove get current user #5181

Merged
merged 530 commits into from
Aug 28, 2024
Merged

10417 remove get current user #5181

merged 530 commits into from
Aug 28, 2024

Conversation

zachrog
Copy link
Contributor

@zachrog zachrog commented Jul 23, 2024 •
edited by jimlerza
Loading

In this PR

  • Removal of .getCurrentUser() off of applicationContext. The current user is passed into all functions.
  • Creation of a new AuthUser entity.

Manual Deploy Steps

  • Run account specific (Added permission to ci/cd to allow cognito add custom attributes)
  • Run script to give petitioner role to all cognito users without role(We expect this to take ~2hrs)
    • Run environment switcher
    • npx ts-node --transpile-only scripts/run-once-scripts/add-petioner-role-to-roleless-user.ts
      • ustc-dev
      • ustc-exp1
      • ustc-irs
      • ustc-stg
      • ustc-test
  • Deploy as normal
  • Manually add all required attributes to all users in the IRS user pool. (custom:userId, email, name, custom:role)
    • ustc-dev
    • ustc-exp1
    • ustc-irs
    • ustc-stg
    • ustc-test

En-8 and others added 30 commits July 17, 2024 15:38
@En-8
10417 remove getCurrentUser from formattedTrialSessions
294a2fb
@En-8
10417 remove getCurrentUser from workQueueHelper
34aea21
@En-8
10417 Remove getCurrentUser from trialSessionsSummaryHelper
ac46bba
@jtdevos
10417: making authorizedUser parameter in updateCaseAndAssociations o…
74deb2a 
…ptional until refactor complete
@En-8
10417 remove getCurrentUser from trialSessionHeaderHelper
9595b75
@En-8
Merge branch '10417-remove-get-current-user' of github.com:flexion/ef…
8a7fca3 
…-cms into 10417-remove-get-current-user
@En-8
10417 remove getCurrentUser from startCaseHelper
df91633
@En-8
10417 remove getCurrentUser from scanHelper
2b29f8e
@En-8
10417 remove getCurrentUser from partiesInformationHelper
02ef906
@En-8
10417 remove getCurrentUser from orderTypesHelper
8a6ae21
@En-8
10417 remove getCurrentUser from messagesHelper
5ffb651
@En-8
10417 remove getCurrentUser from internalPetitionPartiesHelper
9e4ed96
@jtdevos
10417: remove appcontext.getCurrentUser from updateOtherStatisticsInt…
856e907 
…eractor
@jtdevos
Merge branch '10417-remove-get-current-user' of https://github.com/fl…
47d349d 
…exion/ef-cms into 10417-remove-get-current-user
@jtdevos
10417: remove appcontext.getCurrentUser from updateCaseWorksheetInter…
aee9cf0 
…actor
@jtdevos
10417: remove appcontext.getCurrentUser from archiveCorrespondenceDoc…
e94ce30 
…umentInteractor
@jtdevos
10417: remove appcontext.getCurrentUser from fileCorrespondenceDocume…
3cee473 
…ntInteractor
@jtdevos
10417: remove appcontext.getCurrentUser from updateCorrespondenceDocu…
4125fb2 
…mentInteractor
@jtdevos
10417: remove appcontext.getCurrentUser from serveCourtIssuedDocument…
4022f3a 
…Interactor
@jtdevos
10417: remove appcontext.getCurrentUser from createCourtIssuedOrderPd…
e3404ba 
…fFromHtmlInteractor
@jtdevos
10417: remove appcontext.getCurrentUser from updateCourtIssuedOrderIn…
0e52da6 
…teractor
@JayFlexy
10417 rm getCurrentUser from headerHelper
6d9ef86
@jtdevos
10417: remove appcontext.getCurrentUser from createCaseInteractor
c5fa1a4
@jtdevos
Merge branch '10417-remove-get-current-user' of https://github.com/fl…
481f943 
…exion/ef-cms into 10417-remove-get-current-user
@JayFlexy
10417 rm getCurrentUser from formattedMessageDetail and setUserPermis…
e6cc645 
…sionsAction
@JayFlexy
10417 add user to cerbralTest
b9bad96
@jtdevos
10417: remove appcontext.getCurrentUser from createCsvCustomCaseRepor…
3f0af35 
…tFileInteractor
@JayFlexy
10417 fix tests for gotoDashboardSequence
d54b228
@JayFlexy
10417 fix gotoStartCaseWizardSequence
a5d150f
@JayFlexy
10417 fix gotoTrialSessionWorkingCopySequence
1814491
En-8 and others added 15 commits August 7, 2024 13:22
@En-8
10417 fix IRS superuser test
af8bf8b
@En-8
10417 - Add script to give all users a role of petitioner if they do …
83cab2a 
…not have a role
10417: Add script to verify confirming to a user
2c59f91
10417: Remove uneeded script
27ec65a
Merge branch 'staging' of github.com:ustaxcourt/ef-cms into 10417-rem…
f245d7f 
…ove-get-current-user
@En-8
Revert "devex - Remove unused oauth flows from cognito"
8bb4dcc 
This reverts commit f16b6d4.
@En-8
Revert "devex - Replace outdated terraform module"
b25966c 
This reverts commit 3367652.
10417: Remove defaulting of user to petitioner. Rely on role to come …
cad9212 
…in the identity token
@zachrog
Merge branch 'staging' into 10417-remove-get-current-user
8df48a1
@En-8
10417 add ability to update name in cognito
fb9fb7a
@En-8
Merge branch 'staging' into 10417-remove-get-current-user
27edcfc
@zachrog
Merge branch 'staging' into 10417-remove-get-current-user
44f2b40
Merge branch 'staging' of github.com:ustaxcourt/ef-cms into 10417-rem…
2d93daa 
…ove-get-current-user
Merge branch 'staging' of github.com:ustaxcourt/ef-cms into 10417-rem…
488430d 
…ove-get-current-user
10417: Update tests
38588ae
@zachrog zachrog marked this pull request as ready for review August 20, 2024 16:17
@jimlerza jimlerza self-requested a review August 23, 2024 12:41
@jimlerza jimlerza merged commit 9bdc64d into ustaxcourt:staging Aug 28, 2024
43 of 44 checks passed
@jimlerza jimlerza deleted the 10417-remove-get-current-user branch August 28, 2024 19:51
@jtdevos jtdevos mentioned this pull request Sep 5, 2024
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JayFlexy JayFlexy JayFlexy approved these changes

@jimlerza jimlerza jimlerza approved these changes

@jtdevos jtdevos jtdevos approved these changes

Assignees
No one assigned
Labels
Needs Account Specific to staging
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@zachrog @jtdevos @jimlerza @JayFlexy @En-8 @TomElliottFlexion