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

Add timetables config file #23

Merged
merged 3 commits into from
Nov 22, 2023
Merged

Add timetables config file #23

merged 3 commits into from
Nov 22, 2023

Conversation

boozec
Copy link
Contributor

@boozec boozec commented Nov 22, 2023

Seguendo quanto discusso in cartabinaria/informabot#138 (comment) questa PR aggiunge un file per la gestione degli orari delle lezioni dei corsi di laurea.

@foxyseta
Copy link
Member

Perfetto. Potremmo includere anche gli altri CdL citati in degrees.json? Perché il senso della issue iniziale era includere loro.

@boozec
Copy link
Contributor Author

boozec commented Nov 22, 2023

Perfetto. Potremmo includere anche gli altri CdL citati in degrees.json? Perché il senso della issue iniziale era includere loro.

Yes! Decidiamo quali così si aggiorna anche questo timetables.json

@foxyseta
Copy link
Member

degree.json è già aggiornato quindi basta fare coincidere i due file. A meno che tu non voglia fonderli in una cosa sola.

@boozec
Copy link
Contributor Author

boozec commented Nov 22, 2023

A meno che tu non voglia fonderli in una cosa sola.

Preferirei di no visto che timetables.json ha una logica già complessa (nome, tipologia, curriculum) che poi è molto simile a quella di prima per le lezioni. Unirli vorrebbe dire spostare tutta la logica in degrees.json che, nel caso di duplicati, vorrebbe dire scrivere tanta roba in più.

@foxyseta
Copy link
Member

Va benissimo lasciare divise allora. Direi che a timetables.json mancavano AI, triennale di ingegneria e magistrale di Cesena.

@boozec
Copy link
Contributor Author

boozec commented Nov 22, 2023

Ho inserito anche triennale in informatica per il managament e la magistrale in ingegneria informatica

@foxyseta foxyseta merged commit fb24d76 into cartabinaria:main Nov 22, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants