Describe your changes and why you are making these changes
This PR updates all user-facing workflow links after UI refactor. There are only two places we produce such links (in notifications and in sdk) and we changed the way we refer to results from ?workflowDagResultId=<value> to result/<value>.
Related issue number (if any)
ENG-3063
Loom demo (if any)
Verified SDK, notification, and workflow breadcrumbs that they all points to the proper result / dag, for workflow with / without runs.
Checklist before requesting a review
[ ] I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
[ ] I have performed a self-review of my code.
[ ] I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
[ ] If this is a new feature, I have added unit tests and integration tests.
[ ] I have run the integration tests locally and they are passing.
[ ] I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
[ ] All features on the UI continue to work correctly.
[ ] Added one of the following CI labels:
run_integration_test: Runs integration tests
skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)
Describe your changes and why you are making these changes
This PR updates all user-facing workflow links after UI refactor. There are only two places we produce such links (in notifications and in sdk) and we changed the way we refer to results from
?workflowDagResultId=<value>toresult/<value>.Related issue number (if any)
ENG-3063
Loom demo (if any)
Verified SDK, notification, and workflow breadcrumbs that they all points to the proper result / dag, for workflow with / without runs.
Checklist before requesting a review
python3 scripts/run_linters.py -hfor usage).run_integration_test: Runs integration testsskip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)