Skip to content
This repository has been archived by the owner on May 16, 2019. It is now read-only.

Search within Pages #467

Closed
7 of 8 tasks
drwasho opened this issue Jan 7, 2016 · 14 comments
Closed
7 of 8 tasks

Search within Pages #467

drwasho opened this issue Jan 7, 2016 · 14 comments

Comments

@drwasho
Copy link
Member

drwasho commented Jan 7, 2016

For tracking purposes.

We have a search bar for various pages that is yet to be wired up. We can track progress here and close the issue once it's done.

Discover Page

  • Search listings
  • Search pages

Chat

  • Search messages defer for v1.1

User Page

  • Search user page listings on store tab
  • Search followers/following list

Orders Page

  • Search purchases
  • Search sales
  • Search cases
@drwasho drwasho added this to the v1.0 - 'Feature complete' requirement milestone Jan 7, 2016
@jjeffryes
Copy link
Contributor

Are the two bullet points for searching chat and searching messages both referring to the same thing?

@drwasho
Copy link
Member Author

drwasho commented Jan 14, 2016

Fixed.

@rmisio
Copy link
Collaborator

rmisio commented Jan 14, 2016

The two User Page bullet points are complete and ready for QA.

@jjeffryes
Copy link
Contributor

For clarification, what should search messages in chat do?

Currently, the get_chat_conversations API doesn't return anything that can be searched against, other than guid.

Does the search filter the conversation you're looking at?

@hoffmabc
Copy link
Member

I thought I already implemented the search for chat which is just by GUID.

@jjeffryes
Copy link
Contributor

Right, is it listed here because there should be more to search by than guid?

@hoffmabc
Copy link
Member

I mean it would be nice if we could search conversations but that's not a 1.0. Should probably just consider this completed and defer full search until later.

@morebrownies
Copy link

screenshot 2016-01-15 08 10 19

@drwasho
Copy link
Member Author

drwasho commented Jan 18, 2016

Happy to see 'search by conversation' bounce to post-v1.

Search by handle to find an open conversation would be great for v1, but probably a stretch goal.

@drwasho
Copy link
Member Author

drwasho commented Jan 18, 2016

Searching a store page doesn't seem to be working. Can someone confirm?

@jjeffryes
Copy link
Contributor

It's working normally for me in the master branch.

@drwasho
Copy link
Member Author

drwasho commented Jan 18, 2016

I'm on master, and I can't search for any keywords when I go to a store page; Discover tab is working just fine for search.

@SamPatt
Copy link
Member

SamPatt commented Jan 26, 2016

I'm closing, @morebrownies or @jjeffryes feel free to re-open if it's premature.

@drwasho
Copy link
Member Author

drwasho commented Jan 26, 2016

Thanks Sam!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants