Repository navigation
Fixed issues #6 #5 #4 and #2 - #13
Conversation
Pulls events from meetup and displays them
sethetter
left a comment
sethetter
left a comment
There was a problem hiding this comment.
All the functionality works great! Thanks for all these additions. Hopefully all the suggestions here are useful, let me know if you have any questions :)
Co-authored-by: Seth Etter <655500+sethetter@users.noreply.github.com>
Co-authored-by: Seth Etter <655500+sethetter@users.noreply.github.com>
Co-authored-by: Seth Etter <655500+sethetter@users.noreply.github.com>
Co-authored-by: Seth Etter <655500+sethetter@users.noreply.github.com>
|
I think I fixed all the suggestions. Thanks! |
| channel: event.channel, | ||
| text: jobsMessage, | ||
| }); | ||
| respondInThread(event, jobsMessage); |
There was a problem hiding this comment.
I'd just remove this! Better to keep things clean. We can always refer back to git history.
| ts: "1723835381.746579", | ||
| channel: BOT_TESTING_CHANNEL, | ||
| event_ts: "1626252625.000200", | ||
| event_ts: "1723832846.938799", |
There was a problem hiding this comment.
Since this will be the same thread that receives the replies every time this is used now, maybe it would be good to accept as another parameter to the command?
const threadTs = Deno.args[1];, then pass it in with simulate-message.ts "@bot my messsage" 123123123.123123. What do you think?
sethetter
left a comment
sethetter
left a comment
There was a problem hiding this comment.
This all looks good to go! Thanks for the contribution, and handling the feedback 😄
No description provided.