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

Elli (Volkswagen) in overview page #7757

Closed
wants to merge 2 commits into from
Closed

Elli (Volkswagen) in overview page #7757

wants to merge 2 commits into from

Conversation

martinknechtel
Copy link
Contributor

@martinknechtel martinknechtel commented May 1, 2023

My suggestion: when I first read about the project and had a glimpse on the overview I did not recognise the Volkswagen Wallbox in your list of supported devices. I propose to say "Elli (Volkswagen)" there in the Overview. Also because there is a large "VW" logo printed on the front plate of the Wallbox, but no "Elli" logo.

Thx for this nice project and your work. It works like a charm on my site with Elli (Volkswagen) ID. Charger Pro + Sungrow SH8. I use 1-phase loading with 6-16 Ampere. The configuration dialog on the commandline is comfortable. With your fresh release yesterday v0.116.6 also the connect status is fixed and works. Thanks for all that.

Learned from evcc-io/evcc.io#16 that this would require changes in the main project. For this reason, creating this PR

My suggestion: when I first read about the project and had a glimpse on the overview I did not recognise the Volkswagen Wallbox in your list of supported devices. I propose to say "Elli (Volkswagen)" there in the Overview.

Thx for this nice project and your work. It works like a charm on my site with Elli (Volkswagen) ID. Charger Pro + Sungrow SH8. I use 1-phase loading with 6-16 Ampere. The configuration dialog on the commandline is comfortable. With your fresh release yesterday v0.116.6 also the connect status is fixed and works. Thanks for all that.
@martinknechtel martinknechtel marked this pull request as ready for review May 1, 2023 05:41
@DerAndereAndi
Copy link
Contributor

The Wallbox is available with branding for Skoda, Cupra, Audi and VW and also as plain Elli, see https://www.elli.eco/de/privatkunden/produkte/wallbox

In my opinion this change doesn't make it better.

@naltatis
Copy link
Member

naltatis commented May 1, 2023

In general I'm in favor of using the names the products are used in sales and marketing by their vendors. The fact that these three wallboxes are available in five brandings each creates a little product explosion in our listing. But from a users perspective It's not clear that theses are all the same box.

Correction: This only affects two boxes: "Charger Connect" and "Charger Pro", not the simple "Charger" which isn't controllable.

@naltatis
Copy link
Member

naltatis commented May 1, 2023

It's even more confusing than I thought: "Elli Charger Connect" seems to be the same as "Audi Wallbox plus" 🙈
I'll go through the brands and extend this PR.

@naltatis
Copy link
Member

naltatis commented May 1, 2023

If I didn't make a mistake this is the result:

Elli Charger Connect Charger Pro
Volkswagen ID. Charger Connect ID. Charger Pro
Skoda iV Charger Connect iV Charger Connect+
Cupra Charger Connect Charger Pro
Audi Wallbox plus Wallbox pro

@naltatis
Copy link
Member

naltatis commented May 1, 2023

Had trouble to push my changes there. That's why I've created a new PR and close this one. Further discussion please in the new one.

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