search

csunibo / config

Una raccolta centralizzata delle configurazioni per CSUnibo.
https://csunibo.students.cs.unibo.it/wiki/infrastruttura/configurazioni/index.html
GNU Affero General Public License v3.0
0 stars 3 forks source link

Update timetables.json #27

Closed boozec closed 7 months ago

boozec commented 7 months ago

Come richiesto in https://github.com/csunibo/informabot/pull/138/#pullrequestreview-1748516724 ho rimesso il title e fallbackText. Però, dato che /lezioni è gestito tramite callback e non tramite l'interfaccia del comando stesso, ho dovuto salvare titolo e testo d'errore per ogni timetable. È un pelo ridondante ma al momento, senza stravolgere tutto, mi è venuto in mente solo questo :\

foxyseta commented 7 months ago

Se metti title e falbacktext in actions.json sono in un posto solo. E name puoi lasciarlo qui.

boozec commented 7 months ago

Se metti title e falbacktext in actions.json sono in un posto solo. E name puoi lasciarlo qui.

Eh sì, però non posso richiamare la variabile dell'azione da bot/callbacks.go. Oppure sì(?)

foxyseta commented 7 months ago

Oddio non ho seguito il codice quindi non so come vada ristrutturato ma non vedo perché no. Se provi e vedi che hai difficoltà posso guardarci pure io

foxyseta commented 7 months ago

Non sto parlando di linkare json ma del bot che vede entrambe le stringhe da due json diversi e fa l'interpolazione. Scusa se non sono molto chiaro ahahahahah

boozec commented 7 months ago

D'accordo, ho fatto un redesign generale usando il medesimo modo dei comandi ma per le callback, così da potere usare lo stesso approccio per eventuali comandi (https://github.com/csunibo/informabot/pull/138/files#diff-20a35b72edc4f564a5399ff8e89a042559497ae95dccaaff8adf773e0e1e55fd) :tada:

Chiudo questa PR perché il title e fallbackText stanno sui comandi (di nuovo) :)