TheThingsNetwork / arduino-node-lib

Arduino Library for The Things Node
MIT License
22 stars 26 forks source link

Adapt Uno Basic decoder to new formatter API #27

Open rvolosatovs opened 3 years ago

rvolosatovs commented 3 years ago

Adapt to v3 API changes

rvolosatovs commented 3 years ago

@johanstokking I have no rights to request review

johanstokking commented 3 years ago

In general I'm okay with this, but until we migrate the community network to V3, this can create confusion.

Note that V3 is backwards compatible with the V2 codec.

rvolosatovs commented 3 years ago

In general I'm okay with this, but until we migrate the community network to V3, this can create confusion.

Note that V3 is backwards compatible with the V2 codec.

this is done now, right?

johanstokking commented 3 years ago

In general I'm okay with this, but until we migrate the community network to V3, this can create confusion. Note that V3 is backwards compatible with the V2 codec.

this is done now, right?

What is done?

rvolosatovs commented 3 years ago

In general I'm okay with this, but until we migrate the community network to V3, this can create confusion. Note that V3 is backwards compatible with the V2 codec.

this is done now, right?

What is done?

PCN is running v3 now, right? We have a v3 community cluster

johanstokking commented 3 years ago

PCN is running v3 now, right? We have a v3 community cluster

Yeah but it's not community accessible yet. Let's keep this open for now.