You are viewing a single comment's thread from:

RE: Introducing Steem Voice - a Google Assistant app for the Steem blockchain

in #utopian-io5 years ago

Thank you very much for your creative work! This has blown my mind and I am sure this project is great use to many steemians.

  1. for your second problem, you can always set up a server to cache some results so it is quicker to query than the steem blockchain.
  2. I can see from your code there are some duplication e.g. 'what is your username' I am sure this can be refactored into a module.
  3. You might want to organize your source code files into different modules for example, utils, steem-utils etc.
  4. Unit tests are missing - which is good to add from the start of the project.
  5. You can add Vests directly in Steem/Python i.e. you don't need clean. Also you don't need rep_cal as it is already implemented here

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? Write a ticket on https://support.utopian.io/.
Chat with us on Discord.
[utopian-moderator]

Sort:  

Thanks a lot. I'll make sure to follow your advice in the next commit.

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

Coin Marketplace

STEEM 0.29
TRX 0.12
JST 0.033
BTC 63149.00
ETH 3156.91
USDT 1.00
SBD 3.85