-
-
Notifications
You must be signed in to change notification settings - Fork 49
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 company search by name and pagination to listAllCompanies #149
Conversation
Signed-off-by: Derrick Mehaffy <derrickmehaffy@gmail.com>
@arjunkomath question for you, do I need to manually update the docs or is that handled automatically. I assume I need to update the README for the new options yeah? |
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.
Looks good, thank you!
@arjunkomath was there anything missing needed to merge and release? |
Sorry I didn't get time to release it, all done now |
(First time contributing to this project so do let me know if I am missing something)
TODO: