Skip to content

Config entry tweaks#960

Merged
balloob merged 7 commits intomasterfrom
config-entry-select
Mar 5, 2018
Merged

Config entry tweaks#960
balloob merged 7 commits intomasterfrom
config-entry-select

Conversation

@balloob
Copy link
Copy Markdown
Member

@balloob balloob commented Mar 2, 2018

Some fixes for config entries:

  • HA-Form: add support for picking an option
  • Fix overlay of paper-dialog inside app-layout for config flow wizard
  • Handle flows that finish/abort straight away

@andrey-git
Copy link
Copy Markdown
Contributor

The alerts are just for debug, right?

@balloob
Copy link
Copy Markdown
Member Author

balloob commented Mar 2, 2018

They are not for debug, they are more a temporary stopfix until I redesign and localise everything. Previously we would just get a console error.

@balloob
Copy link
Copy Markdown
Member Author

balloob commented Mar 4, 2018

I've removed the alerts. Now if a config flow aborts/finishes right away, it will be displayed in the same form as any "abort" or "finish" message that is received by interacting with a flow.

* The step of the current selected flow, if available.
*/
_flowStep: Object,
_userCreatedFlow: Boolean,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not used in bindings. Maybe make it a non-polymer property?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

this.userCreatedFlow = false;

this.setProperties({
_flowId: null,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only 1 property

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@balloob balloob merged commit e41b523 into master Mar 5, 2018
@balloob balloob deleted the config-entry-select branch March 5, 2018 21:07
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 8, 2022
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.

4 participants