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

Assign UUIDs per identifier and feature in the device config #677

Open
qdot opened this issue Nov 25, 2024 · 1 comment
Open

Assign UUIDs per identifier and feature in the device config #677

qdot opened this issue Nov 25, 2024 · 1 comment
Assignees
Labels
Features New things to add to the project Library - Core Tasks/Bugs pertaining to the core portion of the library Library - Server Tasks/Bugs pertaining to the server portion of the library

Comments

@qdot
Copy link
Member

qdot commented Nov 25, 2024

Trying to keep track of everything based on array indexes is a very bad idea. The amount of bookkeeping to swap between device definitions in different message versions causes a ton of bugs whenever we try to upgrade anything. We also don't really have a good way to tell when differences pop up between user config files and the main config (i.e. we add/remove features). We should be able to assign UUIDs in the config file and compare on those for lookup instead of constantly trying to keep random array indexes aligned.

@qdot qdot added Library - Server Tasks/Bugs pertaining to the server portion of the library Library - Core Tasks/Bugs pertaining to the core portion of the library Features New things to add to the project labels Nov 25, 2024
@qdot qdot self-assigned this Nov 25, 2024
@Alphalilly
Copy link

69th active issue NICE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Features New things to add to the project Library - Core Tasks/Bugs pertaining to the core portion of the library Library - Server Tasks/Bugs pertaining to the server portion of the library
Projects
None yet
Development

No branches or pull requests

2 participants