Skip to content

feat(publish): Status job items open in the editor - #4898

Merged
natechadwick-intsof merged 3 commits into
mainfrom
fix/issue-4886-status-job-items
Sep 25, 2026
Merged

natechadwick-intsof merged 3 commits into
mainfrom
fix/issue-4886-status-job-items

Conversation

@natechadwick-intsof

@natechadwick-intsof natechadwick-intsof commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Parent: #4531. Slice #4886.

PublishingShell Status job detail lists that job's content items from the existing log-details API (POST …/sitemanage/pubstatus/details). Each row shows the content id and a name or title when the payload has one. An empty list is an empty state. Open in editor opens spa.jsp?entry=editor for that content id. A row with no content id has no Open control. HTTP 403/404 on the item list stay in the detail panel.

Operator: Grok: night-issue-prs (model grok-4.6)

Fixes #4886

Test plan

  • Vitest via WebUI clean install (5082 passed), including statusJobItems and status detail panel
  • Playwright surface tests/publishing/statusJobItems.spec.js — 4 passed (list+editor popup, empty state, 403, 404)
  • Human: open a real running job on Status and open one item in the editor

Product documentation

  • Updated product-docs/8.2/admin/publishing.md (Status job detail item list and Open in editor)

Build evidence

modules_built: WebUI, modules/perc-i18n

  • JAVA_HOME=/usr/lib/jvm/java-21-openjdk cd modules/perc-i18n && rtk mvn clean install — BUILD SUCCESS. Tests run: 33, Failures: 0
  • cd WebUI && rtk mvn clean install — BUILD SUCCESS. Java Tests run: 69, Failures: 0. Vitest: Test Files 551 passed, Tests 5082 passed
  • downstream_checked: none (no public/protected signature or final/sealed type change)
  • perc-qa-automation: Playwright specs only (no Java sources); not a Maven install

UI proof (C5)

  • python3 docker/scripts/perc-devctl.py qa-up — TEST_CMS_URL=http://127.0.0.1:9993 QA_CONTAINER=perc-matrix-cms-h2
  • qa-health RESULT:OK HEALTH:healthy (before deploy and after Jetty restart)
  • qa-deploy-webui copied WebUI/target/generated-webui/cm/modern
  • docker cp perc-i18n-8.2.0-SNAPSHOT.jar into webapps/Rhythmyx/WEB-INF/lib/
  • Jetty restarted inside the cell (/opt/Percussion/jetty/StopJetty.sh then StartJetty.sh); no docker restart
  • rtk npm run test:surface -- --path tests/publishing/statusJobItems.spec.js — 4 passed
  • console-clean=yes (spec asserts no pageerror / non-noise console errors)
  • server.log-clean=yes for the test window (no ERROR/FATAL after Jetty came up; earlier install-time empty-DB messages are pre-test)

Pre-push local code review

Summary

Machine analysis found 0 finding(s), 0 bug(s).

Scope

  • Base: origin/main
  • Head: HEAD
  • Files: 10 analyzed
  • Persona: erlang 0.1.1
  • Persona source: /home/nate/.local/share/mkd/agents/erlang

Recommendation

approve

Gate

  • Blocking bugs: 0
  • May commit/push: yes

Issues

No issues.

Co-Authored by Grok Build 1.0.41 using grok-4.6 with agent night-issue-prs-erlang.

Status job detail loads the existing pubstatus details payload, lists content id and name, and opens spa.jsp?entry=editor. Empty lists stay empty; 403/404 stay in the panel.

> Co-Authored by Grok Build 1.0.41 using grok-4.7 with agent night-issue-prs.
> Co-Authored by Grok Build 1.0.41 using grok-4.6 with agent night-issue-prs.
> Co-Authored by Grok Build 1.0.41 using grok-4.6 with agent night-issue-prs-erlang.
@natechadwick-intsof

Copy link
Copy Markdown
Collaborator Author

Independent Erlang review (persona erlang 0.1.1, mkd-code-review 0.1.18). Same-login APPROVE is rejected by GitHub. No in-diff blocking bugs. Recommendation: approve. Squash-merging with admin because review policy blocks the author account.

Co-Authored by Grok Build 1.0.41 using grok-4.6 with agent night-issue-prs-erlang.

@natechadwick-intsof
natechadwick-intsof merged commit ad5457a into main Sep 25, 2026
6 checks passed
@natechadwick-intsof
natechadwick-intsof deleted the fix/issue-4886-status-job-items branch September 25, 2026 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

model:grok-4.6 Session model grok-4.6 operator:grok Changes authored by Grok operator:night-issue-prs night-issue-prs workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

issue 4531 slice 37: PublishingShell status job items open in the editor

1 participant