Aggiungere comando per inviare avvisi nel canale "News"

MozillaItalia / mozitahub_bot

Il bot di Mozilla Italia Hub.

Creative Commons Zero v1.0 Universal

5 stars 8 forks source link

Aggiungere comando per inviare avvisi nel canale "News" #44

Closed Sav22999 closed 4 years ago

Sav22999 commented 4 years ago

Aggiungere un comando che permetta (agli admin!) di inviare avvisi sul canale "Mozilla Italia News", un'idea può essere:

/admin mozitanews -> Invia il messaggio sul canale
/admin moziranews preview -> Mostra un anteprima del messaggio sul bot, ma NON invia al canale

GianluigiMemoli commented 4 years ago

Per quanto riguarda un'eventuale eccezione durante la preview andrebbe utilizzata la Except23 o definita una nuova?

Sav22999 commented 4 years ago

@GianluigiMemoli meglio definirne una nuova, così eventualmente sappiamo esattamente dove si è verificata l'eccezione (invece di avere 2 possibili casi). Grazie per il tuo contributo

GianluigiMemoli commented 4 years ago

Così è stato fatto nella PR. Grazie a voi per la possibilità di contribuire 🚀

Sav22999 commented 4 years ago

@GianluigiMemoli ok, dopo molti millenni ecco che rispondo. Dunque, alcune modifiche:

Se non sono presenti canali, mostra comunque avviso che "Inviato correttamente", invece deve mostrare "Non ci sono canali"
Se non scrivo nulla (/admin mozitanews) non restituisce alcun messaggio di errore -> prevedere questo
/admin mozitanews preview non funziona (invia il testo "preview"). Secondo me bisogna mettere prima "preview" e poi "mozitanews" -> "/admin preview mozitanews"
Se scrivo "/admin canale HHH" non mostra messaggio di errore -> prevedere la condizione "else"

GianluigiMemoli commented 4 years ago

@Sav22999 L'avevo visto ma è stato un periodaccio per via della sessione e mi è sfuggito di mente. Vedo di rimediare entro una settimana, sono pure sviste che potevo evitare se avessi rifletutto sugli use case

Sav22999 commented 4 years ago

@GianluigiMemoli Ok ok, nessun problema 😄 lavoraci quando hai tempo, senza fretta

Sav22999 commented 4 years ago

Dal momento che @GianluigiMemoli non ha più risposto o continuato il lavoro, passo la palla a @dag7dev e gli assegno sia la PR #45, sia questo issue

dag7dev commented 4 years ago

Grazie @Sav22999 , va benissimo! Ci penso io!

Mte90 commented 4 years ago

Direi che può riprendere il codice di @GianluigiMemoli e proseguire da li con le modifiche con una nuova pr per fare prima.

Sav22999 commented 4 years ago

@Mte90 Sì, esatto, gli ho assetato anche la PR (come ho specificato nel mio commento precedente)