Skip to content

fixed field names#4

Open
MClimate wants to merge 2 commits intoactility:mainfrom
MClimate:MClimate-generic
Open

fixed field names#4
MClimate wants to merge 2 commits intoactility:mainfrom
MClimate:MClimate-generic

Conversation

@MClimate
Copy link
Copy Markdown

No description provided.

@mostafaibr mostafaibr self-requested a review April 4, 2022 10:07
@mostafaibr
Copy link
Copy Markdown
Contributor

Hello @MClimate and @yordan-zaychev,

The fixes seems to be great following your updates on the "points" section (even if no need to modify the output of decodeUplink).

However, I see in all the drivers, in the directory "/json-schemas" the json file is copied as it is from the template and was not adapted with the data outputted in your driver. Can you please update it? Otherwise you can remove the directory as it is not mandatory.

Copy link
Copy Markdown
Contributor

@ErwannWICART ErwannWICART left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All seems right.
You still got an "unsupported" -> extThermistorTemperature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants