Skip to content

Conversation

@dorsha
Copy link
Member

@dorsha dorsha commented Sep 27, 2025

Description

Add patch user batch

https://github.com/descope/etc/issues/12241

Must

  • Tests
  • Documentation (if applicable)

@dorsha dorsha requested review from aviadl and Copilot September 27, 2025 17:51
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for batch patching users with a new patch_batch method, extending the existing user patching functionality to handle multiple users in a single operation. It also adds status field support to both individual and batch patch operations.

Key Changes

  • Added patch_batch method to the User management class for bulk user updates
  • Enhanced the existing patch method to support user status updates
  • Added comprehensive test coverage for both individual and batch patching with status validation

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
descope/management/user.py Added patch_batch method and status parameter to patch method with validation logic
descope/management/common.py Added new API endpoint path for batch patch operations
tests/management/test_user.py Added comprehensive test coverage for status validation and batch patching functionality

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link

Coverage report

The coverage rate went from 97.79% to 97.81% ⬆️

100% of new lines are covered.

Diff Coverage details (click to unfold)

descope/management/user.py

100% of new lines are covered (95.94% of the complete file).

descope/management/common.py

100% of new lines are covered (99.57% of the complete file).

@dorsha dorsha enabled auto-merge (squash) September 27, 2025 17:55
@dorsha dorsha merged commit dfb4cc5 into main Sep 27, 2025
28 checks passed
@dorsha dorsha deleted the patch-user-batch branch September 27, 2025 18:00
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.

3 participants