Neeraj Fix LB Dashboard Wishlist Filtering And Navigation#5335
Neeraj Fix LB Dashboard Wishlist Filtering And Navigation#5335Neeraj-Kondaveeti wants to merge 6 commits into
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
The dropdown is working as intended.
Both links navigate to their respective destination pages successfully. However, the destination pages appear to have the same functionality and very similar UI, so there does not seem to be a noticeable difference between them.
The "Chat with Host" button is not working consistently across all pages. It is not clickable on some internal pages.
One suggestion: After selecting a village and navigating to one of the internal pages like "Click for list overview" or "Click here to view availabilities", clicking the browser back button takes the user to the main wishlist page which has "All Villages" instead of the chosen village. It may be worth reviewing the navigation flow to improve the user experience.



Abhi-R0211
left a comment
There was a problem hiding this comment.
Tested locally on branch Neeraj_Fix_Listing_Bidding_Village_Filter_Navigation, and logged in as admin, here is my analysis:
What's Working Well
- Village filter dropdown is populated dynamically from actual wishlist data — Earthbag Village, Cob Village options present
- Go button triggers filtering correctly — selecting Earthbag Village and clicking Go shows only Unit 405
- Click here to view availabilities navigates correctly to
/lbdashboard/wishlist/:id/availability
- Click for list overview navigates correctly to
/lbdashboard/wishlist/:id
Issues
- Chat with Host — routes to 404
Clicking Chat with Host on the main wishlists page navigates to/lbdashboard/wishlist/:id/chat, which returns a PAGE NOT FOUND error
-
Chat with Host on detail/availability page — non-functional
On/lbdashboard/wishlist/:id/availabilitythe Chat with Host button is clickable but does nothing. No navigation or action occurs on click. -
Back button does not retain village filter
After selecting a village, clicking Go, and navigating into a detail page, pressing the browser back button resets the dropdown to All Villages instead of retaining the selected village. -
Theme inconsistency — navbar background in light mode
In light mode the navbar renders with a yellow-green background while the rest of the page is dark, creating a visually inconsistent appearance
- Theme inconsistency — dropdown in dark mode
When opening the village dropdown in dark mode, the unselected area renders with a white background, conflicting with the dark theme
HemanthNidamanuru
left a comment
There was a problem hiding this comment.
Hi,
Tested this PR locally. Found the following issues:
- Filter dropdown is not aligned in dark mode
- "Click here to view availabilities" doesn't navigate to /lbdashboard/wishlist/:id/availability, it goes to /lbdashboard/listingshome instead
- Chat with Host is clickable on the list page but results in a page not found error. On the individual village page, it is not clickable at all.
New bug posted here, thank you all: https://docs.google.com/document/d/1Jt_ulri6zaYq3KT6GKJzWmDlCVtVAdXSbaUlZWRtJBs/edit?tab=t.0#bookmark=id.boslaentzm6t |
Hi Hemanth, Thank you for taking the time to review this PR. I have made the changes addressing the issues. Can you please have a look again and let me know if everything works as intended. Thank You ! |
|
Thanks for retesting. I updated the dark-mode styling for the village filter dropdown so it aligns with the rest of the dark theme. Regarding the availability and list overview links, they currently navigate to different routes as implemented in this PR. Both routes render the existing WishListItem component based on the current route configuration. Creating a separate availability page/view would require additional functionality beyond the navigation fixes included in this task. |




























Description
Related PRS (if any):
This frontend PR is related to the Development Backend PR.
…
Main changes explained:
…
How to test:
npm installand...to run this PR locallyScreenshots or videos of changes:
Note: