You are viewing a single comment's thread from:

RE: SteemTools Rest API has been Released

in #utopian-io6 years ago

Thank you @nawab69 for your contribution!.

  1. I can see you are wrapping APIs upon the official full node - which you might want to extract that into a common configuration - if the node fails, you may want to have a fall-over mechanism to try next one.
  2. The header should really be before your echo. Otherwise, depending on network latency, sometimes the browsers may have a warning.
  3. Your APIs are not unit tested - could be good to have some tests
  4. You don't need the ?> which is recommended in API, otherwise, you might accidently put extra blank lines after - which will be valid HTML outputs, and you don't want that.
  5. Your curl code could be extracted into a function/class - this is to remove code duplication in your APIs.

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.

[utopian-moderator]

Sort:  

Thank you for your review, @justyy! Keep up the good work!

Coin Marketplace

STEEM 0.26
TRX 0.20
JST 0.038
BTC 96662.00
ETH 3660.13
USDT 1.00
SBD 3.84