Skip to content

backend/update jurisprudence attestation text - #2060

Open
ChiefStief wants to merge 5 commits into
csg-org:mainfrom
InspiringApps:feat/jurisprudence-attest-tweak
Open

ChiefStief wants to merge 5 commits into
csg-org:mainfrom
InspiringApps:feat/jurisprudence-attest-tweak

Conversation

@ChiefStief

@ChiefStief ChiefStief commented Oct 29, 2026 •

Copy link
Copy Markdown
Collaborator

Requirements List

Description List

  • Update the shared jurisprudence-confirmation pop-up text in attestations.yml. Counseling, OT, and ASLP all use this file. On deploy, the uploader writes a new version and keeps the previous one, so each privilege still records the wording that licensee accepted.

Testing List

  • yarn test:unit:all should run without errors or warnings
  • yarn serve should run without errors or warnings
  • yarn build should run without errors or warnings
  • For API configuration changes: CDK tests added/updated in backend/compact-connect/tests/unit/test_api.py
  • For API endpoint changes: OpenAPI spec updated to show latest endpoint configuration run compact-connect/bin/download_oas30.py
  • Code review

Closes #2050

Summary by CodeRabbit

  • Updates
    • Clarified the attestation language: applicants confirm they have reviewed each relevant state’s requirements and have completed or will complete them within the required timeframe. The text also notes that failure to meet these requirements may result in revocation and disciplinary action.
    • Updated the audit statement to clarify that both the Commission and states may audit attestations.

@coderabbitai

coderabbitai Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 0b41c317-9a0a-4f70-aac5-17fbb3b1bef2
Commits

Reviewing files that changed from the base of the PR and between 559da5f and 1b9b13a.

Files ignored due to path filters: 1
  • backend/dietitian-app/lambdas/nodejs/yarn.lock is excluded by !**/yarn.lock, !**/*.lock
Files selected for processing: 22
  • backend/compact-connect-ui-app/requirements-dev.txt
  • backend/compact-connect/compact-config/attestations.yml
  • backend/compact-connect/lambdas/python/purchases/requirements-dev.txt
  • backend/compact-connect/requirements-dev.txt
  • backend/cosmetology-app/requirements-dev.txt
  • backend/dietitian-app/lambdas/nodejs/package.json
  • backend/dietitian-app/lambdas/python/cognito-backup/requirements-dev.txt
  • backend/dietitian-app/lambdas/python/common/requirements-dev.txt
  • backend/dietitian-app/lambdas/python/common/requirements.txt
  • backend/dietitian-app/lambdas/python/compact-configuration/requirements-dev.txt
  • backend/dietitian-app/lambdas/python/custom-resources/requirements-dev.txt
  • backend/dietitian-app/lambdas/python/data-events/requirements-dev.txt
  • backend/dietitian-app/lambdas/python/disaster-recovery/requirements-dev.txt
  • backend/dietitian-app/lambdas/python/provider-data-v1/requirements-dev.txt
  • backend/dietitian-app/lambdas/python/purchases/requirements-dev.txt
  • backend/dietitian-app/lambdas/python/search/requirements-dev.txt
  • backend/dietitian-app/lambdas/python/search/requirements.txt
  • backend/dietitian-app/lambdas/python/staff-user-pre-token/requirements-dev.txt
  • backend/dietitian-app/lambdas/python/staff-users/requirements-dev.txt
  • backend/dietitian-app/requirements-dev.txt
  • backend/multi-account/control-tower/requirements-dev.txt
  • backend/social-work-app/requirements-dev.txt

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The pull request revises backend jurisprudence attestation wording and updates pinned development and runtime dependency versions across backend projects.

Changes: Jurisprudence attestation

Layer / File(s) Summary
Update attestation wording
backend/compact-connect/compact-config/attestations.yml
The text addresses state-specific jurisprudence requirements, completion deadlines, possible revocation and disciplinary action, and audits by states and the Commission.

Dependency updates

Layer / File(s) Summary
Update dependency pins
backend/compact-connect-ui-app/requirements-dev.txt, backend/compact-connect/..., backend/cosmetology-app/requirements-dev.txt, backend/dietitian-app/..., backend/multi-account/control-tower/requirements-dev.txt, backend/social-work-app/requirements-dev.txt
Python pins change for msgpack from 1.2.1 to 1.2.3, certifi from 2026.6.17 to 2026.10.19, and filelock from 3.29.4 to 4.2.4. The Node.js package configuration changes sanitize-html from 2.17.7 to 2.18.1, nodemailer from ^10.1.1 to ^10.1.4, and zod from ^3.23.8 to ^4.6.6.

Priority: Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Possibly related issues

  • csg-org/CompactConnect#2050 — The issue concerns replacement backend jurisprudence attestation text, and this change updates that text.
  • csg-org/CompactConnect#2052 — This change updates backend attestation text, but does not include the matching frontend checkbox change described by the issue.

Suggested reviewers: dylanmrowe

Merge Risk: Minimal · up to 1b9b1

The wording and dependency updates have no identified merge-blocking issue. Run the listed checks before merging.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title clearly identifies the main substantive change: updating the jurisprudence attestation text.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Description check Passed The description follows the required template and explains the attestation text update. It includes the testing checklist and issue reference. The Requirements List is empty, and the description does …
Full details: Description check

Explanation

The description follows the required template and explains the attestation text update. It includes the testing checklist and issue reference. The Requirements List is empty, and the description does not report test results, but these omissions are non-critical.

Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ChiefStief ChiefStief changed the title update attestation text backend/update jurisprudence attestation text Oct 29, 2026
@ChiefStief

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@ChiefStief
ChiefStief requested a review from jlkravitz October 29, 2026 19:57
@ChiefStief

Copy link
Copy Markdown
Collaborator Author

@jlkravitz FYI Isabel was hoping to get this and its front end companion in today if possible. Its ready for a look

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change Jurisprudence Attestation Backend

1 participant