Skip to content

[PROD RELEASE] 06/22 - #288

Merged
jmgasper merged 3 commits into
masterfrom
develop
Jun 24, 2026
Merged

[PROD RELEASE] 06/22#288
jmgasper merged 3 commits into
masterfrom
develop

Conversation

@vas3a

@vas3a vas3a commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

This pull request updates the challenge review context feature to allow creating a review context for any existing challenge, not just those in DRAFT or REGISTRATION phases. However, updating a review context is still restricted to challenges in DRAFT status or with an open registration phase. The changes also add comprehensive unit tests for these behaviors and clarify related error messages.

Access control and validation changes:

  • Creating a challenge review context is now allowed for any existing challenge, removing the previous restriction to DRAFT or REGISTRATION phases. [1] [2]
  • Updating a challenge review context remains restricted to challenges in DRAFT status or with an open registration phase, with the validation method and error messages updated for clarity. [1] [2] [3]

Testing improvements:

  • Added comprehensive unit tests in challenge-review-context.service.spec.ts to verify the new and existing behaviors for creating and updating review contexts under different challenge statuses and phases.

@vas3a
vas3a requested review from jmgasper and kkartunov June 22, 2026 07:01
@vas3a vas3a changed the title [PROD RELEASE] [PROD RELEASE] 06/22 Jun 22, 2026
@jmgasper
jmgasper merged commit 9618fc4 into master Jun 24, 2026
8 checks passed
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