Skip to content

Conversation

@cstarcher
Copy link
Member

No description provided.

@cstarcher cstarcher requested a review from Copilot October 31, 2025 14:19
Copy link

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 PR updates the ExcelPackager to use the modern Excel file format by changing the file extension from "xls" to "xlsx".

  • Changed the return value of getFileExtension() from "xls" to "xlsx"
Comments suppressed due to low confidence (1)

src/main/java/org/tdl/vireo/model/packager/ExcelPackager.java:53

  • The MIME type 'application/vnd.ms-excel' is for the legacy .xls format, but the file extension has been changed to .xlsx. The MIME type should be updated to 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet' to match the .xlsx format.
    public String getMimeType() {
        return "application/vnd.ms-excel";
    }

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cstarcher cstarcher requested a review from Copilot October 31, 2025 14:42
Copy link

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 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@smutniak smutniak merged commit d32492d into main Oct 31, 2025
12 checks 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.

3 participants