adafruit / adafruit/Wippersnapper_Components

Migrating I2C sensor subcomponent sensor types (e.g. light to lux) causes issues in UI. Component can't be edited/removed

Open
#322 1 comment 0 reactions 2 assignees Claimed by @lorennorman View on GitHub
bug
Dominant language
No language data
Stars
11
Forks
46
Avg merge
3d 18h
Merged PRs (30d)
2

Description

I migrated a new component (MAX44009 lux sensor) from light sensor type to lux, and now I cannot edit the existing one I registered to a device to remove and re-add it (so the units go from none/value to lux).

**The edit page opens as a create page instead of edit**, due to the max44009:**light** no longer matching the newly migrated / expected components max44009:**lux** definition.

The device must now be deleted to fix/remove the component, I wonder if anyone else has one registered (it still records data okay, just wrong units and uneditable / unremovable).

https://github.com/user-attachments/assets/985ad26b-93eb-4800-8a01-549f15d63a6e

@lorennorman this is v1, on production.
Here's the PR that broke my component on device page, migrating light to lux:
[https://github.com/adafruit/Wippersnapper_Components/pull/321](https://github.com/adafruit/Wippersnapper_Components/pull/321)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.