You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
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.
newSendMessageToChannelUseCase(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 👍
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.
The text was updated successfully, but these errors were encountered: