Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Desacoplar comandos e mensagens do commandUseCaseResolver #64

Open
caioluis opened this issue Feb 17, 2023 · 2 comments · May be fixed by #78
Open

Desacoplar comandos e mensagens do commandUseCaseResolver #64

caioluis opened this issue Feb 17, 2023 · 2 comments · May be fixed by #78
Labels
🔰 bom primeiro issue Bom para iniciantes 🌟 melhoria Nova funcionalidade ou requisito

Comments

@caioluis
Copy link
Contributor

Contexto

Depois de uma alteração que fiz ao comando !ja, notei que as mensagens dos comandos estão todas acopladas ao resolver que lê os comandos. Para facilitar a manutenção, revisão do código e rapidez de aprovação de mudanças, sugiro que os comandos e suas respectivas mensagens sejam desacoplados do resto da lógica. Talvez podíamos fazer uma estrutura em formato json.

Acho que pode ser um bom first issue para alguém, mas se ninguém fizer, eu abro um PR e proponho mudanças.

@IvoPereira
Copy link
Member

Parece-me uma boa ideia, mas que depende sempre de algum contexto específico que é um pouco o que o commandUseCaseResolver tenta transparecer.

Uma coisa que se poderia fazer talvez é, permitir que o commandUseCaseResolver, lê-se por ex. um ficheiro com uma estrutura de dados JSON como sugeriste e verifica-se se existia um match de um dos comandos que simplesmente envia uma mensagem (p.ex. o !ja ou o !oc).

Se sim, pegava na mensagem associada ao comando e chamava o SendMessageToChannelUseCase.

new SendMessageToChannelUseCase(deps).execute({
      channelId: context.channelId,
      message: ":warning: Este servidor é APENAS para questões relacionadas com programação! :warning:", // replace with message from JSON
}),

O que achas?

Se quiseres, fica à vontade para enviares um PR com isso 👍

@IvoPereira IvoPereira added 🌟 melhoria Nova funcionalidade ou requisito 🔰 bom primeiro issue Bom para iniciantes labels Feb 21, 2023
@diomonogatari
Copy link
Contributor

@IvoPereira tb não me importo de fazer isto, é para aprender .ts 😎

diomonogatari added a commit to diomonogatari/discord-bot that referenced this issue Oct 26, 2024
@diomonogatari diomonogatari linked a pull request Oct 26, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔰 bom primeiro issue Bom para iniciantes 🌟 melhoria Nova funcionalidade ou requisito
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants