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

Use default redirect when nothing matches. #89

Merged
merged 1 commit into from
Feb 17, 2023

Conversation

Wim-De-Clercq
Copy link
Contributor

Kheb die code ineens in een method gestoken. Dat maakt de code properder door te kunnen returnen ipv if-else nodig te hebben.

@coveralls
Copy link

coveralls commented Feb 17, 2023

Coverage Status

Coverage: 100.0%. Remained the same when pulling 779633b on feature/86_accept_header_filtering into 7600a1c on develop.

@Wim-De-Clercq Wim-De-Clercq force-pushed the feature/86_accept_header_filtering branch from 45697a7 to 779633b Compare February 17, 2023 14:09
Copy link
Contributor

@goessebr goessebr left a comment

Choose a reason for hiding this comment

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

Functie proper gedocumenteerd ook, I llike!

@goessebr goessebr merged commit d832a15 into develop Feb 17, 2023
@goessebr goessebr deleted the feature/86_accept_header_filtering branch February 17, 2023 14:30
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.

None yet

3 participants