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

Fahrplan-Änderung #363

Merged
merged 6 commits into from
Sep 11, 2024
Merged

Fahrplan-Änderung #363

merged 6 commits into from
Sep 11, 2024

Conversation

cyildiz
Copy link
Contributor

@cyildiz cyildiz commented Sep 10, 2024

Der Semesterbeginn an der TDU wurde auf den 30. September verlegt. Daher ist eine Fahrplan-Änderung notwendig.

Der Semesterbeginn an der TDU wurde auf den 30. September verlegt. Daher ist eine Fahrplan-Änderung notwendig.
@cyildiz cyildiz requested a review from cagix as a code owner September 10, 2024 12:02
@cyildiz
Copy link
Contributor Author

cyildiz commented Sep 10, 2024

Ich habe die Fahrplan-Graphik ansatzweise aktualisiert. @cagix , Verbesserungsvorschläge?

Copy link
Member

@cagix cagix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cyildiz Die neue Grafik finde ich super! Danke :) Ich glaube, für die Lesbarkeit wäre eine horizontale Trennung in KW41 gut, dann verwechselt man das nicht mit den einzelnen Wochentrennern. Wie findest Du die Idee?

@cagix cagix self-assigned this Sep 10, 2024
@cagix cagix added this to the Start Winter 2024/25 milestone Sep 10, 2024
@cyildiz
Copy link
Contributor Author

cyildiz commented Sep 10, 2024

@cyildiz Die neue Grafik finde ich super! Danke :) Ich glaube, für die Lesbarkeit wäre eine horizontale Trennung in KW41 gut, dann verwechselt man das nicht mit den einzelnen Wochentrennern. Wie findest Du die Idee?

Du hast recht, schaut viel besser und verständlicher aus. Danke!

@cagix
Copy link
Member

cagix commented Sep 10, 2024

@cyildiz Wow, danke! Das sieht gut aus :)

Dann würde ich den PR bzw. den Branch hijacken und morgen die Fahrplantabellen entsprechend mit anpassen.

Ich hatte Dir eben per Mail nochmal eine Überlegung zu den Übungen geschickt. Im Prinzip sehe ich zwei Varianten. Schau mal, welche Du bevorzugst ... (ich habe eine leichte Präferenz für die spätere Variante, also die zuletzt geschriebene).

Soll ich bei Deinem Fahrplan auch die Übungen mit an die Vorlesungstabelle hängen, so wie ich das bei mir gemacht hatte? Oder sollen das besser zwei separate Tabellen bleiben? Das würde ich in diesem PR dann auch gleich noch mit erledigen...

@cyildiz
Copy link
Contributor Author

cyildiz commented Sep 10, 2024

@cagix , die spätere Variante passt gut finde ich.

Die Übungen in der Vorlesungtabelle zu haben würde ich auch vorziehen. Die Tabelle der TDU kann ich gerne übernehmen nachdem Du mit den anderen Anpassungen fertig bist.

@cagix
Copy link
Member

cagix commented Sep 11, 2024

@cyildiz Ich habe die Tabellen angepasst. Das ist ja ein furchtbares Gefummel :) Hoffentlich habe ich nix übersehen. Kannst Du bitte nochmal drauf schauen, ob das jetzt alles passt?

Bei Dir habe ich den Übungstag vom letzten Jahr genommen, das musst Du noch anpassen.

@cyildiz
Copy link
Contributor Author

cyildiz commented Sep 11, 2024

Passt alles super, vielen Dank! In einer Tabelle sieht das Ganze viel übersichtlicher aus.

Wegen der Übung habe ich nochmal nachgefragt. Sobald die Zeiten feststehen, trage ich sie in die Tabelle ein. Dazu kann ich eventuell später ein eigenes PR öffnen, oder?

@cagix
Copy link
Member

cagix commented Sep 11, 2024

Wegen der Übung habe ich nochmal nachgefragt. Sobald die Zeiten feststehen, trage ich sie in die Tabelle ein. Dazu kann ich eventuell später ein eigenes PR öffnen, oder?

ja, genau.

@cagix cagix merged commit a890533 into master Sep 11, 2024
2 checks passed
@cagix cagix deleted the fahrplan-aenderung branch September 11, 2024 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants