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: decode logs in results object and remove overflow of caused by text #1444

Merged
merged 4 commits into from
Oct 18, 2023

Conversation

mainawycliffe
Copy link
Contributor

Closes #1436

@netlify
Copy link

netlify bot commented Oct 17, 2023

Deploy Preview for goofy-euclid-75956c ready!

Name Link
🔨 Latest commit d6e1b27
🔍 Latest deploy log https://app.netlify.com/sites/goofy-euclid-75956c/deploys/652f826eca838a00083610a6
😎 Deploy Preview https://deploy-preview-1444--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.

@netlify
Copy link

netlify bot commented Oct 17, 2023

Deploy Preview for flanksource-demo-stable ready!

Name Link
🔨 Latest commit d6e1b27
🔍 Latest deploy log https://app.netlify.com/sites/flanksource-demo-stable/deploys/652f826e4719970008c13c39
😎 Deploy Preview https://deploy-preview-1444--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 Oct 17, 2023

Deploy Preview for clerk-saas-ui ready!

Name Link
🔨 Latest commit d6e1b27
🔍 Latest deploy log https://app.netlify.com/sites/clerk-saas-ui/deploys/652f826e734fb5000846138b
😎 Deploy Preview https://deploy-preview-1444--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.

@moshloop
Copy link
Member

@mainawycliffe we are missing the handling of errors: https://deploy-preview-1444--flanksource-demo-stable.netlify.app/playbooks/runs/018af987-d406-a5df-5a63-a07d8842c7cf

Also, I don't think we should be downloading the result/error on load - only when the step is selected

@mainawycliffe mainawycliffe force-pushed the 1436-playbook-actions-results-improvements branch from d923916 to 05c9efd Compare October 18, 2023 06:31
@mainawycliffe mainawycliffe force-pushed the 1436-playbook-actions-results-improvements branch from 08f7674 to d6e1b27 Compare October 18, 2023 06:59
@moshloop moshloop merged commit d47b62f into main Oct 18, 2023
7 checks passed
@moshloop moshloop deleted the 1436-playbook-actions-results-improvements branch October 18, 2023 07:52
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.

Playbook Actions Results Improvements
2 participants