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 kebab case for locked submissions endpoint #95

Merged
merged 2 commits into from
Oct 11, 2024

Conversation

Strohgelaender
Copy link
Contributor

Artemis PR ls1intum/Artemis#9210 changes endpoints to always use kabab case REST urls. I checked every changed endpoint any only found one used in this project, which I changed accordingly.

@Strohgelaender Strohgelaender requested a review from a team as a code owner August 16, 2024 07:49
@dfuchss
Copy link
Member

dfuchss commented Aug 16, 2024

Thank you in advance for the PR :) After the PR is merged to Artemis, we certainly merge this PR as well :)

# Conflicts:
#	src/main/java/edu/kit/kastel/sdq/artemis4j/client/CourseDTO.java
Copy link

@github-actions github-actions bot added the stale label Sep 3, 2024
@Luro02 Luro02 removed the stale label Sep 3, 2024
@github-actions github-actions bot added the stale label Sep 11, 2024
@dfuchss dfuchss removed the stale label Sep 11, 2024
@github-actions github-actions bot added the stale label Sep 19, 2024
@kit-sdq kit-sdq deleted a comment from github-actions bot Sep 19, 2024
@dfuchss dfuchss removed the stale label Sep 19, 2024
@github-actions github-actions bot added the stale label Sep 27, 2024
@dfuchss dfuchss added no-stale and removed stale labels Sep 27, 2024
@kit-sdq kit-sdq deleted a comment from github-actions bot Sep 27, 2024
@kit-sdq kit-sdq deleted a comment from github-actions bot Sep 27, 2024
@kit-sdq kit-sdq deleted a comment from github-actions bot Sep 27, 2024
@Strohgelaender
Copy link
Contributor Author

The Artemis-PR finally got merged, so this PR is now also ready to merge.

@dfuchss dfuchss merged commit aa610f1 into kit-sdq:main Oct 11, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

4 participants