-
-
Notifications
You must be signed in to change notification settings - Fork 172
Conversation
djyotta
commented
Jun 23, 2024
@lovasoa can you take a look to see if it's ok in principle before I start on documentation?
Fix spinner defaults
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.
I've noticed this whole approach doesn't work inside run_sql for some reason. So I should note that 'loader-start' component should be selected before descending into run_sql.
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.
Yes, run_sql is a function, it returns a single json value all at once, it can't stream its results.
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.
I thought as much but didn't want to assume. It's good to have the clarification.
Uh oh!
There was an error while loading. Please reload this page.
Closes #435