arugyani / shopIQ

LLM powered high-consideration research tool for UTD senior project

1 stars 0 forks source link

64 frontend fix redux #79

Closed arugyani closed 2 months ago

arugyani commented 2 months ago

Redux Overhaul + Feature Integration

This PR overhauls our redux data handling as implemented by @prajwalpkn and connects all application features within redux as implemented by @arugyani .

In addition, it includes QoL styling improvements, uuid's for history objects, modularizes components, and backend API connections.

Closes #64

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
shop-iq	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 12, 2024 9:29pm

prajwalpkn commented 2 months ago

I checked the branch out, here are a couple issues:

After clearing history and typing in a new search, if the filter tags were minimized before, it continues to stay minimized even in the new search. This can be fixed by resetting the minimize state when theres a new search.
The text typed in the search bar is removed when I click enter. It feels awkward to lose the text typed in. The default value of the search bar should be what was typed in at all times.

arugyani commented 2 months ago

Changed behavior for the following:

State minimization is reset for new searches.
Query is cleared only for new searches. Searches with an existing search history (i.e. currentHistoryId) are not cleared.

prajwalpkn commented 2 months ago

The change has introduced a new bug: A new search after clearing history shows no questions on the screen

prajwalpkn commented 2 months ago

Additionally, switching between past searches does not update the query in the search bar, not sure if this was intended.

prajwalpkn commented 2 months ago

Looks good to go!