nohype/main-site
5
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix Wagtail article publish regressions #58

Merged
mark merged 1 commits from fix/issue-57-admin-publish-regressions into main 2026-03-15 17:09:44 +00:00
Conversation 0 Commits 1 Files Changed 8 +161 -34
claude commented 2026-03-15 16:54:09 +00:00
Member
Copy Link

Summary

  • populate article admin defaults before Wagtail parent validation so slugs auto-generate reliably
  • stop rendering shared Django messages on the public frontend and use explicit article comment redirect notices instead
  • avoid unnecessary read-time rewrites on unchanged article bodies and cover the affected regressions with tests

Verification

  • docker compose run --rm web pytest
  • docker compose run --rm web ruff check .

Closes #57

## Summary - populate article admin defaults before Wagtail parent validation so slugs auto-generate reliably - stop rendering shared Django messages on the public frontend and use explicit article comment redirect notices instead - avoid unnecessary read-time rewrites on unchanged article bodies and cover the affected regressions with tests ## Verification - `docker compose run --rm web pytest` - `docker compose run --rm web ruff check .` Closes #57
claude added 1 commit 2026-03-15 16:54:09 +00:00
Fix Wagtail article publish regressions
All checks were successful
CI / nightly-e2e (pull_request) Has been skipped
Details
CI / deploy (pull_request) Has been skipped
Details
CI / ci (pull_request) Successful in 1m17s
Details
CI / pr-e2e (pull_request) Successful in 1m46s
Details
1a0617fbd0
mark merged commit fbc9a1ff0a into main 2026-03-15 17:09:44 +00:00
mark deleted branch fix/issue-57-admin-publish-regressions 2026-03-15 17:09:45 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claude codex_a codex_b codex_c mark
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: nohype/main-site#58
Delete Branch "fix/issue-57-admin-publish-regressions"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?