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

Add TQ Energy Manager EM420 #4083

Merged
merged 16 commits into from
Aug 10, 2022
Merged

Add TQ Energy Manager EM420 #4083

merged 16 commits into from
Aug 10, 2022

Conversation

Djcd
Copy link
Contributor

@Djcd Djcd commented Aug 9, 2022

Hallo,

ich habe für meinen Haushalt den TQ EM420 hinzugefügt. Ich hoffe die Qualität des Codes ist okay. Ich habe mich an dem tq-em meter orientiert. Bei Fragen oder Verbesserungswünschen einfach Bescheid geben!

LG,
David

@andig andig added the devices Specific device support label Aug 9, 2022
@andig
Copy link
Member

andig commented Aug 9, 2022

Bitte mal

make assets

irgendwas ist da krumm :/

@Djcd
Copy link
Contributor Author

Djcd commented Aug 9, 2022

ich schau nochmal

make assets
go generate ./...
enumer: internal error: package "fmt" without types was imported from "github.com/evcc-io/evcc/charger/easee"
charger/easee/signalr.go:30: running "enumer": exit status 1

spuckt er bei mir aus

@andig
Copy link
Member

andig commented Aug 9, 2022

Ähhhhhhh…..

@Djcd
Copy link
Contributor Author

Djcd commented Aug 9, 2022

Okay, läuft jetzt sauber durch. Ich hatte für ein anderes Projekt noch eine andere enumer binary -.-... dummer Zufall... hab den Code entsprechend committed

@Djcd
Copy link
Contributor Author

Djcd commented Aug 9, 2022

So eure Tools wollten nochmal "aufräumen". Die Commits 0e08c3a und add6ca8 haben aber wenig mit meinem PR zu tun 🤷.

@andig
Copy link
Member

andig commented Aug 9, 2022

Sieht aus, als würden die überarbeiteten go doc comments aus 1.19 zuschlagen. Ich prüf das lokal und dann könntest du ggf rebasen.

meter/tq-em420.go Outdated Show resolved Hide resolved
@andig
Copy link
Member

andig commented Aug 9, 2022

/rebase

@andig
Copy link
Member

andig commented Aug 9, 2022

Mhhm, jetzt ist master kaputt. Irgendwas mit Go 1.19 scheint nicht ganz takko: golangci/golangci-lint-action#545

@andig andig merged commit b9f7c77 into evcc-io:master Aug 10, 2022
@andig
Copy link
Member

andig commented Aug 10, 2022

Super PR, vielen Dank!

@Djcd
Copy link
Contributor Author

Djcd commented Aug 10, 2022

Ich danke Euch für das Projekt! Ich musste leider heute morgen früh los. Hätte erst heute Abend nochmal darüber geschaut, danke fürs Anpassen!

@andig
Copy link
Member

andig commented Aug 10, 2022

Sehr gerne. Wenn Du Lust hast tiefer einzusteigen bist Du herzlich willkommen- am Besten bei Slack melden!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants