Skip to content

Conversation

@lisbet-alvarez
Copy link
Collaborator

@lisbet-alvarez lisbet-alvarez commented Nov 4, 2025

Description

Add ignores for existing violations that were uncovered by this PR which causes a different auth method & account type to be chosen from default hence displaying a different form each time (which have violations).

Checklist

  • I have added before and after screenshots for UI changes
  • I have added JSON response output for API changes
  • I have added steps to reproduce and test for bug fixes in the description
  • I have commented on my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have added a11y-tests label to run a11y audit tests if needed

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.
  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.
  • If applicable, I've documented the impact of any changes to security controls.
    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

@vercel
Copy link

vercel bot commented Nov 4, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
boundary-ui Ready Ready Preview Comment Nov 6, 2025 0:30am
boundary-ui-desktop Ready Ready Preview Comment Nov 6, 2025 0:30am

@lisbet-alvarez lisbet-alvarez merged commit 6fca907 into main Nov 6, 2025
16 checks passed
@lisbet-alvarez lisbet-alvarez deleted the ignore-new-ember-a11y-failures branch November 6, 2025 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y-tests Runs our a11y tests when label added to PR admin ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants