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

Devtomaster 9 may 2024 #1229

Merged
merged 27 commits into from
May 9, 2024
Merged

Devtomaster 9 may 2024 #1229

merged 27 commits into from
May 9, 2024

Conversation

barshat7
Copy link
Contributor

@barshat7 barshat7 commented May 9, 2024

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.

Pavitra Khatri and others added 25 commits April 16, 2024 12:13
Added condition on description and error message in aria-describedby
* FORMS-12203: Added localization changes and test cases

* FORMS-12203: Updated test case

* FORMS-12203: Altered to contains instead of get

* FORMS-12203: Resolved comments

* FORMS-12203: Updated test cases

* FORMS-12203: Updated delete button

* FORMS-12203: Updated button

* Retry wizard.locale.spec.js tests

* FORMS-12203: Added theme request wait

* Update wizard.locale.spec.js

* Added latestaddon check

* Delete ui.tests/test-module/package-lock.json

* Update commands.js

* Revert "Delete ui.tests/test-module/package-lock.json"

This reverts commit fd97b53.

* Reverted package-lock.json changes

* FORMS-12203: Removed exception inline

* FORMS-12203: Resolved comments

* FORMS-12203: Update index.js

* FORMS-12203: Removed package-lock.json change

* Update index.js

* Update error message in index.js

* Update index.js

* Update wizard.locale.spec.js

* Removed an intercept

* Updated regex patterns

---------

Co-authored-by: Shivangi Mittal <shivangim@Shivangis-MacBook-Pro.local>
* FORMS-13851 Support for language in date picker and number input
@review @vdua
DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
    @review @vdua
    DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
        @review @vdua
        DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
            @review @vdua
            DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
                @review @vdua
                DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
                    @review @vdua
                    DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
                        @review @vdua
                        DOD(Yes)

* FORMS-13851 Support for language in date picker and number input
                            @review @vdua
                            DOD(Yes)

* Update space character for thousands separator in french lang

* FORMS-13851 Support for language in date picker and number input
                                @review @vdua
                                DOD(Yes)

---------

Co-authored-by: Varun Dua <vdua@adobe.com>
#1196)

* FORMS-12822 : format should not be visible when no pattern is selected

* FORMS-12822 : changes var to let as per ES6 in utils.js
…d submit() rule in examples (#1191)

* FORMS-13209: added check to override hardcoded submitSuccess and submitError hadnlers

* FORMS-13209: updated af2 runtime version

* FORMS-13209: added rule editor authoring cypress tests

* FORMS-13209: added submission handler rule tests

* FORMS-13209: replaced submit(custom:submitSuccess, custom:submitError) examples and it rules with submit()

* FORMS-13209: updated af-custom-functions version

* FORMS-13209: updated submission handler test

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: added rule editor authoring cypress tests

* FORMS-13209: added submission handler rule tests

* FORMS-13209: updated af-custom-functions version

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: added rule editor authoring cypress tests

* FORMS-13209: added submission handler rule tests

* FORMS-13209: updated af-custom-functions version

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: updated tests

* FORMS-13209: added af-custom-functions in FormView and replaced hardcoded submission handlers with defaultSubmit handlers af-custom-functions

* FORMS-13209: minor update

---------

Co-authored-by: Arun Kumar Attri <aattri@adobe.com>
FORMS-13935 update examples for aemcomponents.dev

FORMS-13935 update examples for aemcomponents.dev
…lue without default/placeholder set (#1203)

* Added test case

* Adding actual fix

* Improved test case

* Small fix
* updated code for ui.frontend with test cases

* resolved PR comments

* test case fixed

---------

Co-authored-by: Gulshan Mishra <gulshanm@adobe.com>
Co-authored-by: Rajat Khurana <rajatkhurana@adobe.com>
Co-authored-by: Pavitra Khatri <pavitrakhatri@pavitras-mbp.corp.adobe.com>
Co-authored-by: Pavitra Khatri <pavitrakhatri@Pavitras-MacBook-Pro.local>
…vided (#1210)

* FORMS-14018 - Fix for dropdown not working when enumNames are not provided

* Import fix

* Improved code and test cov

* Revert "Improved code and test cov"

This reverts commit 7a3d69e.

* Refactor for code cov

* Added the removed test case
* Moving to 15860 image

* updating core component version

* Skipping wizard authoring test
* embedding forms-sdk api

* adding hcaptcha and recaptcha models

* :fix hcaptcha model SlingException

* :fix deleting recaptcha model

* FORMS-13811 | Adding rendition for hCaptcha

* FORMS-13811 | Renamed public API

* FORMS-13811 | Embedding hcaptcha js

* FORMS-13811 | Adding unit test for sling model

* FORMS-13811 | Adding cypress test

* FORMS-13811 | Adressing review comments

* Adding e2e test

* FORMS-13811 | fix import statements

* Update pom.xml

* Adding FT for running Tests

* FORMS-13811 | Removing java-8 build for master

* Adding FT in test

* FORMS-13811 | Tests should run only with latest addon

* FORMS-13811 | Tests should run only with latest addon

* increasing coverage

* addressing review comments

---------

Co-authored-by: Navneet Agarwal <navneeta@adobe.com>
* [FORMS-11581] Enabled Draft functionality for V2

* [FORMS-11581] Fixed test case for draft feature

* [FORMS-11581] Fixed test case for draft feature

* [FORMS-11581] Fixed test case for draft feature

* [FORMS-11581] Fixed test case for draft feature

* [FORMS-11581] Fixed test case for draft feature

* [FORMS-11581] Added more test case for draft feature

* [FORMS-11581] Fixed test case for draft feature

* [FORMS-11581] Fixed test case for draft feature

* [FORMS-11581] Fixed test case for draft feature

* [FORMS-11581] Fixed test case for draft feature
…sset path in headless json (#1198)

* [eds adding source property in sling model of image component to get the path in crx of the image.......]

* [eds adding source property in sling model of image component to get the path in crx of the image.......]

* [FORMS-13952] changed the dam repo path property to fd:repoPath

* [FORMS-13952] changed the dam repo path property to fd:repoPath

* fixed some test cases ........

* FORMS-13952 removed unused api of getSource...

* [FORMS-13952] formatter change added in StaticImage......

---------

Co-authored-by: Deep Prakash Dewanji <deepprakashdewanji@labusers-mbp-5.macromedia.com>
Co-authored-by: Deep Prakash Dewanji <deepprakashdewanji@labusers-mbp-9.corp.adobe.com>
* added changes to publish semantic html of form from container[FormContainerEds].....

* added test case to verify formDefinition api....

* added test case to verify formDefinition api....

* updated the license header of body file...

* added changes to publish semantic html of form from container[FormContainerEds].....

* added test case to verify formDefinition api....

* added test case to verify formDefinition api....

* updated the license header of body file...

* created a new version of page component.....

* added a Utils file to move certain logic out of FormContainer sling model

* made changes to remove style elements from container proxy...

* made changes to remove style elements from container proxy...

* removed body html from v1 of page..

* added README for page comp and bumped up minor version of the package containing form sling models....

* added java doc for the isEdgeDeliveryRequest api...

* added test cases for v2 version of page component....

* added test cases for v2 version of page component....

* made suggested review changes.......

---------

Co-authored-by: Deep Prakash Dewanji <deepprakashdewanji@labusers-mbp-5.macromedia.com>
Co-authored-by: Deep Prakash Dewanji <deepprakashdewanji@labusers-mbp-5.corp.adobe.com>
Co-authored-by: Deep Prakash Dewanji <deepprakashdewanji@labusers-mbp-9.corp.adobe.com>
Co-authored-by: Deep Prakash Dewanji <deepprakashdewanji@labusers-MacBook-Pro-9.local>
add data-cmp-valid attribute for checkboxgroup and radiobutton

add data-cmp-valid attribute for checkboxgroup and radiobutton

add data-cmp-valid attribute for checkboxgroup and radiobutton

add data-cmp-valid attribute for checkboxgroup and radiobutton
Exclude from DoR is not working for text field
@adobe-bot
Copy link

Lighthouse scores (desktop)

Performance Accessibility Best-Practices SEO
Scores 100 96 100 75

@adobe-bot
Copy link

Lighthouse scores (mobile)

Performance Accessibility Best-Practices SEO
Scores 94 96 100 75

@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
landmark-one-main moderate
region moderate
target-size serious

2 similar comments
@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
landmark-one-main moderate
region moderate
target-size serious

@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
landmark-one-main moderate
region moderate
target-size serious

Copy link

codecov bot commented May 9, 2024

Codecov Report

Attention: Patch coverage is 78.57143% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 81.26%. Comparing base (2465443) to head (3381636).

Files Patch % Lines
...rms/core/components/internal/form/HtlUtilImpl.java 40.00% 1 Missing and 2 partials ⚠️
...ponents/internal/form/FormStructureParserImpl.java 81.81% 2 Missing ⚠️
...nents/internal/models/v1/form/StaticImageImpl.java 75.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1229      +/-   ##
============================================
+ Coverage     80.89%   81.26%   +0.36%     
- Complexity      791      806      +15     
============================================
  Files            92       93       +1     
  Lines          2146     2167      +21     
  Branches        292      294       +2     
============================================
+ Hits           1736     1761      +25     
+ Misses          253      252       -1     
+ Partials        157      154       -3     

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

@adobe-bot
Copy link

Lighthouse scores (desktop)

Performance Accessibility Best-Practices SEO
Scores 100 96 100 75

@adobe-bot
Copy link

Lighthouse scores (mobile)

Performance Accessibility Best-Practices SEO
Scores 94 96 100 75

@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
landmark-one-main moderate
region moderate
target-size serious

2 similar comments
@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
landmark-one-main moderate
region moderate
target-size serious

@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
landmark-one-main moderate
region moderate
target-size serious

@adobe-bot
Copy link

Lighthouse scores (desktop)

Performance Accessibility Best-Practices SEO
Scores 100 96 100 75

@adobe-bot
Copy link

Lighthouse scores (mobile)

Performance Accessibility Best-Practices SEO
Scores 94 96 100 75

@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
landmark-one-main moderate
region moderate
target-size serious

2 similar comments
@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
landmark-one-main moderate
region moderate
target-size serious

@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
landmark-one-main moderate
region moderate
target-size serious

@barshat7 barshat7 merged commit 22e8c75 into master May 9, 2024
8 of 10 checks passed
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.

None yet