Add support for bot to create webhook from message #3
Conversation
smiile8888
commented
Jul 24, 2020
smiile8888
commented
- command supports to create the channel webhook
- add command support to create webhook - add function to create webhook and interact back to author of message
- update name of the app
| const dotenv = require('dotenv'); | ||
| const movie = require('./modules/movie-fetch.js'); | ||
|
|
||
| dotenv.config(); |
There was a problem hiding this comment.
Do not we need it?
There was a problem hiding this comment.
I import the config via command line so that we don't have to clean up the code when we deploy on to Heroku.
| }).catch(() => { msg.reply("Movie not found!"); }); | ||
| } | ||
|
|
||
| if (cmd.startsWith(cli.WEBHOOK)) { |
There was a problem hiding this comment.
What is the pros using startsWith over ===
There was a problem hiding this comment.
Because it receives the arguments passed by the message (i.e., webhook name and avatar image URL) but has to start with /createHook
There was a problem hiding this comment.
startsWith is misleading cuz cmd is already processed and has the same length as any cli you want to compare.
There was a problem hiding this comment.
There was a problem hiding this comment.
What would you suggest handling this?
| let args = msg.content.split(" ").slice(1); | ||
| if (!args.length) { | ||
| return msg.reply("Give me bot name and avatar image (optional)!"); | ||
| } |
There was a problem hiding this comment.
Why not work on processMsg? ==> we should separate the work?
2 participants
