Skip to content

Conversation

@aglinxinyuan
Copy link
Contributor

@aglinxinyuan aglinxinyuan commented Feb 10, 2026

What changes were proposed in this PR?

Fix the URL typo in the workflow-sharing notification email. The prefix dashboard/user was missing.

For example, the old URL https://hub.texera.io/workflow/2358 should be updated to https://hub.texera.io/dashboard/user/workflow/2358.

How was this PR tested?

Tested with existing test cases.

Was this PR authored or co-authored using generative AI tooling?

No

@aglinxinyuan aglinxinyuan requested a review from aicam February 10, 2026 05:48
@aglinxinyuan aglinxinyuan self-assigned this Feb 10, 2026
@github-actions github-actions bot added frontend Changes related to the frontend GUI and removed fix labels Feb 10, 2026
@aglinxinyuan aglinxinyuan changed the title init fix: the URL typo in the workflow-sharing notification email Feb 10, 2026
@chenlica
Copy link
Contributor

@aglinxinyuan Please include more details about the old URL and the new URL.

@aglinxinyuan
Copy link
Contributor Author

aglinxinyuan commented Feb 10, 2026

@aglinxinyuan Please include more details about the old URL and the new URL.

The prefix dashboard/user was missing.
For example, the old URL https://hub.texera.io/workflow/2358 should be updated to https://hub.texera.io/dashboard/user/workflow/2358.

@chenlica
Copy link
Contributor

I feel the format https://hub.texera.io/workflow/2358 is better than https://hub.texera.io/dashboard/user/workflow/2358, especially since a workflow ID is globally unique. Can we keep the first format?

@aglinxinyuan
Copy link
Contributor Author

I feel the format https://hub.texera.io/workflow/2358 is better than https://hub.texera.io/dashboard/user/workflow/2358, especially since a workflow ID is globally unique. Can we keep the first format?

That will require a lot more changes. We can have a group discussion and do it in a separate PR.

@chenlica
Copy link
Contributor

Let's have an offline discussion and report the results here.

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

Labels

frontend Changes related to the frontend GUI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants