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.

Sign up for GitHub

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

Jump to bottom

Feature/af password component #1407

Open
wants to merge 31 commits into
base: dev
Choose a base branch
from
Open

Feature/af password component #1407

wants to merge 31 commits into from

Conversation

kum-sushil
Copy link

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.

kum-sushil and others added 30 commits February 23, 2024 15:26
@kum-sushil
Initial commit for the AF password box core component
f7fba95
@kum-sushil
AF Password Component| Updated license year and few code comments blo…
eeb34ac 
…ck update
@kum-sushil
AF Password Field| Updated Javascript code, updated apps pom.xml.
670b2a6 
Updated Javascript code for toggle password visibility. Html code cleanups
@kum-sushil
AF Password|Test Cases and html updates supporting responsiveness
a5e10c0
@kum-sushil
AF Password| Test case updates
0c16826
@kum-sushil
AF Password | PR review comments Updates
903601f
@kum-sushil
AF Password| Cypress test cases for editor and runtime JS
d33e8ec
@kum-sushil
AF-Password|PR Comments| Removed Constants from FormConstants to dire…
ed2567d 
…ct values on view
@kum-sushil
Merge branch 'dev' into feature/AF-Password-Component
6bbb81b
@kum-sushil
AF-password||Resolved test cases and password java for exclusive max …
9d1e6e2 
…and min values update
@devgurjar
Merge branch 'dev' into feature/AF-Password-Component
d29456f
@rismehta
Fixing flaky test (adobe#1312)
df83dc8 
* Fixing flaky test

* updating quickstart docker

* Fixing flaky test
@kum-sushil
AF Password Component| Updated license year and few code comments blo…
c2854ae 
…ck update
@kum-sushil
AF Password Field| Updated Javascript code, updated apps pom.xml.
642c630 
Updated Javascript code for toggle password visibility. Html code cleanups
@kum-sushil
AF Password|Test Cases and html updates supporting responsiveness
ba32ad0
@kum-sushil
AF Password| Test case updates
f3741ed
@kum-sushil
AF Password | PR review comments Updates
eac2e41
@kum-sushil
AF-Password|PR Comments| Removed Constants from FormConstants to dire…
e64a6eb 
…ct values on view
@kum-sushil
AF-password||Resolved test cases and password java for exclusive max …
4cd5d24 
…and min values update
@rismehta
Fixing flaky test (adobe#1312)
bff5e52 
* Fixing flaky test

* updating quickstart docker

* Fixing flaky test
@kum-sushil
AF Password Component| Updated license year and few code comments blo…
6204e39 
…ck update
@kum-sushil
AF Password Field| Updated Javascript code, updated apps pom.xml.
fdd90f3 
Updated Javascript code for toggle password visibility. Html code cleanups
@kum-sushil
AF Password|Test Cases and html updates supporting responsiveness
62b3c5e
@kum-sushil
AF Password| Test case updates
2f206f2
@kum-sushil
AF Password | PR review comments Updates
1d52343
@kum-sushil
AF-Password|PR Comments| Removed Constants from FormConstants to dire…
58319b9 
…ct values on view
@kum-sushil
AF-password||Resolved test cases and password java for exclusive max …
42c6f77 
…and min values update
@kum-sushil
Merge branch 'password-comp' into feature/AF-Password-Component
b7dcf21
@kum-sushil
Validation pattern fix
1286d24
@kum-sushil
Merge branch 'feature/AF-Password-Component' of https://github.com/ku…
f1ffc1e 
…m-sushil/aem-core-forms-components into feature/AF-Password-Component
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kum-sushil @devgurjar @rismehta