-
Notifications
You must be signed in to change notification settings - Fork 395
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
vendorProfileId #409
vendorProfileId #409
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@johanstokking: We are free to assign the vendorProfileID
right? Should we follow some internal convention?
Just use linear numbers for each new profile?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed we're free to assign them. They're 16 bit. This is already enforced by the JSON schema validator.
I would indeed make them increasing and not jump to 868.
Update in linear numbers
we can establish that 'vendorProfileId' should be filled linear instead of freely choice, what do you think? |
Ideally users can fill any valid value and we only check that it is unique for a given vendor. But if that's too complicated to validate, then I guess it's ok to make it linear. |
We can make this part of validation in CI. Can you file an issue @Jaime-Trinidad ? |
Yes I file an issue |
Summary
Add unique
vendorProfileId
to The Things Product and The Things Industries devices #405Changes
vendorProfileId
in each profile file, without repeating the IdNotes for Reviewers
...
Release Notes