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

Features/evse improve #305

Merged
merged 1 commit into from
Aug 22, 2020
Merged

Conversation

schenlap
Copy link
Contributor

WIP

@andig wie kann ich die Verwendung der meter configurierbar machen? Da brauche ich noch Unterstützung.

Copy link
Member

@andig andig left a comment

Choose a reason for hiding this comment

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

An sowas hatte ich auch geadacht. Wofür die Konfigurationsoption? Lass uns das doch immer auf alwaysactive schalten, ist wohl auch besser für den controller.

@andig
Copy link
Member

andig commented Aug 22, 2020

Du hast die Messfunktionen einkommentiert. Funktioniert das immer, also hat jede EVSE einen Zähler? Falls nein muss das konfigurierbar sein (meter: true?)

@andig
Copy link
Member

andig commented Aug 22, 2020

Kannst im default meter schauen, das ist aber nixht trivial. Vorschlag: diesen pr fertig machen und auskommentiert lassen, ich mache den rest.

@schenlap
Copy link
Contributor Author

An sowas hatte ich auch geadacht. Wofür die Konfigurationsoption?

du meinst alwaysActive? Das liest er ja aus und ist keine Konfigoption. Ich würde das gerne so lassen, dann geht es unabhängig was der Benutzer eingestellt hat.

Du hast die Messfunktionen einkommentiert. Funktioniert das immer, also hat jede EVSE einen Zähler? Falls nein muss das konfigurierbar sein (meter: true?)

Nein, hat nicht jede. Ich mach kommentiere es wieder raus.

@andig
Copy link
Member

andig commented Aug 22, 2020

du meinst alwaysActive? Das liest er ja aus und ist keine Konfigoption. Ich würde das gerne so lassen, dann geht es unabhängig was der Benutzer eingestellt hat.

Ok- dann sollten wir immer beim Start eine Warnung ausgeben da das anscheinend für den Controller nicht ganz optimal ist falls alwaysactive nicht aktiviert.

}
url = fmt.Sprintf("%s?current=%d", evse.apiURL(evseSetCurrent), current)
} else {
url = fmt.Sprintf("%s?active=%v", evse.apiURL(evseSetStatus), enable)
Copy link
Member

Choose a reason for hiding this comment

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

vor das if ziehen, dann brauchts weder variable noch else

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

charger/evsewifi.go Outdated Show resolved Hide resolved
charger/evsewifi.go Outdated Show resolved Hide resolved
charger/evsewifi.go Outdated Show resolved Hide resolved
@schenlap schenlap force-pushed the features/evse-improve branch from 1725289 to 9b92f42 Compare August 22, 2020 13:24
@schenlap schenlap marked this pull request as ready for review August 22, 2020 13:25
@andig andig merged commit 705d79a into evcc-io:master Aug 22, 2020
@schenlap schenlap deleted the features/evse-improve branch August 23, 2020 05:45
@andig
Copy link
Member

andig commented Aug 24, 2020

@andig wie kann ich die Verwendung der meter configurierbar machen? Da brauche ich noch Unterstützung.

Wird auf Basis von #310 implementiert

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