Skip to content
This repository was archived by the owner on Dec 20, 2025. It is now read-only.

refactor(kubernetes): convert undo rollout manifest stage to react#10159

Merged
mergify[bot] merged 4 commits intospinnaker:masterfrom
cristhian-castaneda:refactor/reactify-undo-rollout-manifest-stage
Jan 28, 2025
Merged

refactor(kubernetes): convert undo rollout manifest stage to react#10159
mergify[bot] merged 4 commits intospinnaker:masterfrom
cristhian-castaneda:refactor/reactify-undo-rollout-manifest-stage

Conversation

@cristhian-castaneda
Copy link
Member

Overview

This PR addresses the refactoring of the Undo Rollout (Manifest) stage, part of the ongoing effort to transition Kubernetes stages to React. For additional context, please refer to spinnaker/spinnaker#4254

Changes Made

  • Refactored Components:
    • The files undoRolloutManifestConfig.html and undoRolloutManifestConfig.controller.ts have been converted into the new components UndoRolloutManifestConfig.tsx and UndoRolloutManifestStageForm.tsx.

Stage Views

Stage Config

image

Execution Details Sections

Undo Rollout Manifest

image

Execution Details Tasks

image

Copy link
Collaborator

@siddhu-opsmx siddhu-opsmx left a comment

Choose a reason for hiding this comment

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

LGTM

@siddhu-opsmx siddhu-opsmx added ready to merge Reviewed and ready for merge and removed ready to merge Reviewed and ready for merge labels Jan 26, 2025
@mergify mergify bot added the auto merged Merged automatically by a bot label Jan 28, 2025
@mergify mergify bot merged commit 824e0e5 into spinnaker:master Jan 28, 2025
4 checks passed
@cristhian-castaneda cristhian-castaneda deleted the refactor/reactify-undo-rollout-manifest-stage branch January 28, 2025 16:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

auto merged Merged automatically by a bot ready to merge Reviewed and ready for merge target-release/1.37

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants