-
Notifications
You must be signed in to change notification settings - Fork 89
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
Add Search Endpoint #9
Conversation
}, onResume: () { | ||
_bufferedPages.forEach(stream.add); | ||
if (_bufferedPages.last.isLast) { | ||
stream.close(); |
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.
just added some formatting here.
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.
Hey @hayribakici,
thanks for you contribution - looks pretty good overall. I've added some comments here and there, please check them out!
- Changes parser types for search result
@rinukkusu hi, I cleaned up the code. Now the I thought to add this comment to clarify potential confusion, because the PR diff is quite messy. |
- renames method
LGTM Thank you for contributing this new endpoint! 😸 |
This is a first attempt to add a search endpoint into this library. This is not a final stage, rather a discussion about the approach and maybe how it can be improved. Basically, this is the solution I came up with by giving the
Pages
class aMap
ofParserFunction
s (instead of only one) to enable distinguishing the different paging json objects in the search result.