Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Remove default modifier for formfield label #3055

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

mannycarrera4
Copy link
Contributor

Summary

Remove default modifier for formfield label. False should be handled by the stencil.

Release Category

Components


Checklist

For the Reviewer

  • PR title is short and descriptive
  • PR summary describes the change (Fixes/Resolves linked correctly)
  • PR Release Notes describes additional information useful to call out in a release message or removed if not applicable
  • Breaking Changes provides useful information to upgrade to this code or removed if not applicable

Where Should the Reviewer Start?

Areas for Feedback? (optional)

  • Code
  • Documentation
  • Testing
  • Codemods

Testing Manually

Screenshots or GIFs (if applicable)

Thank You Gif (optional)

Copy link

cypress bot commented Nov 19, 2024

Workday/canvas-kit    Run #8058

Run Properties:  status check passed Passed #8058  •  git commit aad32d2135 ℹ️: Merge 337fbc3ae46a2f8aa458bbf51fb9542a1ad050ae into cc00a3c53acab8337147f2e305b0...
Project Workday/canvas-kit
Branch Review mc-fix-formfield-issue
Run status status check passed Passed #8058
Run duration 03m 48s
Commit git commit aad32d2135 ℹ️: Merge 337fbc3ae46a2f8aa458bbf51fb9542a1ad050ae into cc00a3c53acab8337147f2e305b0...
Committer Manuel Carrera
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 29
Tests that did not run due to a developer annotating a test with .skip  Pending 24
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 1093
View all changes introduced in this branch ↗︎
UI Coverage  21.8%
  Untested elements 1637  
  Tested elements 454  
Accessibility  99.18%
  Failed rules  5 critical   5 serious   0 moderate   2 minor
  Failed elements 180  

@alanbsmith alanbsmith merged commit 7ca693b into Workday:master Nov 19, 2024
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants