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

Update routes.php #327

Merged
merged 12 commits into from
Sep 22, 2024
Merged

Update routes.php #327

merged 12 commits into from
Sep 22, 2024

Conversation

laxerhd
Copy link

@laxerhd laxerhd commented Sep 18, 2024

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

I made some changes to the links on the frontpage (updated/removed some) and added my own tutorial website.

Copy link
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.

Amazing!
Seems like we're using one of the routes you changed in our tests. Can you adapt that test as well?

tests/General/MainTest.php Outdated Show resolved Hide resolved
laxerhd and others added 2 commits September 20, 2024 10:34
Changed enrol to course in test

Co-authored-by: Joscha Henningsen <[email protected]>
@kordianbruck kordianbruck merged commit 4ecb903 into TUM-Dev:master Sep 22, 2024
1 check passed
@laxerhd
Copy link
Author

laxerhd commented Sep 22, 2024

Great, thanks!

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.

3 participants