Skip to content

Remove score edition feature#2342

Merged
Didayolo merged 19 commits into
developfrom
remove-admin-score-edit
May 12, 2026
Merged

Remove score edition feature#2342
Didayolo merged 19 commits into
developfrom
remove-admin-score-edit

Conversation

@Didayolo
Copy link
Copy Markdown
Member

@Didayolo Didayolo commented Apr 17, 2026

Description

Remove the possibility to edit the scores of a submission in the "ADMIN" tab. This option was available to all competition organizers and simply break the good practices for a scientific benchmark platform.

Capture d’écran 2026-04-17 à 19 45 52

Installation

I needed to run the following commands for the changes to take effect:

npm run build-riot
docker compose exec django ./manage.py collectstatic --noinput

Issues this PR resolves

A checklist for hand testing

  • Check that the "ADMIN" tab disappear
  • Check that the change did not break anything else

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • CircleCi tests are passing
  • Ready to merge

Didayolo and others added 18 commits May 12, 2026 10:58
* Add permission check to download method

* Flake8

* Check for 403 error, not 404

* Fix solution download
Bumps [django](https://github.com/django/django) from 5.2.12 to 5.2.13.
- [Commits](django/django@5.2.12...5.2.13)

---
updated-dependencies:
- dependency-name: django
  dependency-version: 5.2.13
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [cryptography](https://github.com/pyca/cryptography) from 46.0.6 to 46.0.7.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@46.0.6...46.0.7)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-version: 46.0.7
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [pytest](https://github.com/pytest-dev/pytest) from 9.0.2 to 9.0.3.
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest@9.0.2...9.0.3)

---
updated-dependencies:
- dependency-name: pytest
  dependency-version: 9.0.3
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [pillow](https://github.com/python-pillow/Pillow) from 12.1.1 to 12.2.0.
- [Release notes](https://github.com/python-pillow/Pillow/releases)
- [Changelog](https://github.com/python-pillow/Pillow/blob/main/CHANGES.rst)
- [Commits](python-pillow/Pillow@12.1.1...12.2.0)

---
updated-dependencies:
- dependency-name: pillow
  dependency-version: 12.2.0
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Copy link
Copy Markdown
Collaborator

@IdirLISN IdirLISN left a comment

Choose a reason for hiding this comment

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

rebased and coflicts resolved.
Approved.

**When modifying something inside of a riot file, you just need to do docker compose restart and it takes the changes.

@Didayolo Didayolo merged commit 128acc4 into develop May 12, 2026
2 checks passed
@Didayolo Didayolo deleted the remove-admin-score-edit branch May 12, 2026 09:43
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.

submission results editor displays only one task

3 participants