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_fatherbot #113

Closed
foxyseta opened this issue Sep 21, 2023 · 3 comments · Fixed by #136
Closed

/help_fatherbot #113

foxyseta opened this issue Sep 21, 2023 · 3 comments · Fixed by #136
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@foxyseta
Copy link
Member

foxyseta commented Sep 21, 2023

Il comando help autogenera un elenco dei comandi disponibili nel formato:

/nomecomando - Descrizione del comando

Questo formato è utile per noi utenti umani, perché / rende il nome del comando interattivo.

FatherBot, invece, richiede il formato

nomecomando - Descrizione del comando

per specificare quali sono i comandi che il bot mette a disposizione. Per semplificare il lavoro di chi deve comunicare i nuovi comandi del bot a FatherBot, sarebbe utile aggiungere un comando /help_fatherbot il cui risultato sia direttamente inoltrabile a FatherBot senza modifiche (perché già nel formato corretto).

Si tratta quindi di un /help senza / prefisso nel risultato.

Tenere in considerazione #118.

@foxyseta foxyseta added enhancement New feature or request good first issue Good for newcomers labels Sep 21, 2023
@VaiTon
Copy link
Member

VaiTon commented Sep 29, 2023

Magari escluderei il comando in se dalla lista, in modo tale che sia invocabile solo se se ne conosce l'esistenza

@foxyseta
Copy link
Member Author

"Security by obscurity" (da noi è inteso in senso sempre negativo)? Per me rende meno confusionario metterlo pubblico, e non si sa in futuro chi gestisca la nostra o altre istanze del bot. Inoltre non causa spam perché ha la stessa lunghezza di /help.

@VaiTon
Copy link
Member

VaiTon commented Sep 29, 2023

Più che security era giusto per non aggiungere un comando che servirebbe solo per fini amministrativi, però va bene anche non nasconderlo

@foxyseta foxyseta added the help wanted Extra attention is needed label Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request 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