Skip to content

Conversation

@PVeazie951
Copy link
Contributor

This PR updates the deployment flow so the frontend correctly passes repoFullName, yaml, and other contextual fields from the Dashboard to the backend’s /mcp/v1/pipeline_commit endpoint.

✅ Changes
• Refactored startDeploy() in api.ts to include full payload (repoFullName, branch, env, yaml, provider, path).
• Updated useDeployStore.ts to forward the dashboard payload directly.
• Fixed missing usePipelineStore reference by importing the store directly.
• Verified full YAML is now committed to GitHub successfully.

🧠 Context

Previously, only partial payload data (branch, env, provider, path) was being sent, causing 400 errors from the backend (repoFullName and yaml are required). This update resolves that and ensures full commit data alignment between frontend and backend.

🧪 Testing
• Triggered deployment from Dashboard ✅
• Verified full payload in network logs ✅
• Confirmed successful YAML commit to GitHub ✅

Copy link
Contributor

@lorencDedaj lorencDedaj left a comment

Choose a reason for hiding this comment

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

Very Good!

@lorencDedaj lorencDedaj merged commit 2706280 into main Nov 8, 2025
2 checks passed
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.

3 participants