-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: Flows API module #2419
feat: Flows API module #2419
Conversation
Removed vultr server and associated DNS entries |
flowId: z.string(), | ||
}), | ||
body: z.object({ | ||
replaceValue: z.string().length(5), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strictly controlling the length here to 5 matches the current frontend behaviour, and has meant that a few test cases need nodeIds tweaked below.
I'll double check the onboarding scripts to enforce this to 5 characters there as well once this is merged.
02b987b
to
37da275
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good & all working as expected on pizza, thanks again for working through all these !
Thanks for review - kept in draft as I hit a small issue with the download-schema endpoint. Looking at it again now and will push a fix to this branch shortly. |
081696f
to
960b5b5
Compare
🤖 Hasura Change Summary compared a subset of table metadata including permissions: Updated Tables (1)
Untracked Tables (1)
|
What does this PR do?
/flows
endpoints