Skip to content

Conversation

adityathebe
Copy link
Member

Summary

  • Automatically populate object_selector field when a playbook is selected in the permission form
  • Adds playbook name and wildcard configs selector to enable proper resource-based permissions

Changes

  • Added onValueChange callback prop to FormikResourceSelectorDropdown to pass the full selected option data
  • Enhanced playbook option to include the playbook name in the option object
  • Updated FormikPermissionSelectResourceFields to set object_selector with the format: {"playbooks": [{"name": "<selected-playbook>"}], "configs": [{"name": "*"}]}

Test plan

  • Verify that selecting a playbook in the permission form sets the object_selector field correctly
  • Verify that clearing the playbook selection clears the object_selector field
  • Verify that the permission is saved with the correct object_selector data

When a playbook is selected in the permission form, automatically populate
object_selector with the playbook name and wildcard configs selector to enable
proper resource-based permissions.
Copy link

vercel bot commented Oct 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
aws-preview Ready Ready Preview Oct 10, 2025 7:53am
flanksource-ui Ready Ready Preview Oct 10, 2025 7:53am

@adityathebe adityathebe enabled auto-merge (squash) October 10, 2025 07:46
Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for clerk-saas-ui ready!

Name Link
🔨 Latest commit cb26d41
🔍 Latest deploy log https://app.netlify.com/projects/clerk-saas-ui/deploys/68e8b9eb0a29830008280713
😎 Deploy Preview https://deploy-preview-2656--clerk-saas-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for flanksource-demo-stable ready!

Name Link
🔨 Latest commit cb26d41
🔍 Latest deploy log https://app.netlify.com/projects/flanksource-demo-stable/deploys/68e8b9eb39ad30000877e208
😎 Deploy Preview https://deploy-preview-2656--flanksource-demo-stable.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

required
name="playbook_id"
playbookResourceSelector={[{}]}
onValueChange={(value, option) => {
Copy link
Member Author

Choose a reason for hiding this comment

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

Temporary fix until we hadd resource selector to the form

Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for goofy-euclid-75956c ready!

Name Link
🔨 Latest commit cb26d41
🔍 Latest deploy log https://app.netlify.com/projects/goofy-euclid-75956c/deploys/68e8b9ebffda4e000897b0ac
😎 Deploy Preview https://deploy-preview-2656--goofy-euclid-75956c.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant