tomaae / homeassistant-mikrotik_router

Mikrotik router integration for Home Assistant
Apache License 2.0
301 stars 50 forks source link

Next button for first step in optionsflow #266

Closed mvdwetering closed 1 year ago

mvdwetering commented 1 year ago

Proposed change

The first step in the Optionsflow currently shows a "Submit" button which is confusing since there is a next step. I sometimes forget and can't find where the sensors config is because it looks like there is no more pages (and I don't read the 1/2 part of the title... ).

Type of change

Additional information

How it looks after image

Checklist

mvdwetering commented 1 year ago

Hmm, not sure why CI is failing, does not seem related to the change I did.

tomaae commented 1 year ago

Oh, didnt knew they added "Next" option, thanks. No idea whats wrong with flake, but its nothing you did for sure.

tomaae commented 1 year ago

Issue with tests were changes in flake8 itself and how it parses github config. All good now.