Skip to content
This repository has been archived by the owner on Mar 22, 2024. It is now read-only.

Discord send message #9

Closed
Murmeldyret opened this issue Mar 24, 2022 · 1 comment · Fixed by #111
Closed

Discord send message #9

Murmeldyret opened this issue Mar 24, 2022 · 1 comment · Fixed by #111
Assignees
Labels
Backend Node, Express, Databases & other nonvisual elements Frontend HTML, CSS & other visual elements Priority medium Not immediately important, but very relevant Task normal Takes average knowledge and time to do

Comments

@Murmeldyret
Copy link
Collaborator

An action that send a message to a discord message channel. This should be done from an action in the dashboard front end. Further, there should be a backend implementation that communicates with discord.

@Murmeldyret Murmeldyret added Frontend HTML, CSS & other visual elements Backend Node, Express, Databases & other nonvisual elements Priority medium Not immediately important, but very relevant Task normal Takes average knowledge and time to do labels Mar 24, 2022
@Murmeldyret Murmeldyret added this to the Minimal Viable Product milestone Mar 24, 2022
@Aavild
Copy link
Collaborator

Aavild commented Apr 7, 2022

Links:
https://discordjs.guide/popular-topics/webhooks.html

When the bot gets added to a server create a webhook and save the link to the database.
When the user is online the webhook is enabled and when the user logs off the webhook is turned off
Create client side way of typing in messages
Create serverside a listener who sanitizes the message and sends it to discord

Q: Why don't we just have the bot send messages too? A: Webhooks allows changing of names and is easier to implement

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Backend Node, Express, Databases & other nonvisual elements Frontend HTML, CSS & other visual elements Priority medium Not immediately important, but very relevant Task normal Takes average knowledge and time to do
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants