MozillaItalia / mozitahub_bot

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

Commands to send message on channels added #45

Closed GianluigiMemoli closed 4 years ago

GianluigiMemoli commented 4 years ago

Satisfies #44 issue's request

How?

Two new commands have been added to the already existing bot's functionalities. This commands are /admin mozitanews <message> and /admin mozitanews preview <message>. I hope the implementation style fits the other commands' implementations. I have also added two new Exceptions to the Eccezioni.md file to handle errors with this two new features.

How to use

Add a [channels] section to the config.ini file with a parameter news_channel set to the name of the channel.

Minor fix

While testing around I've found a misprint in frasi.json that I have fixed.

Sav22999 commented 4 years ago

@GianluigiMemoli l'userid del canale dovrebbe essere salvato dinamicamente in una lista json invece che nel file di configurazione (in questo modo, in futuro, si può integrare una funzionalità per aggiungere/rimuovere canali molto semplicemente)

GianluigiMemoli commented 4 years ago

@GianluigiMemoli l'userid del canale dovrebbe essere salvato dinamicamente in una lista json invece che nel file di configurazione (in questo modo, in futuro, si può integrare una funzionalità per aggiungere/rimuovere canali molto semplicemente)

Non ho ancora letto il tuo commit, questa modifica l'hai già apportata? In caso negativo me ne occupo in giornata.

Sav22999 commented 4 years ago

@GianluigiMemoli no no, ho fatto un'altra modifica.

GianluigiMemoli commented 4 years ago

@Sav22999 il messaggio andrebbe mandato a tutti i canali presenti nella lista o modifico il comando per far scegliere il canale specifico? Edit: Rileggendo la richiesta in effetti se l'upgrade è previsto per il futuro per ora lascio l'invio su ogni canale nella lista

Sav22999 commented 4 years ago

@GianluigiMemoli a tutti quelli presenti in lista 😉

Sav22999 commented 4 years ago

@GianluigiMemoli scusa se rispondo solamente ora, ho avuto alcuni altri impegni. Dunque, mi sembra perfetto. Ti va di integrare anche il comando per le seguenti azioni?

Aspetto tue notizie 😄

GianluigiMemoli commented 4 years ago

@Sav22999 Certo, posso occuparmene

Sav22999 commented 4 years ago

Perfetto, così è completo 😁

Sav22999 commented 4 years ago

@GianluigiMemoli chiedo venia, non deve essere "rimuovi" ma "elimina".

GianluigiMemoli commented 4 years ago

@Sav22999 di nulla, appena posso sostituisco

Sav22999 commented 4 years ago

@GianluigiMemoli Ho lasciato 7 giorni fa commento su #44, forse non ti è arrivata la notifica, quindi faccio ping qui.

Mte90 commented 4 years ago

Faccio un ping @GianluigiMemoli

dag7dev commented 4 years ago

@Sav22999 finisco io!

dag7dev commented 4 years ago

Closed by #47