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

Update basis.yaml #175

Closed
wants to merge 1 commit into from
Closed

Conversation

cschillings
Copy link

safe_mode hoort niet meer bij OTA, standaard sowieso op true maar met dit in de ota config werkt het nogsteeds niet!

Wat implementeert/repareert dit?

Proposed change

Soorten wijzigingen

  • Bugfix (vaste wijziging die een probleem verhelpt)
  • Nieuwe functie (bedankt!)
  • Breaking change (reparatie/functie waardoor bestaande functionaliteit kapot gaat)
  • Ander

Test Omgeving

  • Watermeter
  • S0 teller
  • ESPHome versie ````yaml # v202 ```
  • Home Assistant versie ````yaml # v202 ```

Additional information

  • Deze PR repareert of sluit het probleem: fixes #
  • Deze PR is gerelateerd aan een probleem of discussie:
  • Link naar pull-aanvraag voor documentatie:

Voorbeeld invoer voor configuration.yaml:

# Example configuration.yaml

Checklijst:

  • De codewijziging is getest en werkt lokaal.
  • De codewijziging is nog niet getest.

Als door de gebruiker zichtbare functionaliteit of configuratievariabelen worden toegevoegd/gewijzigd :

  • Documentatie toegevoegd/bijgewerkt in de readme file.
  • Documentatie toegevoegd/bijgewerkt voor de webpagina [www.huizebruin.nl] of docs-flashpagina

safe_mode hoort niet meer bij OTA, standaard sowieso op true maar met dit in de ota config werkt het nogsteeds niet!
@huizebruin
Copy link
Owner

I fixed it, thanks for your info.
Bedankt voor je hulp, heb hem nu gooed en werkend.
komt zo ook online

huizebruin added a commit that referenced this pull request Jun 20, 2024
* Small fixes for https://s0tool.nl
* Add safe mode #175
* Minimum ESPHome version 2024.6.x
@huizebruin huizebruin closed this Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants