Skip to content

Fix issue with names decoding#28

Merged
KateOrient merged 2 commits into
customizations-for-tagged-pdffrom
fix/names-decoding
Jun 16, 2026
Merged

Fix issue with names decoding#28
KateOrient merged 2 commits into
customizations-for-tagged-pdffrom
fix/names-decoding

Conversation

@KateOrient

@KateOrient KateOrient commented Jun 16, 2026

Copy link
Copy Markdown

Closes: Wrong structure element names

PR includes the following changes:

  • Added a method to convert strings to UTF-8. This method first try classic decodeURIComponent conversion and if it fails uses a safe stringToPDFString method

@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c442d0ae-549d-4352-ab0e-186a76e5c666

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/names-decoding

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@KateOrient

Copy link
Copy Markdown
Author

@KateOrient KateOrient merged commit 7d77c62 into customizations-for-tagged-pdf Jun 16, 2026
2 of 13 checks passed
@KateOrient KateOrient deleted the fix/names-decoding branch June 16, 2026 13:20
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.

3 participants