Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update timetables.json #27

Closed
wants to merge 2 commits into from
Closed

Update timetables.json #27

wants to merge 2 commits into from

Conversation

boozec
Copy link
Contributor

@boozec boozec commented Nov 27, 2023

Come richiesto in cartabinaria/informabot#138 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 :\

@boozec boozec requested a review from foxyseta November 27, 2023 14:26
@foxyseta
Copy link
Member

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

@boozec
Copy link
Contributor Author

boozec commented Nov 27, 2023

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
Copy link
Member

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
Copy link
Member

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
Copy link
Contributor Author

boozec commented Nov 27, 2023

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) 🎉

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

@boozec boozec closed this Nov 27, 2023
@boozec boozec deleted the boozec-patch-1 branch November 27, 2023 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants