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

Add autocomplete=off to login form fields, password fields #4107

Merged
merged 4 commits into from
Aug 10, 2023

Conversation

himdel
Copy link
Collaborator

@himdel himdel commented Aug 8, 2023

Fixes AAH-2625

This PR:

20230808112236

(docs)

@github-actions github-actions bot added the backport-4.7 This PR should be backported to stable-4.7 (2.4) label Aug 8, 2023
@himdel himdel marked this pull request as ready for review August 8, 2023 11:49
@himdel
Copy link
Collaborator Author

himdel commented Aug 10, 2023

tested in firefox, chrome

@himdel himdel merged commit 6dc75bd into ansible:master Aug 10, 2023
17 of 19 checks passed
@himdel himdel deleted the autocomplete-off branch August 10, 2023 23:58
@patchback
Copy link

patchback bot commented Aug 10, 2023

Backport to stable-4.7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4.7/6dc75bd2fafb4a53a61ef3e1aa0cf86d291f4816/pr-4107

Backported as #4109

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Aug 10, 2023
* TextInput - add autoComplete=off to all password fields

No-Issue

* add and use a local copy of patternfly LoginForm

* LoginForm - add autoComplete=off

Issue: AAH-2625

* eslint - remove explicit any

(cherry picked from commit 6dc75bd)
himdel added a commit that referenced this pull request Aug 11, 2023
…4109)

* TextInput - add autoComplete=off to all password fields

No-Issue

* add and use a local copy of patternfly LoginForm

* LoginForm - add autoComplete=off

Issue: AAH-2625

* eslint - remove explicit any

(cherry picked from commit 6dc75bd)

Co-authored-by: Martin Hradil <[email protected]>
@github-actions github-actions bot added the backported-4.7 This PR has been backported to stable-4.7 (2.4) label Aug 11, 2023
@himdel himdel added the backport-4.6 This PR should be backported to stable-4.6 (2.3) label Aug 31, 2023
@patchback

This comment was marked as outdated.

@himdel himdel added the backport-4.5 This PR should be backported to stable-4.5 (2.2) label Aug 31, 2023
@patchback

This comment was marked as outdated.

himdel added a commit to himdel/ansible-hub-ui that referenced this pull request Aug 31, 2023
)

* TextInput - add autoComplete=off to all password fields

No-Issue

* add and use a local copy of patternfly LoginForm

* LoginForm - add autoComplete=off

Issue: AAH-2625

* eslint - remove explicit any

(cherry picked from commit 6dc75bd)
himdel added a commit to himdel/ansible-hub-ui that referenced this pull request Aug 31, 2023
)

* TextInput - add autoComplete=off to all password fields

No-Issue

* add and use a local copy of patternfly LoginForm

* LoginForm - add autoComplete=off

Issue: AAH-2625

* eslint - remove explicit any

(cherry picked from commit 6dc75bd)
himdel added a commit that referenced this pull request Aug 31, 2023
…4160)

* TextInput - add autoComplete=off to all password fields

No-Issue

* add and use a local copy of patternfly LoginForm

* LoginForm - add autoComplete=off

Issue: AAH-2625

* eslint - remove explicit any

(cherry picked from commit 6dc75bd)
@github-actions github-actions bot added the backported-4.6 This PR has been backported to stable-4.6 (2.3) label Aug 31, 2023
himdel added a commit that referenced this pull request Aug 31, 2023
…4161)

* TextInput - add autoComplete=off to all password fields

No-Issue

* add and use a local copy of patternfly LoginForm

* LoginForm - add autoComplete=off

Issue: AAH-2625

* eslint - remove explicit any

(cherry picked from commit 6dc75bd)
@github-actions github-actions bot added the backported-4.5 This PR has been backported to stable-4.5 (2.2) label Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-4.5 This PR should be backported to stable-4.5 (2.2) backport-4.6 This PR should be backported to stable-4.6 (2.3) backport-4.7 This PR should be backported to stable-4.7 (2.4) backported-4.5 This PR has been backported to stable-4.5 (2.2) backported-4.6 This PR has been backported to stable-4.6 (2.3) backported-4.7 This PR has been backported to stable-4.7 (2.4)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant