Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wpgoble opened this issue Jan 11, 2024 · 1 comment
Open

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

wpgoble opened this issue Jan 11, 2024 · 1 comment
Labels
enhancement New feature or request question Further information is requested

Comments

@wpgoble
Copy link
Contributor

wpgoble commented Jan 11, 2024

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 DickinsonCollege/FarmData2#347

@wpgoble wpgoble added enhancement New feature or request question Further information is requested labels Jan 11, 2024
@wpgoble
Copy link
Contributor Author

wpgoble commented Jan 11, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant