Skip to content

feat(admin): harden content asset workflow#12

Merged
coding-hui merged 2 commits into
mainfrom
feat/admin-content-assets
May 17, 2026
Merged

feat(admin): harden content asset workflow#12
coding-hui merged 2 commits into
mainfrom
feat/admin-content-assets

Conversation

@coding-hui
Copy link
Copy Markdown
Owner

Summary

  • Block published questions from being unpublished while they are referenced by published papers.
  • Require programming questions to have at least one sample test case and one hidden test case before publish.
  • Add backend coverage for dependency-safe batch status actions and mark Batch 2 complete in the implementation plan.

Impact

Admins now receive per-item batch status results with clear backend reasons when content asset actions are blocked. Published papers remain protected from silently referencing unpublished questions.

Verification

  • go test ./...
  • go test ./internal/library ./internal/api
  • pnpm --dir apps/admin jest --runInBand
  • pnpm --dir apps/admin lint
  • pnpm --dir apps/admin build
  • Chrome regression for batch status partial failure, programming publish readiness, paper detail save-bar readiness summary, and migrated legacy 404 routes.

@coding-hui coding-hui marked this pull request as ready for review May 17, 2026 03:56
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9e0e043d7b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread internal/library/question.go Outdated
@coding-hui coding-hui merged commit f31f8e6 into main May 17, 2026
2 checks passed
@coding-hui coding-hui deleted the feat/admin-content-assets branch May 17, 2026 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant