Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Fix projects page search #9436

Merged
merged 5 commits into from
Dec 19, 2023
Merged

Fix projects page search #9436

merged 5 commits into from
Dec 19, 2023

Conversation

SYBIOTE
Copy link
Member

@SYBIOTE SYBIOTE commented Dec 13, 2023

This pull request fixes the search functionality on the projects page. Previously, the search was not filtering the projects correctly. With this fix, the search will now filter the projects based on the entered query.

the query now filters and shows across both, the installed and community projects section,

do we want to debounce the query to improve performance ?

Copy link
Member

@speigg speigg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please delete the debug log that was added. Also, yes if you think debouncing would improve the UX, let's do it.

communityProjects.set(data.sort(sortAlphabetical) ?? [])
console.log('DEBUG 2', data, communityProjects, filter.value)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please delete this log

@HexaField HexaField merged commit 1bfeea5 into dev Dec 19, 2023
12 checks passed
@HexaField HexaField deleted the fic-projects-page-search branch December 19, 2023 01:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants