Skip to content

Conversation

@ChmaraX
Copy link
Contributor

@ChmaraX ChmaraX commented Nov 19, 2025

What changed? Why was the change needed?

EE PR: https://github.com/novuhq/packages-enterprise/pull/376

This pull request introduces several updates to the internal SDK, focusing on improved file upload handling, schema and type cleanup, and configuration enhancements. The most significant changes are the refactoring of translation upload functions to support more flexible file inputs and updates to the code generation configuration for better SDK output.

File Upload Handling Improvements

  • Refactored translationsUpload and translationsMasterUpload functions to accept a new request body type, supporting both Blob-like objects and readable streams for file uploads, and improved FormData construction for various file input scenarios. (libs/internal-sdk/src/funcs/translationsUpload.ts, libs/internal-sdk/src/funcs/translationsMasterUpload.ts) [1] [2] [3] [4]

Bug Fixes and Robustness

  • Improved the idempotency interceptor to handle non-serializable request bodies gracefully by falling back to string representations for hashing. (apps/api/src/app/shared/framework/idempotency.interceptor.ts)

Subproject Update

  • Updated subproject commit reference to the latest version. (.source)

@linear
Copy link

linear bot commented Nov 19, 2025

@netlify
Copy link

netlify bot commented Nov 19, 2025

Deploy Preview for dashboard-v2-novu-staging canceled.

Name Link
🔨 Latest commit 6c1f2bf
🔍 Latest deploy log https://app.netlify.com/projects/dashboard-v2-novu-staging/deploys/691dd641205f0800084affaa

@github-actions github-actions bot changed the title fix(api,sdk): translation DTOs/OAS for SDK auto-gen; idempotency fix(api,sdk): translation DTOs/OAS for SDK auto-gen; idempotency fixes NV-6903 Nov 19, 2025
@ChmaraX ChmaraX changed the title fix(api,sdk): translation DTOs/OAS for SDK auto-gen; idempotency fixes NV-6903 fix(api): translation DTOs/OAS for SDK auto-gen; idempotency fixes NV-6903 Nov 19, 2025
@ChmaraX ChmaraX merged commit 84adf3a into next Nov 19, 2025
8 checks passed
@ChmaraX ChmaraX deleted the nv-6903-apisdk-translations-upload-methods-have-incorrectly branch November 19, 2025 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants