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

/help: Bad Request: message is too long #118

Closed
foxyseta opened this issue Oct 29, 2023 · 6 comments · Fixed by #125
Closed

/help: Bad Request: message is too long #118

foxyseta opened this issue Oct 29, 2023 · 6 comments · Fixed by #125
Assignees
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@foxyseta
Copy link
Member

Il comando /help non dà più risposte, ma non ci sono log di errore. Suppongo che la causa sia qualche modifica del json che porta a non poter più raccogliere l'elenco di tutti i valori dei campi description.

@foxyseta foxyseta added bug Something isn't working help wanted Extra attention is needed good first issue Good for newcomers labels Oct 29, 2023
@bogo8liuk
Copy link
Contributor

Facendo prove in locale, la seguente chiamata:
https://github.com/csunibo/informabot/blob/7012beff70c7929c619f864092411a2dde9d6648/utils/util.go#L17C8-L17C8
mi dà questo errore: Bad Request: message is too long

@foxyseta
Copy link
Member Author

foxyseta commented Nov 6, 2023

Penso che potremmo filtrare via i comandi degli insegnamenti (che sono la stragrande maggioranza e tutti uguali). A livello di codice come soluzione è facilissima. Inoltre accorciare ancora più la descrizione di ogni comando è dura.

L'alternativa sarebbe mandare una sequenza di messaggi, ma comunque vista la grande quantità di comandi sarebbe una pessima esperienza utente (la chat si "intasa" con un dizionario lunghissimo ed è difficile da consultare).

This was referenced Nov 6, 2023
@foxyseta foxyseta changed the title /help rotto /help: Bad Request: message is too long Nov 6, 2023
@foxyseta
Copy link
Member Author

foxyseta commented Nov 6, 2023

A proposito: penso che risposte dell'api come Bad request: message is too long sarebbe molto ma molto importante loggarle, quindi potremmo fare pure quello.

@bogo8liuk
Copy link
Contributor

Penso che potremmo filtrare via i comandi degli insegnamenti (che sono la stragrande maggioranza e tutti uguali). A livello di codice come soluzione è facilissima. Inoltre accorciare ancora più la descrizione di ogni comando è dura.

L'alternativa sarebbe mandare una sequenza di messaggi, ma comunque vista la grande quantità di comandi sarebbe una pessima esperienza utente (la chat si "intasa" con un dizionario lunghissimo ed è difficile da consultare).

Sono d'accordo, un ulteriore miglioramento potrebbe essere aggiungere comandi di help specifici che mandano la lista degli insegnamenti di un singolo cdl

@bogo8liuk bogo8liuk self-assigned this Nov 6, 2023
@foxyseta
Copy link
Member Author

foxyseta commented Nov 6, 2023

Ah sisi ci sta tipo /insegnamenti_inf /insegnamenti_ing_mag...

Per fare questo secondo passo aspetterei la chiusura di #111.

@foxyseta foxyseta linked a pull request Nov 7, 2023 that will close this issue
@foxyseta
Copy link
Member Author

foxyseta commented Nov 7, 2023

Ho Chiuso perché #125 ha risolto il problema in sé. @bogo8liuk se pensi che valga la pena di avere anche un help per ogni CdL puoi aprire una nuove issue senza problemi.

@foxyseta foxyseta closed this as completed Nov 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants