Skip to content

Display the task's current position in the queue - #614

Open
julien-nc wants to merge 2 commits into
mainfrom
enh/noid/queue-position
Open

Display the task's current position in the queue#614
julien-nc wants to merge 2 commits into
mainfrom
enh/noid/queue-position

Conversation

@julien-nc

@julien-ncjulien-nc commented Jul 23, 2026

Copy link
Copy Markdown
Member

Poll the selected task's position every 5 seconds. Display it in the "loading" empty content.

  • In the loading empty content
image
  • In the chat UI
image

Todo

  • Adjust the message
  • Improve design

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
…holder
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc
julien-ncforce-pushed the enh/noid/queue-position branch from 4e0f58c to fc2090dCompareJuly 27, 2026 10:36
Comment threadsrc/assistant.js
})
}
// start polling immediately
pollPositionOnce()

@marcelklehrmarcelklehrAug 5, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This will reject immediately, because window.assistantPollPositionTimerId === null here, I think

const taskPosition = response.data?.ocs?.data
this.loading.taskPosition = taskPosition
console.debug('Task position:', taskPosition)
})

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No .catch?

}
this.loading = false
this.showSyncTaskRunning = false
this.taskPosition = null

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we call cancelTaskPositionPolling() here?

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

Labels

3. to reviewenhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@julien-nc@marcelklehr