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

Master65016 oct #1445

Merged
merged 28 commits into from
Oct 17, 2024
Merged

Master65016 oct #1445

merged 28 commits into from
Oct 17, 2024

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 23 commits September 26, 2024 14:24
Co-authored-by: Pavitra Khatri <pavitrakhatri@pavitras-mbp.corp.adobe.com>
* GH actions

* Dummy change

* dummy

* change

* Fix error

* Fix error

* fix error

* Fix error

* fix regex

* Fix code

* dummy change

* Dummy change

* Fix errors
* test pr

* Fixing exporter

* revert changes
…ordion, Wizard, Horizontal and Vertical Tabs (#1329)

* Unable to select the secondary panels individually in Accordion, Tabs on Top, Vertical Tabs

* revert bem class change of vertical tabs

* FORMS-11475 add tests and fix

* FORMS-11475 fix tests

* FORMS-11475 fix test

* Unable to select the secondary panels individually in Accordion, Tabs on Top, Vertical Tabs

* revert bem class change of vertical tabs

* FORMS-11475 add tests and fix

* FORMS-11475 fix tests

* FORMS-11475 fix test

* FORMS-11475 fix test

* Unable to select the secondary panels individually in Accordion, Tabs on Top, Vertical Tabs

* revert bem class change of vertical tabs

* FORMS-15150 update headers

* FORMS-11475 add test

* FORMS-11475 fix site test

---------

Co-authored-by: demaurya <demaurya@adobe.com>
* initial check in

* Adding test collateral

* FORMS-16351 Support for string in file attachment
@review @vdua
DOD(Yes)

* Adding field description for type string

* Adding test collateral

* updating field description

* Adding read attachments

* moving to released version

* @releng updating af-core
* Added new HTML types to field types

* Adding hidden fieldtype

* updated spec version
…date time picker widget (#1432)

* FORMS-16264: Fixed test cases

* FORMS-16264: Remove same instantiation of datetimepicker calendar element

* FORMS-16264: Resolve conflict
Co-authored-by: Pavitra Khatri <pavitrakhatri@pavitras-mbp.corp.adobe.com>
…odel (#1409)

* FORMS-16011 - Binding each core component's fieldType to its respective sling model.

* fixing import

* Fixing test collateral

* fixing import order

* Fixing star imports

* Refactoring getFieldType to super class

* Updating import

* sorting import

* Spec update to 0.14.1 | RTC approved

* updating classpath with schema folder

* fixing paths

* updating schema with buttonType

* incorporating review comments
Co-authored-by: Pavitra Khatri <pavitrakhatri@pavitras-mbp.corp.adobe.com>
* Updating type for buttons

* Updating collateral for button

* Fixing collateral and test

* Fixing code and test

* Fixing test and keeping button disabled

* Adding code comments

* Updating read me and test

* Fixing reset test

---------

Co-authored-by: Muskan Gupta <muskgupta@muskans-mbp.corp.adobe.com>
Co-authored-by: Muskan Gupta <muskgupta@Muskans-MacBook-Pro.local>
Co-authored-by: Rishi Mehta <rismehta@adobe.com>
Copy link

codecov bot commented Oct 16, 2024

Codecov Report

Attention: Patch coverage is 86.84211% with 5 lines in your changes missing coverage. Please review.

Project coverage is 79.11%. Comparing base (d925ab7) to head (1406f10).

Files with missing lines Patch % Lines
...ponents/internal/models/v1/form/FileInputImpl.java 63.63% 1 Missing and 3 partials ⚠️
...ms/core/components/models/form/FileConstraint.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##             release/650    #1445      +/-   ##
=================================================
+ Coverage          79.02%   79.11%   +0.09%     
- Complexity           777      798      +21     
=================================================
  Files                 92       92              
  Lines               1840     1872      +32     
  Branches             272      276       +4     
=================================================
+ Hits                1454     1481      +27     
- Misses               249      252       +3     
- Partials             137      139       +2     
Flag Coverage Δ
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@rismehta
Copy link
Collaborator Author

verified that the test cases are passing on 6.5.21.0 but failing on 6.5.22.0 load 9 build

@rismehta rismehta merged commit 31d35dc into release/650 Oct 17, 2024
5 of 13 checks passed
@rismehta rismehta deleted the master65016Oct branch October 17, 2024 10:56
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.

6 participants