Skip to content
This repository has been archived by the owner on May 19, 2020. It is now read-only.

Bug adding element to an array of numbers #38

Closed
dselman opened this issue Feb 24, 2020 · 1 comment · Fixed by #41
Closed

Bug adding element to an array of numbers #38

dselman opened this issue Feb 24, 2020 · 1 comment · Fixed by #41
Labels
bug Something isn't working

Comments

@dselman
Copy link
Sponsor

dselman commented Feb 24, 2020

Adding a number to an array of numbers, adds a string.

To Reproduce

  1. Go to https://concerto-ui.netlify.com
  2. Click the + to add a new accelerometer reading
  3. Edit the reading from 0 to 10
  4. Note that the JSON for the readings has added "10"

Expected behavior
Numbers should not be added as strings

Screenshots
Screenshot 2020-02-24 at 15 41 07

Desktop (please complete the following information):

  • OS: Mac OS X
  • Browser: Safari
  • Version: Latest
@dselman dselman added the bug Something isn't working label Feb 24, 2020
@dselman
Copy link
Sponsor Author

dselman commented Feb 24, 2020

This also implies that the JSON is not being validated, which would be a useful feature to add.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant