Skip to content

Fixes Bug #170 + minor improvements in BatteriesViewStrategy#171

Closed
JensLorenz wants to merge 3 commits into
TheRealSimon42:mainfrom
JensLorenz:main
Closed

Fixes Bug #170 + minor improvements in BatteriesViewStrategy#171
JensLorenz wants to merge 3 commits into
TheRealSimon42:mainfrom
JensLorenz:main

Conversation

@JensLorenz

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • npm run build laeuft ohne Errors/Warnings
  • Dashboard laedt fehlerfrei (kein Timeout, keine Console-Errors)
  • Strategy-Editor oeffnet sich und Einstellungen sind funktional
  • Smoke Test: Alle Views kurz pruefen (Overview, Lichter, Rollos, Raum-Detail)
  • Betroffene Views/Cards ausfuehrlich getestet

@codacy-production

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 4 high

Alerts:
⚠ 4 issues (≤ 0 issues of at least minor severity)

Results:
4 new issues

Category Results
BestPractice 1 high
ErrorProne 3 high

View in Codacy

🟢 Metrics 4 complexity · 0 duplication

Metric Results
Complexity 4
Duplication 0

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@TheRealSimon42

Copy link
Copy Markdown
Owner

Hi @JensLorenz, vielen Dank für deinen PR! 🙌

Da in der Zwischenzeit ein größerer Merge auf main passiert ist (#172), habe ich deine Änderungen manuell geportet statt zu rebasen — so bleibt die History sauber.

Dein Fix für die Floor-Icons und das Battery-View-Refactoring sind in #174 übernommen. Ich habe eine kleine Anpassung gemacht: Der Voltage-Sensor-Filter (unit && unit !== '%') bleibt erhalten, da Spannungssensoren (V, mV) sonst gegen Prozent-Schwellwerte verglichen werden würden.

Du bist natürlich als Co-Author im Commit eingetragen. Danke für den Beitrag! 🎉

TheRealSimon42 pushed a commit that referenced this pull request Apr 15, 2026
Corrects Co-Authored-By email so GitHub links the contribution
to @JensLorenz's profile (floor icon fallback + battery refactor).

See: #174, #171

Co-Authored-By: Jens Lorenz <2900362+JensLorenz@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@JensLorenz

Copy link
Copy Markdown
Contributor Author

Hi @TheRealSimon42 , auch Dir vielen Dank für deine Arbeit !!!
Du biste einer der Gründe warum ich auf HA gewechselt habe.

Bzgl:

Dein Fix für die Floor-Icons und das Battery-View-Refactoring sind in #174 übernommen. Ich habe eine kleine Anpassung gemacht: Der Voltage-Sensor-Filter (unit && unit !== '%') bleibt erhalten, da Spannungssensoren (V, mV) sonst gegen Prozent-Schwellwerte verglichen werden würden.

Der Vergleich ist eigentlich nicht mehr nötig, da nur folgende Entities in Betracht kommen:

  • Binar Sonsoren mit 'battery' im Namen
  • Normale Sensoren in der Prozent-Einheit -> Spannungssensoren sind nicht mehr in der Liste

TheDave94 pushed a commit to TheDave94/oriel-dashboard that referenced this pull request May 21, 2026
Corrects Co-Authored-By email so GitHub links the contribution
to @JensLorenz's profile (floor icon fallback + battery refactor).

See: TheRealSimon42#174, TheRealSimon42#171

Co-Authored-By: Jens Lorenz <2900362+JensLorenz@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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