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

1353-components-playbooks #1358

Merged
merged 5 commits into from
Oct 10, 2023
Merged

1353-components-playbooks #1358

merged 5 commits into from
Oct 10, 2023

Conversation

mainawycliffe
Copy link
Contributor

Build on top of the #1357 PR

Closes #1353

@netlify
Copy link

netlify bot commented Sep 4, 2023

Deploy Preview for flanksource-demo-stable ready!

Name Link
🔨 Latest commit 36d7344
🔍 Latest deploy log https://app.netlify.com/sites/flanksource-demo-stable/deploys/65248897fda26b00080fba83
😎 Deploy Preview https://deploy-preview-1358--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 site configuration.

@netlify
Copy link

netlify bot commented Sep 4, 2023

Deploy Preview for clerk-saas-ui ready!

Name Link
🔨 Latest commit 36d7344
🔍 Latest deploy log https://app.netlify.com/sites/clerk-saas-ui/deploys/65248897886b060008651acc
😎 Deploy Preview https://deploy-preview-1358--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 site configuration.

@mainawycliffe mainawycliffe marked this pull request as draft September 4, 2023 12:23
@netlify
Copy link

netlify bot commented Sep 4, 2023

Deploy Preview for goofy-euclid-75956c ready!

Name Link
🔨 Latest commit 36d7344
🔍 Latest deploy log https://app.netlify.com/sites/goofy-euclid-75956c/deploys/65248897b59ef4000883020a
😎 Deploy Preview https://deploy-preview-1358--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 site configuration.

@mainawycliffe mainawycliffe force-pushed the 1353-components-playbooks branch 4 times, most recently from 6e0b747 to 93491a9 Compare September 6, 2023 17:40
@mainawycliffe mainawycliffe marked this pull request as ready for review September 6, 2023 17:40
@mainawycliffe mainawycliffe force-pushed the 1353-components-playbooks branch 5 times, most recently from 6e4f3e0 to 3283b3c Compare September 12, 2023 07:22
@mainawycliffe
Copy link
Contributor Author

@moshloop this is good to go as well

@mainawycliffe mainawycliffe force-pushed the 1353-components-playbooks branch 3 times, most recently from 587d6a3 to e801159 Compare September 13, 2023 17:31
@moshloop
Copy link
Member

@mainawycliffe Can we change the button into menu button called Playbooks - which then loads the list of playbooks that can be run from /playbooks/list - https://github.com/flanksource/incident-commander/blob/main/playbook/controllers.go#L167

@mainawycliffe
Copy link
Contributor Author

@moshloop got it

@mainawycliffe mainawycliffe force-pushed the 1353-components-playbooks branch 5 times, most recently from 6f16c64 to 581e5e9 Compare September 20, 2023 21:33
@mainawycliffe mainawycliffe force-pushed the 1353-components-playbooks branch from 581e5e9 to 367a9b6 Compare September 21, 2023 16:30
@mainawycliffe
Copy link
Contributor Author

@moshloop is something like this what you had in mind?

image

@mainawycliffe mainawycliffe force-pushed the 1353-components-playbooks branch 3 times, most recently from 53e69a3 to 23d6904 Compare September 27, 2023 06:42
@mainawycliffe mainawycliffe force-pushed the 1353-components-playbooks branch 2 times, most recently from ea4aabd to 5991538 Compare October 3, 2023 04:09
This is added to both the configs and component side panel

Closes #1353

feat: add dropdown for components to submit playbook runs

Build on top of #1352  PR

Closes #1353

fix: close modal on save

fix: fix submit relaoding full page and wrong endpoint

fix: fix playbook submission params

feat: add run playbook dropdown to configs sidepanel

fix: fix issue with dropdown menu item hover
fix: make row clickable for sidepanel

fix: fix duration issue with side panel
fix: fix imports

refactor: fix and organize imports
@mainawycliffe mainawycliffe force-pushed the 1353-components-playbooks branch 4 times, most recently from ea462eb to 36d7344 Compare October 9, 2023 23:11
@moshloop moshloop merged commit 48e5a34 into main Oct 10, 2023
9 checks passed
@moshloop moshloop deleted the 1353-components-playbooks branch October 10, 2023 07:04
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.

Components -> Playbooks
2 participants