codeforboston / codeforboston/windfall-elimination

Add validation of number input (to c1, possibly other pages)

Open
#156 1 comment 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
JavaScript
Stars
23
Forks
44
PR merge metrics
No merged PRs in 30d

Description

`parseFloat()` is being used to parse the number input on the c1 step, which may not end up with the right result because of how permissive this function is. Instead, we should have a front end validation to get the user to input numbers only.

There may be other inputs where the same thing is happening, so might be a good idea to audit the app for these and use the validation on all of them.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.