search

csunibo / informabot

A Telegram bot to assist other undergraduate Computer Science students at the University of Bologna
https://t.me/UniboInformabot
GNU General Public License v3.0
8 stars 12 forks source link

Lezioni per generici corsi di laurea #138

Closed boozec closed 10 months ago

boozec commented 10 months ago

Questa PR aggiunge la possibilità di avere le lezioni per qualsiasi corso di laurea. Tengo in Draft perché ci sto lavorando ancora su. Una volta finito basterà inserire i file di configurazione che fanno il map dei corsi di laurea che si vogliono supportare ai loro link. Inoltre aggiunge la gestione delle callback.

Resolves https://github.com/csunibo/informabot/issues/122

foxyseta commented 10 months ago

Il test fallisce perché non hai formattato il tuo nuovo json (il progetto usa prettier). Dubbio di nomenclatura: CDLS lo fa sembrare un po' un clone di json/config/degrees.json (se non lo vedi forse devi fare un rebase da main e aggiornare i sottomoduli).

boozec commented 10 months ago

Il test fallisce perché non hai formattato il tuo nuovo json (il progetto usa prettier). Dubbio di nomenclatura: CDLS lo fa sembrare un po' un clone di json/config/degrees.json (se non lo vedi forse devi fare un rebase da main e aggiornare i sottomoduli).

Pensavo a timetable.json o lectures.json, che dici?

foxyseta commented 10 months ago

timetables.json mi piace. Magari lo traslocherei nel sottomodulo csunibo/config siccome sono informazioni potenzialmente utili anche ad altre repo

boozec commented 10 months ago

Ho aggiunto anche i test. Bisogna giusto inserire le modifiche di https://github.com/csunibo/config/pull/25, fare un check generale degli altri corsi e revisionare il codice :) Tolgo la bozza :tada:

foxyseta commented 10 months ago

Accettata la PR citata. Dovresti poter inserire le modifiche senza problemi ora. Hai collaudato gli altri CdL anche a mano?

boozec commented 10 months ago

Hai collaudato gli altri CdL anche a mano?

Sì