Skip to content

Fix buildlist UI: include failed executions when retrieving end timestamps - #73

Open
krixkrix wants to merge 1 commit into
alexanderoster:developfrom
krixkrix:fix_buildlist_for_failed_jobs
Open

krixkrix wants to merge 1 commit into
alexanderoster:developfrom
krixkrix:fix_buildlist_for_failed_jobs

Conversation

@krixkrix

@krixkrix krixkrix commented Feb 7, 2026 •

Copy link
Copy Markdown

Now also failed/aborted builds will show their completion time in the execution list.
Before, the UI showed the end time stamp as a formatted Unix time stamp 0 (Jan 1, 1970).

Rebased on the current develop: the execution list entries are now written in Implementation/API/amc_api_handler_executions.cpp (/api/executions), so the fix moved there from amc_ui_module_contentitem_executionlist.cpp. CBuildJobExecution::GetEndTimeStampInMicroseconds() already accepts both Finished and Failed executions.

Failed (e.g. aborted) executions also have an end time stamp, but the
execution list only read it for finished ones, so the client showed the
end time as a formatted Unix time stamp 0 (Jan 1, 1970).
@krixkrix
krixkrix force-pushed the fix_buildlist_for_failed_jobs branch from 8563a69 to 8f669a6 Compare October 9, 2026 13:55

This branch has not been deployed

No deployments
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.

1 participant