search

maiera / gde-app

Apache License 2.0
22 stars 9 forks source link

Allow empty impact field #214

Closed Scarygami closed 9 years ago

Scarygami commented 9 years ago

Not every metadata necessarily needs to have impact assigned right away. At the moment you get an ugly alert window that impact needs to be a valid integer. You could also just default to "0" if no value has been entered.

SmokyBob commented 9 years ago

Sanity check got mixed up and defaulted the numeric values to 0 only if the value is numeric.

This commit fixes the problem. Deploying in minutes with other fixes

SmokyBob commented 9 years ago

Deployed fix