Skip to content

Conversation

@rismehta
Copy link
Collaborator

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes and the overall coverage did not decrease.
  • All unit tests pass on CircleCi.
  • I ran all tests locally and they pass.

rismehta and others added 7 commits July 25, 2024 11:39
* Fixing incorrect id issue with checkbox

* updating component readme

* updating read me
* adding email test

* Fixing test collateral

* Fixing test collateral

* Fixing email test
* updating eventsource while updating model value

* Adding ui change event to update model value

* Fixing package lock

* Fixing text input code

---------

Co-authored-by: vdua <vdua@adobe.com>
* Fixing exclusive maximum and minimum issue with number input

* Fixing exclusive maximum and minimum issue with date picker

* Adding test cases as per the customer use-case

* updating package lock

* Fixing test case

* Fixing test case
* Approval checkbox required by default

* FORMS-15481 Adding ui change event to update model value (#1327)

* updating eventsource while updating model value

* Adding ui change event to update model value

* Fixing package lock

* Fixing text input code

---------

Co-authored-by: vdua <vdua@adobe.com>

---------

Co-authored-by: Pavitra Khatri <pavitrakhatri@Pavitras-MacBook-Pro.local>
Co-authored-by: Rishi Mehta <69448117+rismehta@users.noreply.github.com>
Co-authored-by: vdua <vdua@adobe.com>
@adobe-bot
Copy link

Lighthouse scores (desktop)

Performance Accessibility Best-Practices SEO
Scores 100 96 96 75

@adobe-bot
Copy link

Lighthouse scores (mobile)

Performance Accessibility Best-Practices SEO
Scores 91 96 96 75

@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
target-size serious

2 similar comments
@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
target-size serious

@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
target-size serious

@codecov
Copy link

codecov bot commented Jul 25, 2024

Codecov Report

Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 81.71%. Comparing base (6bb9ff0) to head (d137730).

Files Patch % Lines
.../cq/forms/core/components/util/ComponentUtils.java 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1334      +/-   ##
============================================
- Coverage     81.73%   81.71%   -0.03%     
- Complexity      853      855       +2     
============================================
  Files            97       97              
  Lines          2267     2269       +2     
  Branches        305      306       +1     
============================================
+ Hits           1853     1854       +1     
  Misses          255      255              
- Partials        159      160       +1     

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

@rismehta rismehta merged commit 2141537 into master Jul 25, 2024
@rismehta rismehta deleted the dev_to_master_25july branch July 25, 2024 06:53
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.

4 participants