basic search channel pages #2887

Merged
neb-b merged 2 commits from channel-search into master 2019-09-19 19:08:32 +02:00
neb-b commented 2019-09-16 07:25:48 +02:00 (Migrated from github.com)

So i can search for stuff on channels

This is super basic, and probably should live in the component file, although after doing it this seems a lot simpler than running everything through redux (and a lot faster to write).

It does not include deep links to channel search queries. It was just something quick so I could use it.

Screen Shot 2019-09-16 at 1 21 32 AM
So i can search for stuff on channels This is super basic, and probably should live in the component file, although after doing it this seems a lot simpler than running everything through redux (and a lot faster to write). It does not include deep links to channel search queries. It was just something quick so I could use it. <img width="400" alt="Screen Shot 2019-09-16 at 1 21 32 AM" src="https://user-images.githubusercontent.com/16882830/64935797-218e3900-d821-11e9-80d4-04ce36ef80cf.png">
kauffj (Migrated from github.com) requested changes 2019-09-19 00:21:48 +02:00
@ -86,0 +107,4 @@
setSearchResults(null);
});
}
kauffj (Migrated from github.com) commented 2019-09-19 00:15:17 +02:00
This should probably be in https://github.com/lbryio/lbry-redux/blob/master/src/redux/actions/search.js
neb-b (Migrated from github.com) reviewed 2019-09-19 00:37:55 +02:00
@ -86,0 +107,4 @@
setSearchResults(null);
});
}
neb-b (Migrated from github.com) commented 2019-09-19 00:37:55 +02:00

I initially thought that and I think it makes sense to put it there eventually, but I'm starting to think small stuff like this is fine if live in the components.

All it is is one lighthouse call

I initially thought that and I think it makes sense to put it there eventually, but I'm starting to think small stuff like this is fine if live in the components. All it is is one lighthouse call
neb-b commented 2019-09-19 19:08:24 +02:00 (Migrated from github.com)

I'm going to merge this as-is since I don't have time to do more work on it. I think this is fine for the first basic version of channel search.

I'm going to merge this as-is since I don't have time to do more work on it. I think this is fine for the first basic version of channel search.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#2887
No description provided.