Skip to content

[CL-1130] Fix storybook a11y and console errors for dirt files#19919

Open
vleague2 wants to merge 8 commits intomainfrom
uif/cl-1130/dirt
Open

[CL-1130] Fix storybook a11y and console errors for dirt files#19919
vleague2 wants to merge 8 commits intomainfrom
uif/cl-1130/dirt

Conversation

@vleague2
Copy link
Copy Markdown
Contributor

@vleague2 vleague2 commented Apr 1, 2026

🎟️ Tracking

CL-1130

📔 Objective

We are working through Storybook errors related to accessibility and general console errors (usually story setup issues). This PR fixes current errors in dirt-owned code. Specifically the fixes in this PR are:

I think it might be good for the team's designer to take a look at the UI Review prior to approving this PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

Logo
Checkmarx One – Scan Summary & Details2cdc0651-bd13-406c-8d1a-a4155604af3d


New Issues (1) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 MEDIUM Missing_HSTS_Header apps/cli/src/auth/commands/login.command.ts: 571
detailsThe web-application does not define an HSTS header, leaving it vulnerable to attack.
Attack Vector

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.05%. Comparing base (8320244) to head (942117c).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #19919   +/-   ##
=======================================
  Coverage   47.05%   47.05%           
=======================================
  Files        3894     3894           
  Lines      117403   117403           
  Branches    17975    17975           
=======================================
  Hits        55245    55245           
  Misses      59671    59671           
  Partials     2487     2487           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vleague2 vleague2 marked this pull request as ready for review April 2, 2026 13:33
@vleague2 vleague2 requested a review from a team as a code owner April 2, 2026 13:33
@sonarqubecloud
Copy link
Copy Markdown

"message": "Your bank account needs to be verified before it can be used for this upgrade. Please verify your bank account or choose a different payment method."
},
"export": {
"message": "Export"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❓ Do we still need to add the message if the export text was removed?

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.

2 participants