cds-snc / cds-snc/platform-forms-client

Inconsistency in how Attestation is marked "required" in card vs. modal

Open
#7,791 0 comments 0 reactions 0 assignees View on GitHub
:vertical_traffic_light: Minor bug enhancement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.