feat: Add support model T2258 - RoboVac G20 Hybrid#424
Conversation
|
@damacus Please add this. |
| | T2252 | RoboVac G30 Verge | 3.3 | | ||
| | T2253 | RoboVac G30 Hybrid | 3.3 | | ||
| | T2254 | RoboVac G20 Hybrid | 3.3 | | ||
| | T2258 | RoboVac G20 Hybrid | 3.3 | |
There was a problem hiding this comment.
What's the difference between this and the 2254?
There was a problem hiding this comment.
T2258 is Eufy RoboVac G20 Hybrid, and T2254 is Eufy RoboVac G40 Hybrid, which supports supports self-empty stations unlike G20 Hybrid.
|
Thanks for adding T2258 support. The registry wiring looks good, but please verify the T2258-specific command values before merge. The public T2258/G20 Hybrid manual only documents Auto + Spot cleaning modes and Quiet/Standard/Turbo/Max suction levels, while this PR adds SmallRoom/Edge/Nosweep and omits Quiet. Please either include a real T2258 DPS dump confirming these values or adjust the mapping/tests to the documented T2258 capabilities. Also please fix the supported-model docs so T2258 is listed as G20 Hybrid without leaving the existing conflicting T2254 G20 Hybrid entry. |
damacus
changed the title
3 participants
Add support model T2258 - RoboVac G20 Hybrid