simon-s-99 / games-r-us

A Blazor Web App created by Samuel Lööf, Adam Kumlin, Gabriel Wedin & Simon Sörqvist.
MIT License
3 stars 0 forks source link

Add breadcrumb navigation functionality #43

Closed adamkumlin closed 5 months ago

adamkumlin commented 5 months ago

image

Click the buttons circled in red to instantly perform a search with that platform or game category.

adamkumlin commented 5 months ago

The search functionality does not work in this branch, i do not know if this is present in main or in other branches aswell.

I get this error in ListingDisplayAll.cs image

This is most likely due to a change @Samuel-Loof made where prices are stored in a dictionary and now the program cant find the post in the dictionary. (<- this is a guess tho)

The error also occurs if i try to use the breadcrumb navigation: image

and then try to sort: image

https://github.com/simon-s-99/games-r-us/assets/71759619/ce76f352-f55e-4172-93f5-26965a3a2289

Weird. Seems to be fixed now though by merging main into this branch.

simon-s-99 commented 5 months ago

I think this: image

from pr #61 will introduce a similar bug (still just a guess tho)

simon-s-99 commented 5 months ago

@adamkumlin get latest from main and then i will re-review Right now it looks like the breadcrumb navigation does not work at all but this might be fixed by getting latest from main.