search

nervousnet / nervousnet-HUB-iOS

Nervousnet HUB client for iOS
GNU General Public License v3.0
2 stars 0 forks source link

use map for parameter names -> types in GeneralSensorConfig #12

Closed a-enz closed 7 years ago

a-enz commented 7 years ago

makes it much more convenient to look up types

lewinernst commented 7 years ago

I think i agree with both emails, but i'll have to check properly tomorrow. I think it might introduce more complicated handling of false configurations, since we can't force types for incoming jasons.

On 22 Feb 2017, at 18:11, a-enz notifications@github.com wrote:

makes it much more convenient to look up types

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

lewinernst commented 7 years ago

*json

On 22 Feb 2017, at 18:11, a-enz notifications@github.com wrote:

makes it much more convenient to look up types

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

a-enz commented 7 years ago

This issue is getting more and more annoying. I will try to refactor code to fit this change soon. I think we should go even further and do something like [para Name -> (type, value)]

a-enz commented 7 years ago

Implemented