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

Fix login password form #18619

Merged
merged 1 commit into from
Nov 13, 2023
Merged

Fix login password form #18619

merged 1 commit into from
Nov 13, 2023

Conversation

karwosts
Copy link
Contributor

Proposed change

Broke from #18607

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@karwosts karwosts mentioned this pull request Nov 10, 2023
9 tasks
@piitaya
Copy link
Member

piitaya commented Nov 13, 2023

Thanks for the fix.
I think we should have a proper way to use ha-form without hass. Some other form elements have the same issue but we don't use them in onboarding so we don't have the issue yet.

@piitaya piitaya merged commit bff09a7 into home-assistant:dev Nov 13, 2023
13 checks passed
@karwosts karwosts deleted the fix-login branch November 13, 2023 18:46
@OACDesigns
Copy link

Hi @karwosts, I just noticed that on the onboarding screen the password field is still missing.
It looks like this fix only targets the password field in the login form, but the original MR #18607 where the issue was introduced also included changes to ha-selector-text.ts file.
Perhaps you missed it?

@karwosts
Copy link
Contributor Author

I didn't realize onboarding used a different form element than login for password, so I didn't think that selector-text needed the same fix. I'll PR the same for selector-text.

@karwosts karwosts mentioned this pull request Nov 15, 2023
9 tasks
@karwosts
Copy link
Contributor Author

@OACDesigns - any chance you could test if #18665 resolves the problem? I don't have a setup to run onboarding flow at the moment.

@OACDesigns
Copy link

Hi, to be honest I'm not exactly sure how to do that...
I only came across this because I'm learning how to do addon development and was using the addon devcontainer which seems to use the latest frontend.

Do you know anything the devcontianers and how I might be able to point it to using your branch?

@karwosts
Copy link
Contributor Author

No problem, don't worry about it.

Sorry I don't know how to give you instructions for that at the moment, since I don't use the containers for frontend.

@github-actions github-actions bot locked and limited conversation to collaborators Nov 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants