MozillaItalia / mozitahub_bot

Il bot di Mozilla Italia Hub.
Creative Commons Zero v1.0 Universal
5 stars 8 forks source link

v1.5.4: Vento di Freschezza - fresh wind #47

Closed dag7dev closed 4 years ago

dag7dev commented 4 years ago

Closes #44 - replaces #45

Sono stato costretto ad effettuare una nuova pr in quanto quella vecchia già presente non poteva essere utilizzata: troppe parti non funzionanti, troppi bug e troppo codice non mantenuto.

Ecco la lista dei cambiamenti:

- aggiunti nuovi comandi (quelli richiesti sono stati generalizzati per permettere una più efficiente gestione)
    - Gestione messaggi nei canali:
        - /admin messaggio preview |canale| |Messaggio da inviare in un canale| => Anteprima del messaggio da inviare, per verificare che tutto venga visualizzato correttamente
        - /admin messaggio | canale | |Messaggio da inviare in un canale in particolare|
        - /admin messaggio broadcast |Messaggio da inviare in tutti i canali|

    - Gestione canali:
        - /admin canale mostra
        - /admin canale aggiungi |Channel_name|
        - /admin canale elimina |Channel_name|

Originariamente erano stati richiesti:

/admin mozitanews <message>
/admin mozitanews preview <message>
/admin canale aggiungi
/admin canale mostra
/admin canale rimuovi

Tutto è stato possibile grazie anche al grandissimo @Sav22999 che ha avuto una pazienza infinita e ha testato il tutto!

Non è molto ma è un lavoro onesto

GG! Happy Hacktoberfest!

Sav22999 commented 4 years ago

Ok, faccio alcuni test e vi faccio sapere :smile:

Sav22999 commented 4 years ago

/admin messaggio -> Rinominarlo in /admin canale

"Messaggio" potrebbe trarre in inganno e far pensare che sia "Avviso"

Quindi avere:

/admin canale broadcast CIAO CIAO CIAO /admin canale aggiungi CIAO —> Mette in automatico la "@" /admin canale elimina CIAO /admin canale CIAO messaggio Messaggio /admin canale mostra

Sav22999 commented 4 years ago

Sto testando il bot e ho trovato alcune migliorie. Ne sto discutendo direttamente con Damiano in chat privata

dag7dev commented 4 years ago

Riporto per completezza il testo del commit qui:

- bugfixes
- renamed /admin messaggio in /admin canale: now they belong to a 'single' kind
- fixed: bot used to show improper error message if channel wasnt present
- improved: regardless of user input, everything is converted in lowercase
- improved: added @ in front of a username if needed
- added: config-sample.ini since it didn't exist