Move search suggestion logic to selector #67
Labels
No labels
area: devops
area: discovery
area: docs
area: livestream
area: proposal
consider soon
dependencies
Epic
good first issue
hacktoberfest
help wanted
icebox
Invalid
level: 1
level: 2
level: 3
level: 4
needs: exploration
needs: grooming
needs: priority
needs: repro
needs: tech design
on hold
priority: blocker
priority: high
priority: low
priority: medium
resilience
Tom's Wishlist
type: bug
type: discussion
type: improvement
type: new feature
type: refactor
type: task
type: testing
unplanned
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: LBRYCommunity/lbry-redux#67
Loading…
Reference in a new issue
No description provided.
Delete branch "autcomplete"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Changes
query
key so previous suggestions are preserved when using the backspace/retyping the same queryLooks good.
@ -40,0 +58,4 @@
// They are probably typing/pasting in a lbry uri
return [
{
value: query,
Just an idea I had. I was thinking we should have an additional selector called
selectLocalSearchSuggestions
which could possibly be used in cases where an API call doesn't necessarily have to be made.@ -40,0 +58,4 @@
// They are probably typing/pasting in a lbry uri
return [
{
value: query,
Maybe based on previous searches?
@akinwale I'll create a PR to lbry-android after adding this into the app. The changes should be very minor.
@ -40,0 +58,4 @@
// They are probably typing/pasting in a lbry uri
return [
{
value: query,
Yeah, that makes sense.