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

/rappresentanti #146

Closed
Jabbar03 opened this issue Nov 28, 2023 · 5 comments · Fixed by #150
Closed

/rappresentanti #146

Jabbar03 opened this issue Nov 28, 2023 · 5 comments · Fixed by #150
Assignees
Labels
enhancement New feature or request

Comments

@Jabbar03
Copy link
Contributor

Jabbar03 commented Nov 28, 2023

Questo comando dovrebbe taggare tutti i rappresentanti del corso di Informatica della Triennale e Magistrale di Bologna.

@Jabbar03 Jabbar03 added the enhancement New feature or request label Nov 28, 2023
@Jabbar03 Jabbar03 self-assigned this Nov 28, 2023
@samuelemusiani
Copy link
Member

Direi decisamente meglio della proposta di prima :)

Essendo ora il bot legato a tutto il disi come vogliamo elencare i rappresentanti? Perché ovviamente noi non copriamo ne ingegneria ne cesena, di conseguenza il comando dovrebbe poterli elencare tutti.

@Jabbar03
Copy link
Contributor Author

Pensavo un po' come i mantainer. Fare un campo che identifica il "gruppo" di cui fai parte e uno che elenchi quali sono le persone. Se vogliamo differenziarli all'interno di un "gruppo" per anno andranno modificati ogni anno, cosa abbastanza scomoda. Perciò direi di mettere solo un lista dato che qua gli array non vengono apprezzati.

@samuelemusiani
Copy link
Member

Per me ci può stare, procedi pure. In questo caso per le persone comunque l'array va benissimo ahaha

@Jabbar03
Copy link
Contributor Author

Ho dei dubbi su come facciamo a distingure le chiamate del bot perchè non possiamo sempre stampare tutti i rappresentanti, quando facciamo il comando dovremmo anche scrivere di quale corso tipo /rappresentanti@(codicecorso).
I miei dubbi sono sul fatto che @(codicecorso) com'è codificato? è parte del comando?

@foxyseta
Copy link
Member

Siccome vogliamo renderlo il più intuitivo da usare configlio bottoni nella UI (tastiera o tastiera in linea). Confrontati @boozec magari che sta studiando la stessa cosa per il comando /lezioni che puoi già collaudare in produzione.

@samuelemusiani samuelemusiani linked a pull request Dec 5, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants