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 ability to POST language executables via the API. #2129

Merged
merged 5 commits into from
Aug 27, 2023

Conversation

meisterT
Copy link
Member

This makes preparation of contests easier without modifying DOMjudge's source code to update default settings.

This makes preparation of contests easier without modifying
DOMjudge's source code to update default settings.
Copy link
Member

@nickygerritsen nickygerritsen left a comment

Choose a reason for hiding this comment

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

We might want to add a test?

webapp/src/Controller/API/LanguageController.php Outdated Show resolved Hide resolved
@nickygerritsen
Copy link
Member

I like the idea. This also makes #2108 obsolete

@meisterT
Copy link
Member Author

We might want to add a test?

Done

@meisterT meisterT merged commit 18bb708 into DOMjudge:main Aug 27, 2023
18 checks passed
meisterT added a commit to meisterT/domjudge that referenced this pull request Sep 3, 2023
This is an improvement on top of DOMjudge#2129, since you do not need to check
in ZIP files into a ccsconfig repository, but directories which are
zipped up on the fly.
This makes changing and reviewing changes much easier.
meisterT added a commit that referenced this pull request Sep 3, 2023
This is an improvement on top of #2129, since you do not need to check
in ZIP files into a ccsconfig repository, but directories which are
zipped up on the fly.
This makes changing and reviewing changes much easier.
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.

2 participants