Adding a product_id mapping to show more information about devices

[mnederlof]

Not really sure if it is what i expected, but so far it looks like this:

envoy device:

inverter device:

relay device:

[mnederlof]

Open to suggestions for improvements 😄

[vincentwolsink]

Nice! I do think we need to lose the -rXX in the mapping. I’ve seen multiple Envoys of the same type with different numbers there. Probably the revision or something.

[mnederlof]

Yeah, that is already done, while matching the hardware id; while resolving, it also loses the 800- prepend in a second round of trying to match; see https://github.com/vincentwolsink/home_assistant_enphase_envoy_installer/pull/73/files#diff-0ac076ba3a419c22e8f034c216efb6b0347ddbac74545ec2eac8ceb10b067e4aR58

[vincentwolsink]

Ah yes indeed. 🙂

[vincentwolsink]

Now we need to crowdsource for product IDs I guess 😆

[mnederlof]

I'm just a bit sad the SKU with hardware id does not fit on one line for the inverter... 😄

[vincentwolsink]

If you drop the 800 and revision in the mapping already, it does 😜

[mnederlof]

If you drop the 800 and revision in the mapping already, it does 😜

yeah, but that kind of defeats the point of showing it alltogether 🤔

[vincentwolsink]

Ah, you were talking about the screenshot. I was looking at the code. I agree but let’s keep it like this, it’s fine.

[mnederlof]

And the CT model is now also generated based on either the EnvoyData class name, or the ALIAS value set in that class (which is the same value for you, but it will now show to which data class it has resolved to)