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

Phasenumschaltung #477

Merged
merged 5 commits into from
Jun 25, 2024
Merged

Phasenumschaltung #477

merged 5 commits into from
Jun 25, 2024

Conversation

pama87
Copy link
Contributor

@pama87 pama87 commented May 29, 2024

Problem:
Die Einstellung zur Verzögerung der Phasenumschaltung befindet sich unter dem Punkt Ladeeinstellungen Phasenumschaltung und ist nur sichtbar, wenn diese auf Automatik gestellt ist. Zudem beeinflusst die Verzögerung aber auch das Zielladen, welche diese Einstellung allerdings in den Optionen Ladeeinstellungen -> Zielladen nicht bietet. Die Phasenumschaltung in PV-Laden beeinflusst nicht die Phasenumschaltung beim Zielladen, auch nicht, wenn im Modus Zielladen mit PV-Überschuss geladen wird.

Lösung:
Die Einstellung Verzögerung der Phasenumschaltung wandert in die Option Übergreifendes und kann dort immer eingestellt werden, wobei sich diese Einstellung auf PV-Laden sowie Zielladen auswirkt. Dazu wurde:

  1. Das topic geändert
  2. Der Slider Verzögerung Phasenumschaltung in Übergreifendes migriert
  3. Neue Auswahloption Phasenumschaltung bei PV-Laden unter Zielladen hinzugefügt

Dadurch lässt sich die Phasenumschaltung im Modus Zielladen bei PV-Überschuss nun einstellen

@pama87 pama87 requested a review from benderl May 29, 2024 12:37
@pama87 pama87 self-assigned this May 29, 2024
Copy link
Contributor

@benderl benderl left a comment

Choose a reason for hiding this comment

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

Bitte vor einem PR immer den Linter verwenden.

@@ -279,6 +311,7 @@ export default {
"openWB/general/prices/pv",
"openWB/optional/et/provider",
"openWB/system/configurable/electricity_tariffs",

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

</template>
</openwb-base-button-group-input>
</div>
</openwb-base-card>
Copy link
Contributor

Choose a reason for hiding this comment

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

Es gibt ja schon eine Karte "Phasenumschaltung". Inhaltlich passt die neue Einstellung da rein, daher bitte beide Einstellungen in eine Karte.
Die bestehende Einstellung könntest Du in "Anzahl Phasen zum Ziel erreichen" umbenennen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Es gibt ja schon eine Karte "Phasenumschaltung". Inhaltlich passt die neue Einstellung da rein, daher bitte beide Einstellungen in eine Karte. Die bestehende Einstellung könntest Du in "Anzahl Phasen zum Ziel erreichen" umbenennen.

Wurde angepasst.

Comment on lines 230 to 235
Insofern Freigabe durch ID-Tags im
Ladepunkt-Profil aktiviert wurde, müssen die den
Fahrzeugen zugeordnete ID-Tags auch in das
entsprechende Ladepunkt-Profil eingetragen
werden, um zuzuordnen, an welchen Ladepunkten
die ID-Tags verwendet werden dürfen.<br />
Copy link
Contributor

Choose a reason for hiding this comment

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

Soll das hier rein? Das RFID-Thema ist doch ein eigener PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Soll das hier rein? Das RFID-Thema ist doch ein eigener PR.

wurde entfernt.

@pama87 pama87 force-pushed the Phasenumschaltung branch from ec70c90 to f885b40 Compare June 20, 2024 06:56
@LKuemmel LKuemmel merged commit 21765e3 into openWB:main Jun 25, 2024
1 check passed
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.

3 participants