You are viewing a single comment's thread from:

RE: Steem Wallet Email Notification

in #utopian-io6 years ago

Your contribution has been evaluated according to Utopian rules 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

A couple of feedbacks:

  • You don't need to store node_modules on your repository. (Considered bad practice.) npm can install the dependencies based on the config. In some enterprise environments, storing node_modules for locking the 3rd party libraries may make sense but it's not required for a small scale project like this.

  • Since this is designed as a long running process, a small instruction to use it with pm2 would be good.

  • That would be also super-cool if you create an adapter pattern on notification providers so I can add a "SMS Adapter" and get the wallet notifications in SMS. :)


Need help? Write a ticket on https://support.utopian.io/.
Chat with us on Discord.
[utopian-moderator]

Sort:  

Thanks @emrebeyler, for your wonderful feedback. Actually I did not know about pm2. Seems pm2 will be a wonderful process manager for this application . I am trying to work on it.

Thanks again for your SMS Notification idea, I am also thinking about SMS notification. I will must add this option with this application. 😑😑😑

Hey @emrebeyler
Here's a tip for your valuable feedback! @Utopian-io loves and incentivises informative comments.

Contributing on Utopian
Learn how to contribute on our website.

Want to chat? Join us on Discord https://discord.gg/h52nFrV.

Vote for Utopian Witness!

Coin Marketplace

STEEM 0.19
TRX 0.15
JST 0.029
BTC 63466.72
ETH 2683.95
USDT 1.00
SBD 2.80