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

made sure that we are ready for the comming ConnecTUM API-Change #1281

Merged
merged 1 commit into from
Jul 21, 2024

Conversation

CommanderStorm
Copy link
Member

@CommanderStorm CommanderStorm commented Jul 3, 2024

Fixes #1273

The PR needs https://gitlab.campusonline.community/tum/connectum/-/merge_requests/90 merged first

Proposed Changes (include Screenshots if possible)

  • renamded stp_title_{en,de} => title_{en,de}
  • made stp_type optional

How to test this PR

  1. Let it import all calendar entries

How has this been tested?

  • see above

Checklist

  • I have updated the documentation / No need to update the documentation
  • I have run the linter

@CommanderStorm CommanderStorm marked this pull request as ready for review July 10, 2024 23:13
@CommanderStorm CommanderStorm force-pushed the calendar-fix branch 2 times, most recently from 408062e to c4328e2 Compare July 11, 2024 00:21
@CommanderStorm CommanderStorm force-pushed the calendar-fix branch 4 times, most recently from 121f17a to e749abc Compare July 20, 2024 21:51
@CommanderStorm
Copy link
Member Author

preemptive merge to ensure nav.tum.de is ready for tomorrows tumonline update

@CommanderStorm CommanderStorm enabled auto-merge (squash) July 21, 2024 17:06
@CommanderStorm CommanderStorm disabled auto-merge July 21, 2024 17:07
@CommanderStorm CommanderStorm merged commit 2419b82 into main Jul 21, 2024
10 checks passed
@CommanderStorm CommanderStorm deleted the calendar-fix branch July 21, 2024 17:07
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.

Calendar should show appointments
1 participant