search

ActoKids / AD440_W19_CloudPracticum

3 stars 1 forks source link

Filter input box #95

Closed coultergeist closed 5 years ago

coultergeist commented 5 years ago

For this final sprint, work with API team to configure the filter box on the event Dashboard. See also the Demo Issue for more info.

Once assigned, annotate your estimated time. Then, keep regular updates on your issue. Comment your approach, tutorials, articles, updates, links, photos, etc. Comment often. When closing your issue include the following in your final comment.

Time estimated: 15 hours Time spent: Pull Request link: Wiki link: Deployed Code link: Testers: What I tested:

coultergeist commented 5 years ago

Thanks to @MrSamsa42 for the link to this tutorial.

API route: https://api.2edusite.com/v1/events

coultergeist commented 5 years ago

The search function is working by both activity type and disability type (See Bug #109 Filter/search options)

Time so far: 7 hours

Problems: 1) See Bug 110 to ignore case of search input 2) See Bug 111 Default search items.

coultergeist commented 5 years ago

Time so far: 12 hours

1) Closed Bug 111 because filteredUsers was called incorrectly, causing the list of events to not appear. Now the events appear by default. 2) Ignore case on indexOf is still causing problems because of filtering array items. Going to try Regex

coultergeist commented 5 years ago

Feature highlight: By using fuse.js library achieved two exciting things while filtering events: 1) Enabled "dynamic" searching meaning matching results populate immediatly with all matching items and without having to press enter 2) Dramatically increases performance, as opposed to the original for loop which caused serious lagging. Fuse also enables the code to be expanded easily for new search criteria with less burden on the speed

Time estimated: 15 hours

Time spent: 20 hours

Pull Request link:

Wiki link: Configure filter input box on Dashboard

Deployed Code link: dev.2edusite

Testers: @erikNeville, "Filter is functioning correctly. Ignores case and follows the clients search criteria (see Bug #109)."

What I tested: