Skip to content

Display workflow link in queue only on assigned tasks#5545

Merged
6543 merged 1 commit into
mainfrom
fix-queue-info
Sep 23, 2025
Merged

Display workflow link in queue only on assigned tasks#5545
6543 merged 1 commit into
mainfrom
fix-queue-info

Conversation

@xoxys

@xoxys xoxys commented Sep 23, 2025

Copy link
Copy Markdown
Member

@xoxys xoxys requested a review from a team September 23, 2025 14:58
@xoxys xoxys added ui frontend related regression fix a bug that was not released yet labels Sep 23, 2025
@6543 6543 merged commit 9565722 into main Sep 23, 2025
7 checks passed
@6543 6543 deleted the fix-queue-info branch September 23, 2025 21:47
@anbraten

anbraten commented Sep 24, 2025

Copy link
Copy Markdown
Member

Not sure if this fix is ideal. The pipeline is known as soon as we create it from the webhook call. I think the issue is coming from

if task.AgentID == 0 {
result = append(result, taskResponse)
continue
skipping the pipeline number assignment in case there was no agent
taskResponse.PipelineNumber = p.Number

Instead it should only skip the pipeline number assignment in case of task.PipelineID being 0, but that should actually never be the case.

@xoxys

@xoxys

xoxys commented Sep 24, 2025

Copy link
Copy Markdown
Member Author

Fair point, Ill take another look 👍

@xoxys xoxys mentioned this pull request Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

regression fix a bug that was not released yet ui frontend related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants