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

readallproperties wrapping - values as string #984

Closed
danielpeintner opened this issue May 8, 2023 · 0 comments · Fixed by #985
Closed

readallproperties wrapping - values as string #984

danielpeintner opened this issue May 8, 2023 · 0 comments · Fixed by #985

Comments

@danielpeintner
Copy link
Member

danielpeintner commented May 8, 2023 •
edited
Loading

While quickly checking I noticed some more things that do not seem correct.

  • http://plugfest.thingweb.io:8083/counter/properties reports the properties but for all properties the value is wrapped into a string. Number should remain number and strings should not be wrapped in another string.
  • Locally checking the counter sample does report the following {"count":"0","countAsImage":"\"2023-04-14T08:47:10.678Z\""}. Wrapped values again and countAsImage reports the lastChange value !?
  • Sidenode: I also think that http://plugfest.thingweb.io:8083/counter/properties/countAsImage should not report the SVG as text "0". The surrounding quotes are not correct...

Originally posted by @danielpeintner in #949 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: report allproperties with proper type danielpeintner/thingweb.node-wot
1 participant
@danielpeintner