-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Convert onboarding to Lit #2894
Conversation
We should allow password show clear text here (even set it as default). Because we don't have password_confirm field, if user either forget or have a typo in the password, he/she will have to redo on-boarding again. Of course, it is better than current situation, if user clicked |
That's a good point. We should do that for a future PR . Now that it's Lit, things will be easier. |
class HaOnboarding extends litLocalizeLiteMixin(LitElement) { | ||
public translationFragment = "page-onboarding"; | ||
|
||
@property() private _name = ""; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are these not initialized if you set the type to string?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nope, it's undefined
"ui.panel.page-onboarding.user.required_field" | ||
)}" | ||
></paper-input> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a password confirmation input and require the two fields to match
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Co-Authored-By: balloob <paulus@home-assistant.io>
I've added a password confirm. |
Requires home-assistant/core#21777