Skip to content

feat(admin): curator job run report and run log download#801

Open
TahaKhan998 wants to merge 1 commit into
CERNDocumentServer:masterfrom
TahaKhan998:feat/harvester-reports-curator
Open

feat(admin): curator job run report and run log download#801
TahaKhan998 wants to merge 1 commit into
CERNDocumentServer:masterfrom
TahaKhan998:feat/harvester-reports-curator

Conversation

@TahaKhan998
Copy link
Copy Markdown

Closes #789

  • when an inspire harvester job finishes we email people. that email again has a “harvester actions” section with two links: one to grab a text log for that run (“download error log”) and one to open harvester reports filtered to that run (“view list of changes”)

  • removed the old “view full details” button from that email — it pointed at the system jobs admin page and a lot of people on the mailing list don’t have access, so it was confusing

  • new page: /administration/harvester-reports/<run_id>/report (curators only). shows the same kind of run summary + log lines as the admin job ui, with colours by level so it’s easier to read than raw text

  • the existing download endpoint still returns a .log file; both that file and the html page use the real invenio jobs log search for the run, not a random audit log export

image image

- Register HTML report at /administration/harvester-reports/<run_id>/report (curator-only)

- Add report Jinja templates and extend HarvesterDownloadResource (report context + formatting)

- Search bar: effectiveRunId for download/view; DownloadButton: run log labels and report navigation

- LESS: harvester report log layout and wrapping
>
<Icon name="download" />
{i18next.t("Download")}
{i18next.t("Download run logs")}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
{i18next.t("Download run logs")}
{i18next.t("Download error logs")}

we need to be precise, otherwise I already saw some confusion on which logs report on failed records and which on the succeeded ones

size="small"
>
<Icon name="file alternate outline" />
{i18next.t("View run logs")}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
{i18next.t("View run logs")}
{i18next.t("View error logs")}

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.

feedback: harvester e-mail

2 participants