Skip to content

Allow deeplinking of workflows for import#603

Merged
leszko merged 3 commits intomainfrom
workflow-deeplink
Mar 6, 2026
Merged

Allow deeplinking of workflows for import#603
leszko merged 3 commits intomainfrom
workflow-deeplink

Conversation

@thomshutt
Copy link
Copy Markdown
Contributor

To test, go to a page like https://app.daydream.live/workflows/thomshutt/pixel-art-man and click Install Workflow (once you have the desktop Scope app running locally on this branch).

Also includes a change to improve LoRA download errors from:
image

To:
Failed to download

@thomshutt thomshutt force-pushed the workflow-deeplink branch from 542b4de to 59e78fc Compare March 6, 2026 10:21
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 6, 2026

🚀 fal.ai Preview Deployment

App ID daydream/scope-pr-603--preview
WebSocket wss://fal.run/daydream/scope-pr-603--preview/ws
Commit 8271f20

Testing

Connect to this preview deployment by running this on your branch:

uv run build && SCOPE_CLOUD_APP_ID="daydream/scope-pr-603--preview/ws" uv run daydream-scope

🧪 E2E tests will run automatically against this deployment.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 6, 2026

❌ E2E Tests failed

Status failed
fal App daydream/scope-pr-603--preview
Run View logs

Test Artifacts

Check the workflow run for screenshots, traces, and failure details.

response.json() can raise JSONDecodeError (a ValueError subclass) when
the response body is not valid JSON. The previous except/re-raise pattern
would propagate that parse error as an HTTP 400 with a confusing message.
Separate JSON parsing from error-raising so parse failures return
(None, None) gracefully.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
Signed-off-by: Rafał Leszko <[email protected]>
Copy link
Copy Markdown
Collaborator

@leszko leszko left a comment

Choose a reason for hiding this comment

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

Pushed a super minor fix. Other than that it looks good. Merging.

@leszko leszko merged commit 0264574 into main Mar 6, 2026
7 checks passed
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.

2 participants