cds-snc / cds-snc/platform-forms-client
Inconsistency in how Attestation is marked "required" in card vs. modal
- Dominant language
- TypeScript
- Stars
- 46
- Forks
- 16
- Avg merge
- 1d 8h
- Merged PRs (30d)
- 85
Description
## Context
Issue found as part of audit by Sam
https://docs.google.com/spreadsheets/d/1mEVvvo5-_IlgpatxPe7fThdyTL-QV3eHcfMzko21p5M/edit?gid=1553464146#gid=1553464146
## Description
Check box is deactivated on the card but available on the modal
## Expected behaviour
Since the component has the functionality of "all required" embedded within it, this should be locked down and not unselectable.
## Screencapture
https://github.com/user-attachments/assets/a9a4197d-3a6d-4335-9c9b-7f2de802f41e
Contributor guide
Research direction
Start by reproducing the Attestation behavior in both the card and modal, then trace the component responsible for the checkbox. Done means the required checkbox is locked and cannot be unselected in both views, matching the expected behavior and screenshot.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- next.js, typescript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 65/100