Repository navigation
Clarify document PDF font permissions and trusted template export - #218
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #218 +/- ##
=====================================
Coverage 1.85% 1.85%
=====================================
Files 643 643
Lines 34921 34921
Branches 5808 5808
=====================================
Hits 647 647
Misses 34202 34202
Partials 72 72 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: db00e19ed5
ℹ️ 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".
PrzemyslawKlys
changed the title
Trusted-template examples now explicitly enable both system-font and document-font permissions, preserve the conversion report through
-PdfConversionReportVariable, and explain thatFontFamilyis a fallback for unspecified families.Word, Excel and PowerPoint option help now accurately describes embedding installed fonts selected by a document. The module build and website export regenerate the checked-in help artifacts and include the new template example in the website bundle.
Validation: all 11 PDF export tests pass with the current public OfficeIMO 3.4.4 dependency and with locally rebuilt source payloads on PowerShell 7 and Windows PowerShell 5.1. All nine website documentation checks pass, including the 536-command help bundle and example snapshot.
The template fidelity fixes are owned by OfficeIMO PR #2669. These wrapper/docs changes work with the existing public dependency; delivery of the engine fixes requires publishing OfficeIMO, updating this dependency, and rebuilding both module payloads. No unpublished dependency version is introduced here.