Skip to content

Comments

Move review POST endpoint to review section#473

Merged
PGijsbers merged 3 commits intodevelopfrom
maint/move-submission-endpoints
Mar 10, 2025
Merged

Move review POST endpoint to review section#473
PGijsbers merged 3 commits intodevelopfrom
maint/move-submission-endpoints

Conversation

@PGijsbers
Copy link
Contributor

@PGijsbers PGijsbers commented Mar 6, 2025

Change

Moves the reviewing endpoint to the reviewing section.
Will also move the submit & retract over, but that's better after #471 since that one already has pending changes to submissions.

How to Test

Covered by test, no new tests needed as there is no change in functionality. Can try in the front end.

Checklist

  • Tests have been added or updated to reflect the changes, or their absence is explicitly explained.
  • Documentation has been added or updated to reflect the changes, or their absence is explicitly explained.
  • A self-review has been conducted checking:
    • No unintended changes have been committed.
    • The changes in isolation seem reasonable.
    • Anything that may be odd or unintuitive is provided with a GitHub comment explaining it (but consider if this should not be a code comment or in the documentation instead).
  • All CI checks pass before pinging a reviewer, or provide an explanation if they do not.

Related Issues

@PGijsbers PGijsbers added the enhancement New feature or request label Mar 6, 2025
@PGijsbers PGijsbers requested a review from Taniya-Das March 6, 2025 15:57
Copy link
Collaborator

@Taniya-Das Taniya-Das left a comment

Choose a reason for hiding this comment

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

Looks good.

@PGijsbers PGijsbers merged commit 79ec98d into develop Mar 10, 2025
1 check passed
@PGijsbers PGijsbers deleted the maint/move-submission-endpoints branch March 10, 2025 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants