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 spätzle liveticker #261

Merged
merged 1 commit into from
Jul 13, 2023
Merged

Add spätzle liveticker #261

merged 1 commit into from
Jul 13, 2023

Conversation

leonwind
Copy link

  • I made sure that the arrays in routes.php are still alphabetically sorted. ✨

As I am a big Spätzle Fan, I always want to know when I can get my favorite Mensa food again!

Very minimal website which would get added: https://leonwind.github.io/spaetzle-ticker/

Copy link
Sponsor Member

@joschahenningsen joschahenningsen left a comment

Choose a reason for hiding this comment

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

Looks fun :D

If you want, you can omit the resource heavy web scraping in the browser and utilize the eat-api instead: https://github.com/TUM-Dev/eat-api#api

@joschahenningsen joschahenningsen merged commit 8da3a93 into TUM-Dev:master Jul 13, 2023
1 check passed
@leonwind
Copy link
Author

Thanks for merging so quickly 🎉

Cool, didn't know there was a public API available! :o
Will take a look at it if I have some time

Either way, is there no CD anymore? Thought this deploys automatically, or did I break something?
spaetzle.tum.sexy is still not redirecting

If there is no automatic CD, then no rush, just wanted to check in if there is something broken :)

@joschahenningsen
Copy link
Sponsor Member

We do have cd in place 🤔

@CommanderStorm Do you know why argo didn't update the pod?
image

@CommanderStorm
Copy link
Member

We don't restart the deployment though.
=> Deployment targeting "latest" stays deployed, even though we have updated latest.

I have a plan on how to refactor this out, but this is an after-exams project

@joschahenningsen
Copy link
Sponsor Member

Any way I can assist here?

@CommanderStorm
Copy link
Member

CommanderStorm commented Jul 13, 2023

Plan is

'sp18' => [
'description' => 'Studienplan B.Sc. Informatik beginn WS 18/19',
'target' => 'https://www.in.tum.de/fuer-studierende/bachelor-studiengaenge/informatik/studienplan/studienbeginn-wise-201819-bis-sose-2021/',
],
'spaetzle' => [
Copy link
Member

Choose a reason for hiding this comment

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

The indentation here is seemingly wrong.

@kordianbruck
Copy link
Member

For now you can recreate (replace) the deployment in argocd and it will restart the pod at the newest latest.

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.

4 participants