Skip to content

Update the checks on the application/compose modal to use the checks as others#4471

Open
Alt-iOS wants to merge 3 commits into
Dokploy:canaryfrom
Alt-iOS:canary
Open

Update the checks on the application/compose modal to use the checks as others#4471
Alt-iOS wants to merge 3 commits into
Dokploy:canaryfrom
Alt-iOS:canary

Conversation

@Alt-iOS
Copy link
Copy Markdown

@Alt-iOS Alt-iOS commented May 22, 2026

What is this PR about?

When sharing git access to users or even admin users, in the UI for the app/compose section, they had an error message saying they could switch it to a git source they had access but couldn't change anything in the current one. This simply re-uses the existing func that takes into account not only if the user is the owner of the git source but if they have access via their role or if the git instance is shared with the org.

Checklist

Before submitting this PR, please make sure that:

  • You created a dedicated branch based on the canary branch.
  • You have read the suggestions in the CONTRIBUTING.md file https://github.com/Dokploy/dokploy/blob/canary/CONTRIBUTING.md#pull-request
  • You have tested this PR in your local instance. If you have not tested it yet, please do so before submitting. This helps avoid wasting maintainers' time reviewing code that has not been verified by you.

Issues related (if applicable)

closes #4469

Screenshots (if applicable)

@Alt-iOS Alt-iOS requested a review from Siumauricio as a code owner May 22, 2026 22:46
@dosubot dosubot Bot added size:S This PR changes 10-29 lines, ignoring generated files. pr-open labels May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-open size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Admin users blocked from changing git branch on compose deployments

1 participant