search

dickinson-comp190 / GitKit-FarmData2-S24

Other
0 stars 17 forks source link

BarnKit- Seeding report: Limit Dates to Existing Data? #45

Open wpgoble opened 8 months ago

wpgoble commented 8 months ago

Description: Users could enter invalid input data for checking Seeding Report in BarnKit tab

Steps to replicate the error:

  1. Log in using your account.
  2. Go to the BarnKit tab and click on the Seeding report at the left corner of the window.
  3. Enter some weird value for the year and don't do anything more!

This is an image When it comes to this page, there is no alerts for entering such non-sense dates.

Only shows up error when people click on "Generate Report" button.

Oberseved output This is an image

Desired output: Users should not be allowed to choose inputs for year that is earlier than the creation of the farm or laster than today's date.

Issue by AdiaWu Saturday Dec 18, 2021 at 08:11 GMT Originally opened as https://github.com/DickinsonCollege/FarmData2/issues/347

wpgoble commented 8 months ago

Interesting idea! It would be possible to limit the dates to the range of dates for which data is available in some situations. This would be a good discussion to have with the farmer as the project moves forward. There are some situations in which future dates will be necessary (e.g. as records used for scheduling new seedings or harvests).

Note: To limit the date ranges Issues #351 and #352 will need to be resolved.

Comment by braughtg Wednesday Dec 22, 2021 at 16:10 GMT