Skip to content

fix: open deployments tab when opening a service from the deployments list - #4583

Merged
narcisonunez merged 1 commit into
Dokploy:canaryfrom
TpmyCT:fix/deployments-open-tab
Aug 13, 2026
Merged

fix: open deployments tab when opening a service from the deployments list#4583
narcisonunez merged 1 commit into
Dokploy:canaryfrom
TpmyCT:fix/deployments-open-tab

Conversation

@TpmyCT

@TpmyCT TpmyCT commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

What is this PR about?

In the centralized Deployments list (/dashboard/deployments), the Open action on each row navigated to the related application/compose page but landed on its General tab — forcing you to manually click into the Deployments tab to reach the deployment you came from. This appends ?tab=deployments to the generated link so Open opens the Deployments tab directly.

Tested locally: Open now lands on the Deployments tab for both applications and compose services. It uses the same ?tab=deployments deep-link the app already applies after a deploy (application/general/show.tsx, compose/general/actions.tsx), so the behavior is consistent with the rest of the app.

Checklist

Screenshots (if applicable)

Before: Open on a deployment row landed on the service's General tab.
After: Open lands directly on the Deployments tab.

… list

The Open action in the centralized deployments list navigated to the service's General tab instead of its Deployments tab. Append `?tab=deployments` to the generated link so it deep-links straight to the Deployments tab, matching the redirect the app already performs after a deploy.
@TpmyCT
TpmyCT marked this pull request as ready for review June 9, 2026 01:59
@TpmyCT
TpmyCT requested a review from Siumauricio as a code owner June 9, 2026 01:59
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 9, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 13, 2026
@narcisonunez
narcisonunez merged commit 52aca46 into Dokploy:canary Aug 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants