Repository navigation
feat(publish): open a log item in the content editor (#4766) - #4780
Merged
natechadwick-intsof merged 2 commits intoSep 23, 2026
Merged
Conversation
Publishing log item details probe item fields, then open the React editor for that content id. HTTP 403 and 404 stay on the log row. Fixes #4766 > 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.7 with agent night-issue-prs.
Collaborator
Author
|
LGTM. Independent Erlang pass (did not author): mkd-code-review 0.1.18, pack percussion, gate advisory, 0 bugs. Persona: erlang 0.1.1. Persona source: ~/.local/share/mkd/agents/erlang. Base origin/main, head 9c61335. Publish-log open probes item fields; 403/404 stay on the row and close the reserved popup. Vitest, Playwright, and product-docs are present. Required checks were green on this snapshot. GitHub rejected same-login APPROVE; squash-merging.
|
natechadwick-intsof
deleted the
fix/issue-4766-publish-log-open-editor
branch
September 23, 2026 15:49
3 of 4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
PublishingShell log item details can open the published content id in the React content editor. The shell probes
GET …/itemmanagement/item/fields/{id}first. HTTP 403 and 404 stay on the log item as an error and do not navigate the editor. A click reserves a popup so the open is not blocked after the probe; a failed probe closes that window.Parent: #4531. Fixes #4766.
Operator: Grok: night-issue-prs (model grok-4.7)
Test plan
python3 docker/scripts/perc-devctl.py qa-upthenqa-health(TEST_CMS_URL=http://127.0.0.1:9993, HEALTH:healthy). CopiedWebUI/target/generated-webui/cm/modern/assetsintoperc-matrix-cms-h2/opt/Percussion/jetty/base/webapps/Rhythmyx/cm/modern/assets/.qa-healthagain HEALTH:healthy.npm run test:surface -- --path tests/publishing-log-open-editor.spec.js— 3 passed. console-clean=yes (intentional 403/404 resource lines filtered). server.log-clean=yes.qa-downremoved the cell.Checklist
product-docs/8.2/admin/publishing.md(Logs: Open in editor, 403/404).openLogItemInEditorand LogDetailsPanel Vitest.modules/perc-qa-automation/frontend/tests/publishing-log-open-editor.spec.js(3 passed on H2).C3 evidence
cd WebUI && rtk mvn clean install— BUILD SUCCESS. Surefire Tests run: 69, Failures: 0. Vitest Test Files 528 passed, Tests 4870 passed. Pre-existing javadoc/dependency warnings only; no new compiler warnings from this change. npm EBADENGINE on Node 22.22.0 is the pinned frontend-maven Node (jsdom wants 22.22.2); Vitest stubsURL.createObjectURLwhen that runtime throws.C5 UI proof
python3 docker/scripts/perc-devctl.py qa-up— TEST_CMS_URL=http://127.0.0.1:9993 QA_CMS_HOST_PORT=9993cd modules/perc-qa-automation/frontend && npm run test:surface -- --path tests/publishing-log-open-editor.spec.js— 3 passedPre-push local code review