Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Null value should not be "null" but null? #354

Closed
oscgonfer opened this issue Sep 4, 2024 · 1 comment · Fixed by #362
Closed

Null value should not be "null" but null? #354

oscgonfer opened this issue Sep 4, 2024 · 1 comment · Fixed by #362
Assignees
@timcowlishaw
Labels
bug
Milestone
Usability and int...

Comments

@oscgonfer
Copy link
Contributor

When we query the devices/device-id/readings endpoint, we should make sure that we don't return strings containing "null", but a null.
@oscgonfer oscgonfer added the bug label Sep 4, 2024
@oscgonfer oscgonfer added this to the Usability and integration milestone Sep 4, 2024
@oscgonfer oscgonfer mentioned this issue Sep 6, 2024
Open
@oscgonfer oscgonfer mentioned this issue Sep 27, 2024
Open
@timcowlishaw
Copy link
Contributor

This should be fixed by the new parser

@timcowlishaw timcowlishaw linked a pull request Oct 14, 2024 that will close this issue
New Raw MQTT Message parser #362
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@timcowlishaw timcowlishaw

Labels
bug
Projects
None yet
Milestone
Usability and integration
Development

Successfully merging a pull request may close this issue.

New Raw MQTT Message parser fablabbcn/smartcitizen-api
2 participants
@timcowlishaw @oscgonfer