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

Revert password field selection using shortcut #4451

Merged
merged 1 commit into from Feb 20, 2018
Merged

Revert password field selection using shortcut #4451

merged 1 commit into from Feb 20, 2018

Conversation

andreasstieger
Copy link
Contributor

@andreasstieger andreasstieger commented Feb 20, 2018

Rebase of original #4448 (with passing ci)

Change introduced in
PR#4437
caused more new problems, and was trying to fix problem that happened
just once. Somehow, even keypresses are correct, we do not type password
correctly. So I remove change to select password field and use tab key
instead. I was not able to reproduce exactly same issue, when password
is typed only once.

Verification runs (from #4448)
http://g226.suse.de/tests/829
http://g226.suse.de/tests/828

Change introduced in
[PR#4437](#4437)
caused more new problems, and was trying to fix problem that happened
just once. Somehow, even keypresses are correct, we do not type password
correctly. So I remove change to select password field and use tab key
instead. I was not able to reproduce exactly same issue, when password
is typed only once.
@DimStar77
Copy link
Contributor

Would love to see this fixed, as it currently blocks allso stagings (plus prod test)

@okurz okurz merged commit 681923a into os-autoinst:master Feb 20, 2018
@andreasstieger andreasstieger deleted the user_settings branch February 20, 2018 19:24
@okurz
Copy link
Member

okurz commented Feb 20, 2018

merged as requested by @andreasstieger and @DimStar77 but I suggest @rwx788 to still take a look how to proceed.

I retriggered the staging tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants