Added ability to search API logs - #24
Conversation
JasonStoltz
left a comment
There was a problem hiding this comment.
I am sincerely sorry for taking so long to review this. This looks good to me, just a couple of tweaks please.
If you can make these changes and update your branch I am happy to merge and include in a release next week.
| "status": "429", | ||
| } | ||
| }) | ||
| {'results': [], |
There was a problem hiding this comment.
Could you update the formatting here to match the formatting of other responses in this README?
There was a problem hiding this comment.
Sure, I don't recall if that was the copied/pasted output or where that formatting came from. Not be facetious at all, but I'm not sure which format you prefer. I see examples of "pretty" formatted output, one-liners, and one-liners with elided output:
There was a problem hiding this comment.
You have a great point. How about "pretty"?
|
Thanks again for adding this. I'm going to merge and include in a patch release, 7.8.1, shortly. |
No description provided.