Skip to content
This repository has been archived by the owner on Nov 30, 2018. It is now read-only.

Added search functionality #57

Closed
wants to merge 1 commit into from
Closed

Added search functionality #57

wants to merge 1 commit into from

Conversation

balepc
Copy link

@balepc balepc commented Aug 24, 2016

No description provided.

@coveralls
Copy link

coveralls commented Aug 24, 2016

Coverage Status

Changes Unknown when pulling 232ec97 on balepc:master into * on chadrem:master*.

@chadrem
Copy link
Owner

chadrem commented Aug 26, 2016

@balepc Thanks for building the search feature! Are you able to help maintain this feature in the future?

@chadrem
Copy link
Owner

chadrem commented Aug 26, 2016

@balepc I tried out the code and it looks like it returns 20 apps max:

s = MarketBot::Play::Search.new('Facebook').update; puts s.result.length

I'd like to get this fixed before merging any search related pull requests. If you have time, please check out issue #38 (it has a link to a possible fix).

@chadrem
Copy link
Owner

chadrem commented Mar 5, 2018

I’m closing this pull request due to inactivity.

@chadrem chadrem closed this Mar 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants