You are viewing a single comment's thread from:
RE: SteemRadar.js - A New Major Release, New Features, Update 1.0.0
Thank you for your contribution!
- You may add the unit tests part of the CI using travers-ci i.e. showing a green tag on the project page along with others.
- The tests you added previously may seem weak, which doesn't test your latest changes.
- Lack of Sample code to use your aliases
- Have you considered statusCode of 301 and 302 redirection? which might not mean Request Failed
- You might want to extract the
for (const trx of transactions) {
and use a filter call back to remove the duplication code in getPosts, getComments.
Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.
To view those questions and the relevant answers related to your post, click here.
Need help? Chat with us on Discord.
Thanks for the review.
Also parameters documented here
Thank you for your review, @justyy! Keep up the good work!