refactor: ♻️ teachings & degrees #20
Conversation
To improve generic degrees as discussed in cartabinaria/dynamik#122 and #17, I've created 2 separate files: `teachings.json`: organizes all the teachings by year, giving an id to each course; `degrees.json`: organizes all the degrees, taking for each year the courses referenced in teachings by id.
|
TLDR:
File
|
|
@ali-benny @foxyseta confermate? |
Non un array di coppie id + booleano
Queste includono anche un id univoco giusto? Che nel caso di lab deve coincidere con il nome della repo? E lab ha un campo groupchat che gli altri CdL propriamente detti non hanno? |
confermo |
A questo punto direi di fare due fields in teachings:
Forse sto capendo male io, ma l'id univoco non potrebbe essere il nome del repository? Riguardo alla groupchat, non dovrebbe essere specificata per tutti gli insegnamenti? |
Sisi infatti.
Penso di sì. Solo voi di ing-info fate eccezione quindi lascerei come ora questo campo. |
@VaiTon @ali-benny Si stava parlando dei CdL nella parte di messaggio a cui ho risposto, non di insegnamenti. |
ops, comunque sì, riconfermo la frase. |
|
TLDR2:
File
|
|
Siccome il pulsante della chat vogliamo già aggiungerlo al "CdL" |
|
OT: impostata questa PR come "bozza" siccome:
|
Mi sembrano ottime idee |
|
btw attualmente il "nome repo" è già assegnato al campo "url", possiamo sfruttare questo campo o lo duplichiamo in un nuovo "id"? |
NOTA: dividendo year così perdiamo però quali insegnamenti sono in quali anni... |
Come detto sopra io rimuoverei l'attuale campo
Ogni anno contiene un elenco di insegnamenti obbligatori e opzionali. Non basta vedere lì? |
Okay, ma così per aggiungere un nuovo insegnamento devo modificare sia degrees che teachings |
|
Vedi: #20 (comment) |
Sinceramente va bene così secondo me |
|
Ho menzionato l'ultimo TLDR dove ho integrato tutti i vostri commenti nella issue collegata in modo da documentare lo schema desiderato. @ali-benny se è troppo pesante/tedioso come lavoro dimmi pure che quando avrò più tempo potrò darti una mano. |
|
Domanda: ho riadattato i file in base al TLDR 2. |
|
Se ho capito bene |
|
Ho creato una branch https://github.com/csunibo/config/tree/stable dove aggiungere i nuovi insegnamenti nel vecchio formato finché dynamik non supporta quello nuovo. Questo nel caso ci mettiamo tanto ad aggiornare dynamik per qualche motivo o se la gente ha fretta di vedere i nuovi insegnamenti in produzione per motivi di pubbliche relazioni. @csunibo/ingegneria-informatica @csunibo/artificial-intelligence-1 questa modifica impatterà il leggermente il vostro flusso di lavoro per la creazione di nuove repo, ma aggiorneremo la documentazione. Nulla di preoccupante! |
|
@VaiTon durante il rebase non hai lasciato fuori uno dei due insegnamenti di IA? |
|
@foxyseta non ho fatto rebase ma merge, quindi mi sembra strano che abbia tolto roba. Potrebbe essere che non erano ancora stati convertiti al nuovo "formato"? |
|
Sì scusa devo smettere di usare rebase per intendere "merge/rebase da main verso branch secondaria". È sbagliato. Comunque l'idea è che tutto quello che è su main è nel vecchio formato, quindi chiunque faccia il merge per ora deve per forza riportare tutte le modifiche nel nuovo (in questo caso è solo un insegnamento quindi ci metti un secondo). |
|
Ho visto che hai specificato il |
sì, so di diverse persone al 3 che lo seguiranno |
|
Ottimo. ALlora lo aggiungerei agli opzionali di informatica in |
|
@ali-benny quando vuoi puoi unire a |
Per migliorare i corsi generici come discusso in cartabinaria/dynamik#122 e #17, ho creato 2 file separati:
teachings.json: organizza tutti gli insegnamenti per anno, assegnando un id a ogni corso;degrees.json: organizza tutti i corsi di laurea, prendendo per ogni anno gli insegnamenti, referenziati in teachings per id.Osservazione:
teachings.jsonmi è venuto naturale suddividerlo per anni dato che si trattava di insegnamenti, ho il dubbio però che forse l'idea originale era di organizzarlo per id insegnamento (?)This close #17