Skip to content

JCC - update provider ids in transaction history table when SSN is corrected #1845

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

Merged
isabeleliassen merged 27 commits into from
Aug 21, 2026
Merged

JCC - update provider ids in transaction history table when SSN is corrected #1845

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
556dfaa
Initial implementation of transaction record corrections
landonshumway-ia Aug 19, 2026
01e64ce
Add logs/simplify update logic
landonshumway-ia Aug 19, 2026
96ef150
simplify logic to collect transaction ids from previous snapshots
landonshumway-ia Aug 19, 2026
26757b8
refactor to extract common test logic into test data generator
landonshumway-ia Aug 19, 2026
d4cfb08
add comment about migration failure and retry case
landonshumway-ia Aug 19, 2026
e9e8ac2
Add transaction licensee id check for SSN correction smoke tests
landonshumway-ia Aug 19, 2026
0224424
Ensure system owned fields are preserved with every license re-upload
landonshumway-ia Aug 19, 2026
19f89d2
Ensure encumbrance status is preserved on the provider record
landonshumway-ia Aug 20, 2026
972459e
update nanoid version for node check
landonshumway-ia Aug 14, 2026
db6689d
Updating existing top level record with encumbered and military statu…
landonshumway-ia Aug 20, 2026
ddca8ca
Add guard for field additions to provider schema
landonshumway-ia Aug 20, 2026
b478936
refactor
landonshumway-ia Aug 20, 2026
51e4866
Enhance ssn migration smoke tests to check for top level provider fields
landonshumway-ia Aug 20, 2026
737dc20
Feedback - improve test assertions
landonshumway-ia Aug 20, 2026
b8f9df2
Feedback - add condition expression for update and improve smoke test…
landonshumway-ia Aug 20, 2026
11e6703
Pass investigationStatus forward on privilege renewal
landonshumway-ia Aug 20, 2026
47b9685
feedback - define provider fields not supplied by the schema
landonshumway-ia Aug 20, 2026
9a4417c
Feedback - rename field and clarify comments
landonshumway-ia Aug 21, 2026
3541490
feedback - fix comment blocks and test setup
landonshumway-ia Aug 21, 2026
e362e4d
PR feedback - clarify comments and tests
landonshumway-ia Aug 21, 2026
a5a8354
Remove duplicate definition
landonshumway-ia Aug 21, 2026
bbf7ff7
Update pip version in GH action runners
landonshumway-ia Aug 21, 2026
67070d4
Add logs to denote partial and full migrations for easier cloudwatch …
landonshumway-ia Aug 21, 2026
eaff514
Update multi-account deps
landonshumway-ia Aug 21, 2026
0e80bf6
preserve log markers when context marker keys in multiple callers match
landonshumway-ia Aug 21, 2026
d0aef79
Apply marker fix to other compacts
landonshumway-ia Aug 21, 2026
12b42ad
Add explicit log markers to migration logs
landonshumway-ia Aug 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .github/workflows/check-common-cdk.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@ jobs:
- uses: actions/checkout@v5

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
run: pip install --upgrade 'pip>=26.1'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'
Comment thread
coderabbitai[bot] marked this conversation as resolved.

- name: Install dev dependencies
run: "pip install -r backend/common-cdk/requirements-dev.in"
Expand All @@ -50,8 +50,8 @@ jobs:
python-version: '3.14'

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
run: pip install --upgrade 'pip>=26.1'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

- name: Install dev dependencies
run: "pip install -r backend/common-cdk/requirements-dev.in"
Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/check-compact-connect-ui-app.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@ jobs:
- uses: actions/checkout@v5

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
run: pip install --upgrade 'pip>=26.1'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

- name: Install dev dependencies
run: "pip install -r backend/compact-connect-ui-app/requirements-dev.txt"
Expand Down Expand Up @@ -87,8 +87,8 @@ jobs:
python-version: '3.14'

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
run: pip install --upgrade 'pip>=26.1'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

# Setup Node
- name: Setup Node
Expand Down
15 changes: 6 additions & 9 deletions .github/workflows/check-compact-connect.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,8 @@ jobs:
- uses: actions/checkout@v5

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
# Held below 26.2, which drops pip._internal.utils.compat.stdlib_pkgs and breaks pip-sync.
run: pip install --upgrade 'pip>=26.1,<26.2'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

- name: Install dev dependencies
run: "pip install -r backend/compact-connect/requirements-dev.txt"
Expand Down Expand Up @@ -88,9 +87,8 @@ jobs:
python-version: '3.14'

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
# Held below 26.2, which drops pip._internal.utils.compat.stdlib_pkgs and breaks pip-sync.
run: pip install --upgrade 'pip>=26.1,<26.2'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

# Setup Node
- name: Setup Node
Expand Down Expand Up @@ -131,9 +129,8 @@ jobs:
python-version: '3.12'

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
# Held below 26.2, which drops pip._internal.utils.compat.stdlib_pkgs and breaks pip-sync.
run: pip install --upgrade 'pip>=26.1,<26.2'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

- name: Install dependencies
run: "cd backend/compact-connect/lambdas/python/purchases; pip install -r requirements.txt"
Expand Down
10 changes: 4 additions & 6 deletions .github/workflows/check-cosmetology-app.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,8 @@ jobs:
- uses: actions/checkout@v5

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
# Held below 26.2, which drops pip._internal.utils.compat.stdlib_pkgs and breaks pip-sync.
run: pip install --upgrade 'pip>=26.1,<26.2'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

- name: Install dev dependencies
run: "pip install -r backend/cosmetology-app/requirements-dev.txt"
Expand Down Expand Up @@ -87,9 +86,8 @@ jobs:
python-version: '3.14'

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
# Held below 26.2, which drops pip._internal.utils.compat.stdlib_pkgs and breaks pip-sync.
run: pip install --upgrade 'pip>=26.1,<26.2'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

# Setup Node
- name: Setup Node
Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/check-multi-account.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@ jobs:
- uses: actions/checkout@v2

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
run: pip install --upgrade 'pip>=26.1'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

- name: Install dev dependencies
run: "pip install -r backend/multi-account/control-tower/requirements-dev.txt"
Expand All @@ -50,8 +50,8 @@ jobs:
python-version: '3.12'

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
run: pip install --upgrade 'pip>=26.1'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

- name: Install dev dependencies
run: "pip install -r backend/multi-account/control-tower/requirements-dev.txt"
Expand Down
10 changes: 4 additions & 6 deletions .github/workflows/check-social-work-app.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,8 @@ jobs:
- uses: actions/checkout@v5

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
# Held below 26.2, which drops pip._internal.utils.compat.stdlib_pkgs and breaks pip-sync.
run: pip install --upgrade 'pip>=26.1,<26.2'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

- name: Install dev dependencies
run: "pip install -r backend/social-work-app/requirements-dev.txt"
Expand Down Expand Up @@ -87,9 +86,8 @@ jobs:
python-version: '3.14'

- name: Upgrade pip
# Runner image ships pip 25.3; Upgrade to 26.1+ to include fix for CVE-2026-3219.
# Held below 26.2, which drops pip._internal.utils.compat.stdlib_pkgs and breaks pip-sync.
run: pip install --upgrade 'pip>=26.1,<26.2'
# Runner image ships pip 25.3; Upgrade to 26.2+ for the fixes to CVE-2026-3219 and PYSEC-2026-3721.
run: pip install --upgrade 'pip>=26.2'

# Setup Node
- name: Setup Node
Expand Down
2 changes: 1 addition & 1 deletion backend/compact-connect-ui-app/requirements-dev.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ pip-audit==2.10.1
# via -r requirements-dev.in
pip-requirements-parser==32.0.1
# via pip-audit
pip-tools==7.6.0
pip-tools==7.6.1
# via -r requirements-dev.in
platformdirs==4.10.0
# via pip-audit
Expand Down
6 changes: 3 additions & 3 deletions backend/compact-connect/lambdas/nodejs/yarn.lock
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4674,9 +4674,9 @@ ms@^2.1.1, ms@^2.1.3:
integrity sha512-6FlzubTLZG3J2a/NVCAleEhjzq5oxgHyaCU9yYXvcLsvoVaHJq/s5xXI6/XXP6tz7R9xAOtHnSO/tXtF3WRTlA==

nanoid@^3.3.16:
version "3.3.16"
resolved "https://registry.yarnpkg.com/nanoid/-/nanoid-3.3.16.tgz#a04d8ec4b1f10009d2d533947aefe4293737816c"
integrity sha512-bzlKTyNJ7+LdGIIwy8ijFpIqEQIvafahV7eYykJ8Cvh42EdJeODoJ6gUJXpQJvej1BddH8OqTXZNE/KfbWAu8Q==
version "3.3.18"
resolved "https://registry.yarnpkg.com/nanoid/-/nanoid-3.3.18.tgz#f66a2de1199ffde0fcf21c8a5f13106b1c081913"
integrity sha512-DTg4MJbGMWkfi6VZFdNt2/caMbQy4Ou+Op/hJQvGEWcnVfoA1QA+xzRKAzw9jD6+GVOOeYr/mIcuDSdug6F6+w==

napi-postinstall@^0.3.4:
version "0.3.4"
Expand Down
4 changes: 4 additions & 0 deletions backend/compact-connect/lambdas/python/common/cc_common/config.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -304,6 +304,10 @@ def transaction_history_table_name(self):
def transaction_history_table(self):
return boto3.resource('dynamodb').Table(self.transaction_history_table_name)

@property
def transaction_history_transaction_id_gsi_name(self):
return os.environ['TRANSACTION_HISTORY_TRANSACTION_ID_GSI_NAME']

@property
def rate_limiting_table_name(self):
return os.environ['RATE_LIMITING_TABLE_NAME']
Expand Down
Loading
Loading