Skip to content

feat: run workflows within builder#1222

Merged
shahargl merged 30 commits intokeephq:mainfrom
onurio:feat/run-workflows-within-builder
Jun 30, 2024
Merged

feat: run workflows within builder#1222
shahargl merged 30 commits intokeephq:mainfrom
onurio:feat/run-workflows-within-builder

Conversation

@onurio
Copy link
Copy Markdown
Contributor

@onurio onurio commented Jun 10, 2024

/claim #932

Closes #932

📑 Description

This is a solution 2 from the plan here

  • New Endpoint - /workflows/test -> Workflows can now be ran synchronously using a definition, without being saved into the db (both workflow and execution).

https://www.loom.com/share/9af9209df66645cc82d4d323042fae5c?sid=5bd861fd-c5f3-47c3-aa8c-0963c1966ab2

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

ℹ Additional Information

@vercel
Copy link
Copy Markdown

vercel bot commented Jun 10, 2024

@onurio is attempting to deploy a commit to the KeepHQ Team on Vercel.

A member of the Team first needs to authorize it.

@algora-pbc
Copy link
Copy Markdown

algora-pbc bot commented Jun 10, 2024

💵 To receive payouts, sign up on Algora, link your Github account and connect with Stripe/Alipay.

@onurio onurio force-pushed the feat/run-workflows-within-builder branch from e3e17cf to ca9a2c9 Compare June 10, 2024 03:56
@onurio onurio force-pushed the feat/run-workflows-within-builder branch from ca9a2c9 to 63d06af Compare June 10, 2024 03:57
@onurio onurio force-pushed the feat/run-workflows-within-builder branch 3 times, most recently from 3daf501 to 903b025 Compare June 11, 2024 03:40
@onurio onurio changed the title feat: run workflows within builder [DRAFT] feat: run workflows within builder Jun 11, 2024
@onurio onurio marked this pull request as ready for review June 11, 2024 03:46
@onurio onurio force-pushed the feat/run-workflows-within-builder branch from 355bac8 to 6140ba9 Compare June 11, 2024 03:51
@Matvey-Kuk
Copy link
Copy Markdown
Contributor

Hi @onurio, thank you for the contribution!

In my latest Chrome on a screen width less than 1400px, there is smth looking like a z-index-related issue:
Screenshot 2024-06-13 at 11 18 09

@onurio
Copy link
Copy Markdown
Contributor Author

onurio commented Jun 13, 2024

thanks! fixed it.

@vercel
Copy link
Copy Markdown

vercel bot commented Jun 26, 2024

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

Name Status Preview Comments Updated (UTC)
keep ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 30, 2024 2:02pm

@talboren
Copy link
Copy Markdown
Member

@onurio I tried to play with branch a little bit but seems like the UI build is broken. Tried to quickly fix it but the issue seems a bit more "deep" - can you fix the build so I can move forward with merging this in? :)

@onurio
Copy link
Copy Markdown
Contributor Author

onurio commented Jun 27, 2024

@talboren sure thing, I'll do it asap.

@onurio
Copy link
Copy Markdown
Contributor Author

onurio commented Jun 27, 2024

@talboren I checked and it works on my end, I also rebuilt the image to make sure. Can you share the error you are seeing?

@talboren
Copy link
Copy Markdown
Member

talboren commented Jun 27, 2024

@onurio

The error was:
CleanShot 2024-06-27 at 16 04 51@2x

and then I tried this: 75fe07e but it still is broken.

@onurio
Copy link
Copy Markdown
Contributor Author

onurio commented Jun 27, 2024

Checking more, very weird that it didn't break on my side.

@onurio onurio force-pushed the feat/run-workflows-within-builder branch from e5d2f36 to 4a8057b Compare June 27, 2024 13:42
@onurio
Copy link
Copy Markdown
Contributor Author

onurio commented Jun 27, 2024

@talboren I managed to reproduce the error and the fix that I pushed solves it.

Copy link
Copy Markdown
Member

@talboren talboren left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@shahargl shahargl left a comment

Choose a reason for hiding this comment

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

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Run workflow from Workflow Builder

4 participants