Skip to content

Conversation

@andruhovski
Copy link
Contributor

No description provided.

AnHolub and others added 30 commits August 14, 2025 12:26
fix: update installation guide for Aspose.PDF for Rust via C++
…/export-import-json/_index.md

Co-authored-by: Andriy Andruhovski <[email protected]>
feat: Add articles in Python/Acroform section
Renamed 'export-import-json' documentation to 'import-export-form-data' for clarity. Updated the sitemap change frequency for the modifying form documentation from weekly to monthly.
…/import-export-form-data/_index.md

Co-authored-by: Copilot <[email protected]>
Updated code examples to use explicit 'data_dir' and 'work_dir' variables for constructing input and output PDF file paths, replacing previous usage of 'self.dataDir'. This improves clarity and consistency in file path management across documentation for modifying and removing PDF forms.
Removed trailing whitespace and corrected minor formatting issues in the extract-form and import-export-form-data documentation. Updated section title to 'Extract Form Fields to JSON Document' for clarity.
feat: Add FAQ section for Aspose.PDF for Python via .NET with common …
AnHolub and others added 15 commits September 22, 2025 17:17
…mages-from-pdf-file/_index.md

Co-authored-by: Andriy Andruhovski <[email protected]>
…images-from-pdf-file/_index.md

Co-authored-by: Andriy Andruhovski <[email protected]>
…images-from-pdf-file/_index.md

Co-authored-by: Andriy Andruhovski <[email protected]>
…images-from-pdf-file/_index.md

Co-authored-by: Andriy Andruhovski <[email protected]>
…images-from-pdf-file/_index.md

Co-authored-by: Andriy Andruhovski <[email protected]>
feat: Added articles to Working with Images (Python)
fix: updated articles in Conversion section (Python)
@andruhovski andruhovski requested a review from Copilot October 1, 2025 13:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request adds new articles in Python via .NET sections, enhancing the documentation with comprehensive code examples and updated content. The changes include adding new release notes for version 25.5, creating several new conversion articles, expanding existing documentation, and adding new functionality examples.

  • Significant content addition to the "What's new" section with detailed code examples for PDF functionality
  • Addition of new file format conversion documentation (PDF/x formats, OFD, LaTeX)
  • Enhancement of existing conversion articles with more detailed examples and improved code structure
  • Addition of new image manipulation and PDF layer functionality documentation

Reviewed Changes

Copilot reviewed 24 out of 26 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
en/python-net/whatsnew/_index.md Comprehensive update with new version release notes and detailed Python code examples
en/python-net/faq/_index.md New FAQ page with structured questions and answers about Aspose.PDF capabilities
en/python-net/converting/convert-pdfx-to-pdf/_index.md New article for converting PDF/x formats to standard PDF
en/python-net/converting/convert-pdf-to-pdfx/_index.md New comprehensive article for converting PDF to various PDF/x formats
en/python-net/converting/convert-pdf-to-word/_index.md Updated with improved code examples and structure
en/python-net/converting/convert-pdf-to-powerpoint/_index.md Enhanced with additional conversion options and examples
en/python-net/converting/convert-pdf-to-other-files/_index.md Expanded with new format support including Markdown and MobiXML
en/python-net/converting/convert-other-files-to-pdf/_index.md Updated with new OFD and LaTeX conversion examples
en/python-net/converting/convert-images-format-to-pdf/_index.md Enhanced with additional image format support and improved examples
en/python-net/converting/convert-html-to-pdf/_index.md Significantly expanded with multiple HTML conversion scenarios
en/python-net/converting/convert-pdf-to-excel/_index.md Updated with improved code structure and additional format support
en/python-net/converting/convert-pdf-to-images-format/_index.md Enhanced with better code organization and additional examples
en/python-net/converting/_index.md Updated index with new conversion article references
en/python-net/advanced-operations/working-with-pages/adding-headers-and-footers-of-pdf-file/_index.md Enhanced with new header/footer examples including tables and LaTeX
en/python-net/advanced-operations/working-with-images/ Multiple new articles for image manipulation including search, replace, and extract functionality
en/python-net/advanced-operations/working-with-documents/working-with-layers/_index.md New comprehensive article for PDF layer manipulation
Comments suppressed due to low confidence (1)

en/python-net/converting/convert-images-format-to-pdf/_index.md:229

  • Missing page assignment after document.pages.add(). The code creates a page but doesn't assign it to a variable, then tries to use methods on an undefined 'page' variable in the next lines.
    page = document.pages.add()
    rectangle = apdf.Rectangle(0, 0, 595, 842, True)  # A4 size in points
    page.add_image(path_infile, rectangle)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@andruhovski andruhovski marked this pull request as ready for review October 1, 2025 14:07
Copy link
Contributor Author

@andruhovski andruhovski left a comment

Choose a reason for hiding this comment

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

Ok

@andruhovski andruhovski requested review from AnHolub and Copilot October 1, 2025 14:36
@andruhovski andruhovski self-assigned this Oct 1, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 20 out of 22 changed files in this pull request and generated 6 comments.

Comments suppressed due to low confidence (1)

en/python-net/converting/convert-pdf-to-other-files/_index.md:1

  • The function uses data_dir but this variable is not defined. It should be self.data_dir to match the pattern used elsewhere in the codebase.
---

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +421 to +424
path_infile = path.join(data_dir, infile)
path_outfile = path.join(data_dir, "python", outfile)
path_template = path.join(data_dir, template)
path_temp_file = path.join(data_dir, "temp.html")
Copy link

Copilot AI Oct 1, 2025

Choose a reason for hiding this comment

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

The function uses data_dir but this variable is not defined. It should be self.data_dir to match the pattern used elsewhere in the codebase.

Suggested change
path_infile = path.join(data_dir, infile)
path_outfile = path.join(data_dir, "python", outfile)
path_template = path.join(data_dir, template)
path_temp_file = path.join(data_dir, "temp.html")
path_infile = path.join(self.data_dir, infile)
path_outfile = path.join(self.data_dir, "python", outfile)
path_template = path.join(self.data_dir, template)
path_temp_file = path.join(self.data_dir, "temp.html")

Copilot uses AI. Check for mistakes.
image_names = list(document.pages[1].resources.images.names)

# Push identity matrix to stack
graphics_state.append(Matrix23(1, 0, 0, 1, 0, 0))
Copy link

Copilot AI Oct 1, 2025

Choose a reason for hiding this comment

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

The Matrix23 class is used but not imported or defined. This should likely be ap.Matrix or the appropriate matrix class from the aspose.pdf module.

Copilot uses AI. Check for mistakes.
@andruhovski andruhovski merged commit b0f9bce into master Oct 3, 2025
1 check passed
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.

2 participants