Skip to content

Support workflow_run_id query param on workflow logs page#34951

Open
ashutoshrana wants to merge 2 commits intolanggenius:mainfrom
ashutoshrana:ashutosh/dify-34911-workflow-run-url
Open

Support workflow_run_id query param on workflow logs page#34951
ashutoshrana wants to merge 2 commits intolanggenius:mainfrom
ashutoshrana:ashutosh/dify-34911-workflow-run-url

Conversation

@ashutoshrana
Copy link
Copy Markdown

Summary

Supports passing workflow_run_id via the workflow logs URL for faster lookup.

Example:

  • /app/<appId>/logs?workflow_run_id=<runId>

This change:

  • initializes the workflow logs search input from workflow_run_id
  • triggers the existing keyword-based log lookup automatically
  • keeps workflow_run_id in sync when the keyword changes
  • clears stale page from the URL when the keyword changes

Scope

This PR is intentionally narrow:

  • no backend/API changes
  • no new query params beyond workflow_run_id
  • no changes to workflow log pagination behavior
  • no auto-opening of a log detail drawer from URL state

Why no backend change

The existing workflow logs backend already supports matching workflow run IDs through the current keyword search path, so this PR only adds the missing URL/state wiring on the frontend.

Validation

Ran:

  • pnpm -C web test -- --run app/components/app/workflow-log/__tests__
  • pnpm -C web test -- --run app/components/app/log/__tests__
  • pnpm -C web test -- --run app/components/app/log-annotation/__tests__

Notes

This PR supports URL-based initialization of the workflow log search from workflow_run_id.

It does not try to implement full post-mount URL/state synchronization for every possible navigation case. I kept that out of scope to keep the first contribution small and aligned with the issue request.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. javascript Pull requests that update javascript code labels Apr 11, 2026
@github-actions github-actions bot added web This relates to changes on the web. and removed javascript Pull requests that update javascript code labels Apr 11, 2026
@ashutoshrana
Copy link
Copy Markdown
Author

Created a feature issue for this PR per the contribution guidelines: fixes #35375

The issue describes the use case and implementation approach. Happy to address any review feedback — let me know if there's anything to update in the PR itself.

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

Labels

size:L This PR changes 100-499 lines, ignoring generated files. web This relates to changes on the web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant