Skip to content

FCE-3683 add template recordings - #99

Open
czerwiukk wants to merge 3 commits into
mainfrom
FCE-3863/add-template-recordings
Open

czerwiukk wants to merge 3 commits into
mainfrom
FCE-3863/add-template-recordings

Conversation

@czerwiukk

Copy link
Copy Markdown
Member

Description

Adds the method for creating a template recording.

Motivation and Context

This allows recording a standalone template instead of mirroring an existing composition output.

Documentation impact

  • Documentation update required
  • Documentation updated in another PR
  • No documentation update required

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)

@czerwiukk
czerwiukk requested a review from Gawor270 September 22, 2026 11:57
@czerwiukk czerwiukk self-assigned this Sep 22, 2026
@linear

linear Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

FCE-3683

FCE-3863

@czerwiukk
czerwiukk removed the request for review from Gawor270 September 22, 2026 11:58
@czerwiukk
czerwiukk marked this pull request as draft September 22, 2026 11:58
@czerwiukk
czerwiukk requested a review from Gawor270 September 23, 2026 12:36
@czerwiukk
czerwiukk marked this pull request as ready for review September 23, 2026 12:36
Comment thread scripts.py
_fix_multi_content_type_endpoints(output_path)


def _fix_multi_content_type_endpoints(output_path: str):

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: I checked openapi-python-client 0.29.1 locally against the new spec. It already sets the multipart Content-Type with a boundary (boundary=+++) that httpx uses for the body, so the header-stripping part of this script would no longer needed. The missing Unset import is still broken in 0.29.1, so that part has to stay.

This branch has not been deployed

No deployments
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.

2 participants