You are viewing a single comment's thread from:

RE: Steem Automation - Automate stuff on steem easily

in #utopian-io6 years ago

Thank you for your contribution. The idea looks promising and interesting, although I haven't seen any examples in your repro or post.

  1. You probably want to add a handler (or callback) to getTransactions so that the logics of handling the automation and the getting blocks/transactions can be separated.
  2. inconsistent usage of let and var, and the line-ending semicolon.
  3. You may want to re-format the utils.js, and add some comments and blank lines between functions.
  4. private activekey, the steem node are better configured/stored off the source code e.g. env parameter
    5.console.log could be replaced by a Logger object which we can decide where to log e.g. output to files.
  5. commented-out code should be removed to keep your codebase clean.

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 @justyy for the review! We are soon adding codacy to our projects for automated code reviews which will help us get a good score here. The post is written by a Developer not a professional content writer so quality might be low. We are in a process of hiring content writers now.

You can see the automated review of this project using the link below :
https://app.codacy.com/app/nirvanaitsolutions/steem-automation/issues/index?bid=11833394#

Posted using Partiko Android

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