TheOdinProject / odin-bot-v2

The bot that breathes life into our Discord community
ISC License
47 stars 74 forks source link

All relevant commands: Replace hardcoded ID's with ID's from config.js where possible #559

Open Asartea opened 1 week ago

Asartea commented 1 week ago

Complete the following REQUIRED checkboxes:

The following checkbox is OPTIONAL:


1. Description of the Feature Request: While a lot of channel/member ID's are already stored in config.js, they are usually still hardcoded in the reply strings. Changing them to instead import and interpolate them has 2 advantages: in the unlikely but possible event a channel gets deleted and recreated there is one source of truth to change, and more importantly <#${channels.FAQChannelId}> is a lot clearer to read at a glance than something like <#823266307293839401>

2. Acceptance Criteria:

CouchofTomato commented 1 week ago

@TheOdinProject/odin-bot Can someone take a look please.

01zulfi commented 1 week ago

@Asartea sounds good to me