project-icp / bee-pollinator-app

The web application front end for the ICP Pollinator Decision Support Tool 🐝
Apache License 2.0
6 stars 1 forks source link

Minor Changes to Surveys #533

Closed rajadain closed 4 years ago

rajadain commented 5 years ago

Overview

Performs minor changes requested by the user to all surveys. These are needed primarily for the Fall survey at this point, but have been applied to all surveys for uniformity (and because the clients have confirmed that they do want these changes on all surveys).

Connects #532

Demo

Negative values are not allowed in numeric fields anymore:

2019-10-18 17 31 22

Text changes to the Survey tab:

image

"Sell nucs/packages" in the User Survey:

image

Notes field on Monthly surveys:

image

Notes field on November surveys:

image

Notes field on April surveys:

image

Testing Instructions

rajadain commented 4 years ago

It is mysterious because I can't find where notes is null or undefined.

Fixed in e544d2c by defaulting to '' in case of null or undefined.

rajadain commented 4 years ago

Ready for another look.

rajadain commented 4 years ago

Thanks for reviewing!

dboyer commented 4 years ago

Does this task also include:

• Add “N/A – no colonies lost” option in Spring survey for the “What do you think was the most likely cause of colony loss?” question

rajadain commented 4 years ago

No, this did not. May have missed that between the Google Doc and the GitHub Issue.

If there's any more additions, we can collect them in a new issue that can be brought on as a ➕ card.