Skip to content

Show background Jobs in table - #830

Merged
four-random-common-words merged 3 commits into
886-cracker-binary-uploadfrom
886-background-jobs
Oct 22, 2026
Merged

four-random-common-words merged 3 commits into
886-cracker-binary-uploadfrom
886-background-jobs

Conversation

@four-random-common-words

@four-random-common-words four-random-common-words commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Issue: hashtopolis/server#886
Backend: hashtopolis/server#2502

Adds a Background Jobs page under Config that lists the server's background jobs with type, status, user, timestamps, exit code, message and payload, gated by its own role. Recounting the lines of a file now runs as a background job: the files table reports the recount as queued instead of finished and offers to turn on auto-refresh so the new count shows up. The backend pin and generated API types are updated to the background jobs stack.

@four-random-common-words
four-random-common-words added this pull request to stack #831 September 29, 2026 11:16
@four-random-common-words four-random-common-words changed the title 886 background jobs Show background Jobs in table Sep 29, 2026
@four-random-common-words
four-random-common-words marked this pull request as ready for review October 19, 2026 08:42
@four-random-common-words
four-random-common-words requested review from jessevz and novasam23 and removed request for jessevz October 20, 2026 08:27
@novasam23

Copy link
Copy Markdown
Contributor

LGTM

Maybe one minor thing, if sorting by either startedAt, finishedAt and exitCode, pagination breaks because these values are nullable and are indeed null in some scenarios.

But I think this is something that should ultimately be fixed in the backend. The backend should be enabled to deal with null values in its OrderFilter.php and PaginationFilter.php. But this is something that can be done later. For now the easiest would be to set isSortable: false for these fields in the frontend.

@s3inlc

@four-random-common-words
four-random-common-words merged commit 303ad18 into master Oct 22, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants