Open ngottlieb opened 2 months ago
beta box changes incorporated into #591
flow tab changes incorporated into #591
The existing state lists, it turns out, are still Laravel-based – e.g. https://www.americanwhitewater.org/content/River/state-summary/?state=AK. I don't think it makes sense to make changes to the Laravel code-base to incorporate retrieving data from the new gauge API, which leaves us with two options:
1) launch new gauge system without a state-by-state table 2) re-write the state by state table in the Vue app
Any thoughts @AWKevin @tallowen ?
I think if it's not too much work, getting the state-by-state tab into vue makes sense to me.
I think if it's not too much work, getting the state-by-state tab into vue makes sense to me.
does an API already exist to support this? I'll look but you may know off the top of your head if the mobile app is doing something similar. I would need: query reaches by state and include correlation details with the response @tallowen
@ngottlieb - we don't have that API yet - the mobile app does all of it's sorting/filtering client side after it already has all of the information right now.
state table work is functional, merged into #591 via #593. It will continue to be updated when new APIs are available from @tallowen