Skip to content

Conversation

@RubenSandwich
Copy link
Contributor

@RubenSandwich RubenSandwich commented Nov 21, 2025

Ticket - UDR deployments fail to consistently kick off necessary dev-portal builds

Description

This PR simplifies the deployment of dev-portal when UDR changes. It does so by fully reloading dev-portal on every change. This is a bit of a heavy hand approach, with a more fine-grain approach being working on here, but we are taking this approach to get this work out ASAP.

Testing

Because of the small and simplified nature of this change/code, I do not believe testing is necessary. Just a manual review from another engineer.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 21, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Fri Nov 21 15:54:13 UTC 2025
Unified Docs API ✅ Ready (Inspect) Visit Preview Fri Nov 21 15:47:30 UTC 2025

@RubenSandwich RubenSandwich marked this pull request as ready for review November 21, 2025 15:48
@RubenSandwich RubenSandwich requested a review from a team as a code owner November 21, 2025 15:48
@@ -0,0 +1,14 @@
name: Force Rebuild Dev Portal
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Before this action required the inputs; base_sha and head_sha which I believe scared folks. It now takes no inputs and redeploys the current main of UDR.

Copy link
Contributor

@williamdalessandro williamdalessandro left a comment

Choose a reason for hiding this comment

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

Cool! I didn't know about those deploy hooks

@RubenSandwich RubenSandwich merged commit 3273a5a into main Nov 21, 2025
15 checks passed
@RubenSandwich RubenSandwich deleted the rn/fix-dev-portal-reloading-try-2 branch November 21, 2025 18:22
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