agrc / electrofishing-query

A web application for viewing and querying electrofishing data
https://electrofishing-query.ugrc.utah.gov
MIT License
0 stars 0 forks source link

Crude Results Grid #228

Closed stdavis closed 1 week ago

stdavis commented 1 week ago

The UI is super-ugly. But the data seems to be loading correctly. No fancy paging. It seems to be fine. I plan on addressing the look and feel in my next PR.

github-actions[bot] commented 1 week ago

Visit the preview URL for this PR (updated for commit 12463d971d16b35d10b55489fa6895619d743098): Firebase Preview: https://ut-dts-agrc-electro-query-dev--results-grid-ra7j0o98.web.app (expires "2024-11-13T23:28:31.741762477Z" )

steveoh commented 1 week ago

Is this caused by the preview environment variables?

image

What's a good filter to use to view the table?

I tried some dates and was shown this...

Request failed with status code 404: POST https://ut-dts-agrc-electro-query-dev--results-grid-ra7j0o98.web.app/maps/feature/dynamicLayer/query

Network console shows

https://ut-dts-agrc-electro-query-dev--results-grid-ra7j0o98.web.app/maps/feature/1?f=json 404'ing with an html page which makes me think the firebase rewrites are fouling up that request.

stdavis commented 1 week ago

@steveoh It appears that passing the alias for the project id works as expected. This is ready for another look when you have a minute.

What's a good filter to use to view the table?

Selecting "Disease certification" and "Presence/absence" should bring up a bunch of data in the table.

stdavis commented 1 week ago

At what point do we want to bring the table code back into the design system?

I was just waiting for a release with the table component in it.

steveoh commented 1 week ago

I was just waiting for a release with the table component in it.

Sounds good.