Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(portal): Fix bug in actor edit page preventing updates #4347

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

bmanifold
Copy link
Collaborator

Why:

  • A bug was present in the actor edit page that prevented updating an actor due to managed and synced groups being sent as part of the form submission. Along with that, if a user manually removed the managed group(s) from the form submission, the actor being edited would be removed from the managed group, which should not be allowed.

  • There was also another small bug which prevent an admin actor from being updated at all if they were the only admin in the account.

Why:

* A bug was present in the actor edit page that prevented updating an
  actor due to managed and synced groups being sent as part of the form
  submission.  Along with that, if a user manually removed the managed
  group(s) from the form submission, the actor being edited would be
  removed from the managed group, which should not be allowed.

* There was also another small bug which prevent an admin actor from
  being updated at all if they were the only admin in the account.
Copy link

vercel bot commented Mar 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated (UTC)
firezone ⬜️ Ignored (Inspect) Mar 27, 2024 7:02pm

Copy link

Terraform Cloud Plan Output

Plan: 15 to add, 14 to change, 15 to destroy.

Terraform Cloud Plan

@bmanifold bmanifold changed the title Fix bug in actor edit page preventing updates fix(portal): Fix bug in actor edit page preventing updates Mar 27, 2024
Copy link

Performance Test Results

TCP

Test Name Received/s Sent/s Retransmits
direct-tcp-client2server 227.3 MiB (+1%) 229.7 MiB (+1%) 318 (+179%)
direct-tcp-server2client 226.7 MiB (+1%) 228.5 MiB (+1%) 391 (-38%)
relayed-tcp-client2server 149.2 MiB (+6%) 150.0 MiB (+6%) 155 (+17%)
relayed-tcp-server2client 152.2 MiB (-1%) 152.6 MiB (-1%) 171 (-19%)

UDP

Test Name Total/s Jitter Lost
direct-udp-client2server 50.0 MiB (+0%) 0.27ms (-14%) 0.00% (NaN%)
direct-udp-server2client 50.0 MiB (-0%) 0.01ms (-47%) 0.00% (NaN%)
relayed-udp-client2server 50.0 MiB (0%) 0.10ms (+8%) 0.00% (NaN%)
relayed-udp-server2client 50.0 MiB (+0%) 0.06ms (+19%) 0.00% (NaN%)

@AndrewDryga AndrewDryga merged commit ab35a5e into main Mar 27, 2024
152 checks passed
@AndrewDryga AndrewDryga deleted the bug/actor-edit-groups branch March 27, 2024 19:20
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.

None yet

2 participants