I'm encountering an issue with the Novu Python SDK while trying to create a layout. The API returns a 400 Bad Request error with the message indicating that the identifier should not be null or undefined and must be a string. However, the LayoutDto class in the SDK does not provide an option to pass the identifier, which seems to be required by the API.
Error Message
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://api.novu.co/v1/layouts {'message': ['identifier should not be null or undefined', 'identifier must be a string'], 'error': 'Bad Request', 'statusCode': 400}
To Reproduce
Create a LayoutDto instance without an identifier field.
Attempt to create a layout using the Novu Python SDK.
Observe the 400 Bad Request error.
Expected behavior
The layout should be created successfully without a 400 Bad Request error.
Screenshots
The API returns a 400 Bad Request error indicating that the identifier field is missing.
Versions
Novu version: Novu SaaS
Python version: 3.12.3
Novu-python version: 1.14.0
Additional context
I have added a fix for the same
Pull request for the fix : pull-294
Describe the bug
I'm encountering an issue with the Novu Python SDK while trying to create a layout. The API returns a 400 Bad Request error with the message indicating that the identifier should not be null or undefined and must be a string. However, the LayoutDto class in the SDK does not provide an option to pass the identifier, which seems to be required by the API.
Error Message
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://api.novu.co/v1/layouts {'message': ['identifier should not be null or undefined', 'identifier must be a string'], 'error': 'Bad Request', 'statusCode': 400}To Reproduce
Expected behavior
The layout should be created successfully without a 400 Bad Request error.
Screenshots
The API returns a 400 Bad Request error indicating that the identifier field is missing.
Versions
Additional context I have added a fix for the same Pull request for the fix : pull-294