You are viewing a single comment's thread from:

RE: Steem Randow Draw 1.4

in #utopian-io6 years ago

Thank you for your contribution. It is good to see the project has been resumed after 6 months. I'd like to give a few suggestions if you don't mind.

  1. It is better to use === instead of == in your JS code.
  2. It is better to separate your commits into a few, and meaning ones instead of a single commit. Also, the commit messages need to be more descriptive.
  3. Please start using PR to group your commits that will be easier to follow.
  4. Your JS and HTML are mixed in the same file - which is a code smell - it is better to separate the logics from the presentation.
  5. please start using let and const which will improve your code readability.

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:  

Thank you for the suggestions, I will put all this in place as soon as possible, beautiful day

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

Coin Marketplace

STEEM 0.19
TRX 0.14
JST 0.030
BTC 59715.05
ETH 3186.24
USDT 1.00
SBD 2.42