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

Rimuovere _hard coding_ a ID gruppi #121

Closed
foxyseta opened this issue Nov 3, 2023 · 3 comments · Fixed by #154
Closed

Rimuovere _hard coding_ a ID gruppi #121

foxyseta opened this issue Nov 3, 2023 · 3 comments · Fixed by #154
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 Nov 3, 2023

https://github.com/csunibo/informabot/blame/f6c357eb3bdf69f380acbda61da9a3cd95c4f235/model/controller.go#L202

Questo abominio sono colpevole io di averlo distrattamente recensito positivamente a un certo punto forse. Pare fosse presente anche prima della riscrittura in Go, e quindi non mi sono messo a risalire all'origine con git blame (@Bitrey ?).

Gli id dei gruppi sono presenti in json/settings.json e non andrebbero mai harcodati nel codice. Siccome abbiamo la cattiva abitudine di creare gruppi degli anni nuovi ogni volta (a differenza degli insegnamenti), potremmo smettere di usare settings.json e cercare la parola chiave "anno" nel titolo del gruppo forse, anche se è solo un'euristica alla buona.

@foxyseta foxyseta added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Nov 3, 2023
@samuelemusiani
Copy link
Member

Direi che questa è collegata a #128 e #129

@foxyseta
Copy link
Member Author

Già. Io fattorizzerei il controllo per i gruppi degli anni in modo che dopo sia richiamabile pure qui. Quindi prerequisito per questa issue diventa #129

@foxyseta
Copy link
Member Author

Ora che #129 è chiusa, io sostituirei quel controllo con una chiamata alla stessa funzione di utilità che viene usata là per capire se siamo in un gruppo annuale o meno, commentandola siccome Losco è andato via. @Bitrey sei d'accordo? Così non abbiamo più limiti per /ludopatico ma se mai ci fossero problemi scommentiamo il controllo e diventa bandito su ogni gruppo annuale.

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
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants