Skip to content

Conversation

@kamilramocki
Copy link
Member

@kamilramocki kamilramocki commented Dec 29, 2025

Important

Enhances bulkUpdate in participants_attributes_controller.ts to validate participant IDs against event ID before updating attributes.

  • Behavior:
    • In bulkUpdate method of ParticipantsAttributesController, added validation to ensure all participantIds belong to the specified eventId.
    • Returns a bad request response if any participant does not belong to the event.
  • Code Changes:
    • Moved attribute query to occur before participant validation in bulkUpdate.
    • Removed redundant findOrFail for attribute after validation.

This description was created by Ellipsis for 52d4582. You can customize this summary. It will automatically update as commits are pushed.

@kamilramocki kamilramocki self-assigned this Dec 29, 2025
@kamilramocki kamilramocki linked an issue Dec 29, 2025 that may be closed by this pull request
@loudsheep loudsheep merged commit 9c60d16 into main Jan 11, 2026
5 checks passed
@loudsheep loudsheep deleted the fix/update-participant-attributes-controller branch January 11, 2026 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Naprawa ParticipantsAttributesController

3 participants