Skip to content

Conversation

@robot9706
Copy link
Contributor

@robot9706 robot9706 commented Aug 7, 2023

  • Add "Protected" toggle to "Edit deployments"
  • Move startDeploy logic to a custom hook
  • Fix unable to delete versions
  • Fix selecting instances to deploy not working

@robot9706 robot9706 self-assigned this Aug 7, 2023
@github-actions github-actions bot added source:web The scope of the issue or pull request is web. lang:typescript lang:sql pr:feat labels Aug 7, 2023
@robot9706 robot9706 marked this pull request as ready for review August 8, 2023 09:31
@robot9706 robot9706 requested a review from a team as a code owner August 8, 2023 09:31
Copy link
Contributor

@AmorfEvo AmorfEvo left a comment

Choose a reason for hiding this comment

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

LGTM

@robot9706 robot9706 force-pushed the feat/protected-deployment branch from 140a732 to fce0cb9 Compare August 15, 2023 11:42
Copy link
Collaborator

@polaroi8d polaroi8d left a comment

Choose a reason for hiding this comment

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

Tested, works like a charm 🤩

Copy link
Contributor

@m8vago m8vago left a comment

Choose a reason for hiding this comment

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

Just some minor inconvenience in ifs. We can fix that in a later pr.

@m8vago m8vago merged commit 8701d61 into develop Aug 18, 2023
@m8vago m8vago deleted the feat/protected-deployment branch August 18, 2023 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants