Skip to content

fix: display preview error for ZIP files#3485

Open
kyodai0000 wants to merge 1 commit into
inveniosoftware:masterfrom
oarepo:oarepo-contribution-display-zip-preview-error
Open

fix: display preview error for ZIP files#3485
kyodai0000 wants to merge 1 commit into
inveniosoftware:masterfrom
oarepo:oarepo-contribution-display-zip-preview-error

Conversation

@kyodai0000

Copy link
Copy Markdown
Contributor

Description

When previewing a ZIP file, an exception may be raised, for example when the maximum uncompressed size is exceeded. As a result, the preview window does not render the file preview and displays the landing page for its record instead.

This PR fixes the problem by catching the exception and displaying its error message inside the preview window. This allows the user to see the actual reason why the ZIP file preview cannot be displayed.

Checklist

Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

Frontend

Reminder

By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

  1. You license your contribution under the same terms as the current repository’s license.
  2. You agree that you have the right to license your contribution under the current repository’s license.

@kyodai0000 kyodai0000 added the bug Something isn't working label Jun 19, 2026
@kyodai0000 kyodai0000 added this to v14 Jun 19, 2026
@kyodai0000 kyodai0000 moved this to Triage in v14 Jun 19, 2026
@kyodai0000 kyodai0000 force-pushed the oarepo-contribution-display-zip-preview-error branch from 85e7d51 to bd22789 Compare June 19, 2026 13:11
@kyodai0000 kyodai0000 marked this pull request as ready for review June 19, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

1 participant