Skip to content

Fix post annotations lost on direct publish#10114

Merged
ruibaby merged 2 commits into
halo-dev:mainfrom
ruibaby:codex/fix-post-setting-publish-annotations
Jun 24, 2026
Merged

Fix post annotations lost on direct publish#10114
ruibaby merged 2 commits into
halo-dev:mainfrom
ruibaby:codex/fix-post-setting-publish-annotations

Conversation

@ruibaby

@ruibaby ruibaby commented Jun 23, 2026

Copy link
Copy Markdown
Member

What type of PR is this?

  • Feature
  • Bug fix
  • Improvement
  • Cleanup
  • Documentation

What this PR does / why we need it:

Syncs the post annotation form before publishing from the post settings modal. This makes direct publish use the same metadata collection path as Save, so a new unsaved post keeps the metadata configured in the publish flow.

Adds regression coverage for direct publish from PostSettingModal.

Which issue(s) this PR fixes:

Fixes #7420

Special notes for your reviewer:

Implemented with AI assistance and reviewed before submission.

Does this PR introduce a user-facing change?

修复首次发布文章时,设置的元数据可能出现没有保存的问题

@ruibaby ruibaby marked this pull request as ready for review June 23, 2026 09:25
@ruibaby ruibaby added this to the 2.25.4 milestone Jun 23, 2026
@codecov

codecov Bot commented Jun 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.95%. Comparing base (42c374d) to head (5893237).
⚠️ Report is 652 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #10114      +/-   ##
============================================
+ Coverage     59.55%   61.95%   +2.39%     
- Complexity     3812     4245     +433     
============================================
  Files           677      713      +36     
  Lines         23248    24398    +1150     
  Branches       1500     1623     +123     
============================================
+ Hits          13846    15116    +1270     
+ Misses         8764     8508     -256     
- Partials        638      774     +136     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ruibaby ruibaby merged commit 8206752 into halo-dev:main Jun 24, 2026
9 checks passed
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

新建文章未保存时直接提交会丢失设置的元数据

1 participant