You are viewing a single comment's thread from:

RE: SteemTools Rest API has been Released

in #utopian-io5 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.28
TRX 0.12
JST 0.033
BTC 62789.36
ETH 3158.62
USDT 1.00
SBD 3.82