Skip to content

AO3-6960 Admin post drafts and previews#5822

Open
marcus8448 wants to merge 18 commits into
otwcode:masterfrom
marcus8448:AO3-6960
Open

AO3-6960 Admin post drafts and previews#5822
marcus8448 wants to merge 18 commits into
otwcode:masterfrom
marcus8448:AO3-6960

Conversation

@marcus8448

@marcus8448 marcus8448 commented May 17, 2026

Copy link
Copy Markdown
Member

Issue

https://otwarchive.atlassian.net/browse/AO3-6960

Purpose

Allows admins to save unpublished (draft) admin posts and preview changes before publication.

Testing Instructions

See Jira.

Credit

marcus8448 (he/him)

@github-actions github-actions Bot added Has Migrations Contains migrations and therefore needs special attention when deploying Awaiting Review labels May 17, 2026

@pmonfort pmonfort left a comment

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.

Great work! The code looks good. Tested locally and the core flow works well. Left a few comments, mostly small things.

Comment thread app/controllers/admin_posts_controller.rb Outdated
Comment thread app/controllers/admin_posts_controller.rb Outdated
Comment thread app/controllers/admin_posts_controller.rb
Comment thread app/views/admin/_admin_nav.html.erb Outdated
Comment thread app/views/admin_posts/_posting_fieldset.html.erb Outdated
Comment thread features/admins/admin_post_draft_news.feature Outdated
Comment thread spec/helpers/admin_post_helper_spec.rb Outdated
Comment thread spec/models/admin_post_spec.rb
Comment thread app/models/admin_post.rb Outdated
Comment thread app/views/admin_posts/_admin_post.html.erb Outdated
Comment thread app/views/admin_posts/preview.html.erb Outdated
* copy translated post tags on validation
* don't save new tags on preview of newly-created admin post
  * Posts that are already persisted still need work
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Coder Has Actioned Review Has Migrations Contains migrations and therefore needs special attention when deploying

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants