plane icon indicating copy to clipboard operation
plane copied to clipboard

[SILO-820] fix: update serializer for module detail API endpoint to use ModuleUpdateSerializer

Open dheeru0198 opened this issue 1 week ago • 2 comments

Description

This fixes the issue with adding memebers to module for API

Type of Change

  • [x] Bug fix (non-breaking change which fixes an issue)
  • [ ] Feature (non-breaking change which adds functionality)
  • [ ] Improvement (change that would cause existing functionality to not work as expected)
  • [ ] Code refactoring
  • [ ] Performance improvements
  • [ ] Documentation update

Screenshots and Media (if applicable)

Test Scenarios

References


[!NOTE] Aligns the module update endpoint with the correct update schema.

  • In ModuleDetailAPIEndpoint.patch, replace ModuleSerializer with ModuleUpdateSerializer for partial updates
  • Keeps OpenAPI request/response consistent with actual behavior and enables correct validation for update fields (e.g., members)

Written by Cursor Bugbot for commit 451ec266d5e7a1f5e0da7f17477e53d762dc8683. This will update automatically on new commits. Configure here.

Summary by CodeRabbit

  • Bug Fixes
    • Enhanced module update processing to ensure accurate handling of partial updates.

✏️ Tip: You can customize this high-level summary in your review settings.

dheeru0198 avatar Jan 07 '26 11:01 dheeru0198