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

Lezioni per generici corsi di laurea #138

Merged
merged 21 commits into from
Nov 27, 2023
Merged

Lezioni per generici corsi di laurea #138

merged 21 commits into from
Nov 27, 2023

Conversation

boozec
Copy link
Contributor

@boozec boozec commented Nov 20, 2023

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 #122

@boozec boozec force-pushed the buttons-for-lessons branch from 1e14b20 to 6ddead4 Compare November 20, 2023 19:52
@foxyseta
Copy link
Member

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
Copy link
Contributor Author

boozec commented Nov 21, 2023

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
Copy link
Member

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

@boozec
Copy link
Contributor Author

boozec commented Nov 24, 2023

Ho aggiunto anche i test. Bisogna giusto inserire le modifiche di cartabinaria/config#25, fare un check generale degli altri corsi e revisionare il codice :) Tolgo la bozza 🎉

@boozec boozec marked this pull request as ready for review November 24, 2023 10:59
@foxyseta
Copy link
Member

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

@foxyseta foxyseta added the enhancement New feature or request label Nov 24, 2023
@boozec
Copy link
Contributor Author

boozec commented Nov 24, 2023

Hai collaudato gli altri CdL anche a mano?

json/actions.json Outdated Show resolved Hide resolved
Also, fix the mod for arrays of weekdays and months
@boozec boozec force-pushed the buttons-for-lessons branch from 654f340 to 22e8102 Compare November 27, 2023 18:38
@boozec boozec requested a review from foxyseta November 27, 2023 19:18
@foxyseta foxyseta merged commit 140c84f into main Nov 27, 2023
4 checks passed
@boozec boozec deleted the buttons-for-lessons branch November 27, 2023 21:31
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
Status: Done
Development

Successfully merging this pull request may close these issues.

/lezionioggi e /lezionidomani per altri cdl
2 participants