Merging the Application Client feature into master

TheThingsArchive / python-app-sdk

The Things Network Application SDK for Python

MIT License

29 stars 16 forks source link

Closed emmlejeail closed 6 years ago

emmlejeail commented 6 years ago

The PR provides the python SDk with an Application manager feature such as the one that can be found in the Node SDK or the Go SDK.

johanstokking commented 6 years ago

Great; I'm with @dotpy3 for code and @romeovs for consistency with node-app-sdk review comments

egourlao commented 6 years ago

For some reason GitHub marked a lot of comments of my review as outdated. You'll have to open them up separately, sorry!

emmlejeail commented 6 years ago

@romeovs do you have more reviews to make on the PR or are we good to go?