Skip to content

ref(pipeline): Remove unused react_pipeline template and render_react_view#113439

Merged
evanpurkhiser merged 1 commit into
masterfrom
evanpurkhiser/ref-pipeline-remove-unused-react-pipeline-template-and-render-react-view
Apr 20, 2026
Merged

ref(pipeline): Remove unused react_pipeline template and render_react_view#113439
evanpurkhiser merged 1 commit into
masterfrom
evanpurkhiser/ref-pipeline-remove-unused-react-pipeline-template-and-render-react-view

Conversation

@evanpurkhiser

@evanpurkhiser evanpurkhiser commented Apr 20, 2026

Copy link
Copy Markdown
Member

These were the last remnants of the old "django template renders react"
pipeline flow. With all integrations that used this migrated to the
API-driven pipeline, the template and its helper are no longer referenced.

Refs VDY-32: Migrate integration setup pipelines to API-driven flows

…_view

These were the last remnants of the old "django template renders react"
pipeline flow. With all integrations migrated to the API-driven pipeline,
the template and its helper are no longer referenced.
@evanpurkhiser
evanpurkhiser requested review from a team as code owners April 20, 2026 15:50
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 20, 2026
@linear-code

linear-code Bot commented Apr 20, 2026

Copy link
Copy Markdown

@leeandher leeandher left a comment

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.

🔥

@evanpurkhiser
evanpurkhiser merged commit c686bfd into master Apr 20, 2026
58 checks passed
@evanpurkhiser
evanpurkhiser deleted the evanpurkhiser/ref-pipeline-remove-unused-react-pipeline-template-and-render-react-view branch April 20, 2026 16:37
joshuarli added a commit that referenced this pull request Apr 22, 2026
c97c2a8e498 accidentally replaced the whole file with the pre-migration
legacy class (which patches render_react_view, a function removed in
#113439), instead of adding a skip to the new API-driven test class.
The legacy test test_node_lambda_setup_layer_success doesn't exist in
the new file, so there's nothing to skip.
@github-actions github-actions Bot locked and limited conversation to collaborators May 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants