Skip to content

Conversation

@ramprasadagarwal
Copy link
Collaborator

What changes were proposed in this pull request?

  • When we are not able to read the file content due to any reason, the message will be displayed to indicate that preview is not supported.

How was this patch tested?

  • Added test case

Please review Hue Contributing Guide before opening a pull request.

@ramprasadagarwal ramprasadagarwal self-assigned this Mar 18, 2025
@github-actions
Copy link

⚠️ No unit test files modified. Please ensure that changes are properly tested. ⚠️

@github-actions
Copy link

github-actions bot commented Mar 18, 2025

Python Code Coverage

Python Coverage Report •
FileStmtsMissCoverMissing
TOTAL537132701149% 
report-only-changed-files is enabled. No files were changed during this commit :)

Pytest Report

Tests Skipped Failures Errors Time
1090 106 💤 0 ❌ 0 🔥 5m 53s ⏱️

Copy link
Contributor

@JohanAhlen JohanAhlen left a comment

Choose a reason for hiding this comment

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

Nice!

@github-actions
Copy link

github-actions bot commented Apr 9, 2025

✅ Test files were modified. Ensure that the tests cover all relevant changes. ✅

Instead of displaying raw binary content when file decoding fails, returns a user-friendly error message suggesting file download as an alternative.

Adds error logging to help diagnose encoding issues in server logs.
@ramprasadagarwal ramprasadagarwal changed the title [ui-storagebrowser] fixes file preview for non-readable file [ui-sb] fixes file preview for non-readable file Apr 28, 2025
@ramprasadagarwal ramprasadagarwal changed the base branch from master to fix-preview-sb April 28, 2025 14:53
Base automatically changed from fix-preview-sb to master April 29, 2025 06:45
@ramprasadagarwal ramprasadagarwal marked this pull request as ready for review April 29, 2025 10:16
@ramprasadagarwal ramprasadagarwal enabled auto-merge (squash) May 2, 2025 08:33
@ramprasadagarwal ramprasadagarwal merged commit 7f9d293 into master May 2, 2025
7 of 8 checks passed
@ramprasadagarwal ramprasadagarwal deleted the feat/storage-browser-37 branch May 2, 2025 09:33
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.

5 participants