Skip to content

Conversation

Martin-Rehr
Copy link
Contributor

Remove deprecated and non-maintained image preview functionality along with it's helper modules

@jonasbardino jonasbardino added the refactor Non-functional changes to simplify or clean up label Aug 11, 2025
@@ -961,7 +961,6 @@ Alias /status-events.json "/home/mig/state/wwwpublic/status-events.json"
RewriteCond %{REQUEST_URI} ^/favicon.ico$ [OR]
RewriteCond %{REQUEST_URI} ^/(images|assets|public|.well-known)/ [OR]
RewriteCond %{REQUEST_URI} ^/(seafmedia|seafhttp|seafile)/ [OR]
RewriteCond %{REQUEST_URI} ^/paraview/
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a reminder that this fixture file needs all the same updates after fixing the [OR] issue above in the template.

Copy link
Contributor

@jonasbardino jonasbardino left a comment

Choose a reason for hiding this comment

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

Looks good and ready for merge as soon as the RewriteCondition comment issue is handled.

Copy link
Contributor

@jonasbardino jonasbardino left a comment

Choose a reason for hiding this comment

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

Looks almost complete, but the fixtures MiG.conf file still needs to be updated accordingly as the continuous integration checks point out. Feel free to merge once resolved.

@Martin-Rehr Martin-Rehr merged commit ba904c2 into next Sep 1, 2025
5 of 7 checks passed
@Martin-Rehr Martin-Rehr deleted the adjust/remove-deprecated-image-preview branch September 1, 2025 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Non-functional changes to simplify or clean up
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants