Skip to content

Update order.py#147

Merged
pierrerondel merged 3 commits intomasterfrom
pierrerondel-patch-1
Sep 10, 2025
Merged

Update order.py#147
pierrerondel merged 3 commits intomasterfrom
pierrerondel-patch-1

Conversation

@pierrerondel
Copy link
Copy Markdown
Contributor

new function:

def command_call_check_duplicate(self, command: str, mode: int, params, timeout=300) -> None:
"""
Command call with duplicate check for CREATE.

    Parameters
    -----------
    command: str
        CRUD method in microservice to call ("CREATE", "UPDATE", etc.)
    mode: int
        0 - No application
        1 - Apply to base
        2 - Apply to device
    params: dict
        Parameters for the command.
        When using CREATE, must contain:
          - object_name (str): microservice object name
          - object_id (str): instance ID timeout: int Timeout for the request in seconds (default=300)

new function:

def command_call_check_duplicate(self, command: str, mode: int, params, timeout=300) -> None:
        """
        Command call with duplicate check for CREATE.

        Parameters
        -----------
        command: str
            CRUD method in microservice to call ("CREATE", "UPDATE", etc.)
        mode: int
            0 - No application
            1 - Apply to base
            2 - Apply to device
        params: dict
            Parameters for the command.
            When using CREATE, must contain:
              - object_name (str): microservice object name
              - object_id (str): instance ID
        timeout: int
            Timeout for the request in seconds (default=300)
Comment thread msa_sdk/order.py Outdated
replace word object by instance in the error message
remove a blank line that made pydocstring to fail during the tests.
@pierrerondel pierrerondel merged commit b07c7c5 into master Sep 10, 2025
2 checks passed
@pierrerondel pierrerondel deleted the pierrerondel-patch-1 branch September 10, 2025 07:03
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