Sort:  

Nice cleanup, @crokkon.

Looks like it also helps to make it easier to contribute to the repository.

Small note:

Regarding this type of check, I find it more convenient to use groups for the permissions. (Ex: creating a editors group in the discord, and check for the user has that role.)

Since everyone do that in strict ID checks, wondering is there a specific reason for that? :)


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]

Thanks for the review, Emre! The main reason for using discord user IDs here is that is was like this before and I didn't want to introduce conceptual changes during this refactoring phase. Using a dedicated discord role is indeed a good idea here - thanks! Roles vs. ID checks is a bit a trade-off between where the setup and config is done and stored. User IDs allow for a minimal discord setup, any bare server can be used for testing. It may need adjustments in the bot config but all config is then under version control. Testing with roles may need fewer bot config changes but requires the discord server to be prepared accordingly - > out of git. Members wouldn't be revealed with roles, if this is considered sensitive information, though...

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

Wow thanks Crokkon. Your help has made a big difference. I am glad we have you on our team.
We are all learning so much from your skills.
We are super excited for the future of our project
Thanks for the mention, Ive been so busy I just noticed it.

Thank you for helping us! Moving those to a config file is definitely a good move.

Thank goodness we have you as one of the devs working on the bot.

I second this comment.

Third!

Hi, how can I join and flag spam posts?

Hi @jmehta, join the SFR Discord server: https://discord.gg/7pqKmg5
There is detailed information on how SFR works and the people there will be glad to help you getting started!

Hi @crokkon!

Your post was upvoted by @steem-ua, new Steem dApp, using UserAuthority for algorithmic post curation!
Your post is eligible for our upvote, thanks to our collaboration with @utopian-io!
Feel free to join our @steem-ua Discord server

Hello, as a member of @steemdunk you have received a free courtesy boost! Steemdunk is an automated curation platform that is easy to use and built for the community. Join us at https://steemdunk.xyz

Upvote this comment to support the bot and increase your future rewards!

Hey, @crokkon!

Thanks for contributing on Utopian.
We’re already looking forward to your next contribution!

Get higher incentives and support Utopian.io!
Simply set @utopian.pay as a 5% (or higher) payout beneficiary on your contribution post (via SteemPlus or Steeditor).

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

Vote for Utopian Witness!

Coin Marketplace

STEEM 0.28
TRX 0.11
JST 0.034
BTC 66272.75
ETH 3183.00
USDT 1.00
SBD 4.09