-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated search by name function #55
Conversation
This is ready for review Note: The final test will continue to fail (and may need rework) till we merge #28 and fix up the json of the already stored data |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Think the change might be causing a problem when putting any name into the top searchbox, the filter does not work and there's an error:
Uncaught TypeError: d.common_names.concat(...).join is not a function
at index.vue:96:46
at Proxy.filter ()
at Proxy.filterByName (index.vue:95:21)
at Proxy.applyAllFilters (index.vue:53:22)
at Proxy.handleSearch (index.vue:257:12)
at callWithErrorHandling (chunk-SBFBVZFQ.js:1779:18)
at callWithAsyncErrorHandling (chunk-SBFBVZFQ.js:1787:17)
at emit (chunk-SBFBVZFQ.js:2228:5)
at Proxy.search (NavigationBar.vue:65:12)
at _createElementVNode.onInput._cache.._cache. (NavigationBar.vue:16:46)
Unfortunately, this is because the json data in the main branch is out of date with the structure the new code expects, we need to fix the data with the correct structure as per my previous comment in #28 and merge those changes before the code in this PR works. Perhaps we should merge that branch in first? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New search works well, tests passing
changes requested have been address and approved
closes #54