Skip to content

CSCEXAM-1690 Fixed collaborative assessment copy - #1463

Merged
lupari merged 1 commit into
masterfrom
CSCEXAM-1690
Oct 9, 2026
Merged

lupari merged 1 commit into
masterfrom
CSCEXAM-1690

Conversation

@lupari

@lupari lupari commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator
  • Receiving a collaborative exam assessment failed on the question parent foreign key since 6.4.3. Question ids of collaborative exams are generated by the collaboration service and do not refer to local questions. CSCEXAM-1561 started setting them as parents again.
  • Collaborative assessments are copied without question parents. Checking for existence alone is not enough as a generated id may match a local question by coincidence.
  • External assessments keep the parent relation (CSCEXAM-1561) if the question still exists locally, it may have been deleted meanwhile.
  • Added tests for collaborative assessment, deleted question and the parent relation of external assessment.

- Receiving a collaborative exam assessment failed on the question
parent foreign key since 6.4.3. Question ids of collaborative exams
are generated by the collaboration service and do not refer to local
questions. CSCEXAM-1561 started setting them as parents again.
- Collaborative assessments are copied without question parents.
Checking for existence alone is not enough as a generated id may
match a local question by coincidence.
- External assessments keep the parent relation (CSCEXAM-1561) if the
question still exists locally, it may have been deleted meanwhile.
- Added tests for collaborative assessment, deleted question and the
parent relation of external assessment.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@lupari
lupari merged commit d0b14d9 into master Oct 9, 2026
4 checks passed
@lupari
lupari deleted the CSCEXAM-1690 branch October 9, 2026 09:53
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.

1 participant