Skip to content

use minimagick for image processing#252

Open
hari-co wants to merge 1 commit intomainfrom
alec/fix-journal-images
Open

use minimagick for image processing#252
hari-co wants to merge 1 commit intomainfrom
alec/fix-journal-images

Conversation

@hari-co
Copy link
Contributor

@hari-co hari-co commented Mar 16, 2026

TL;DR

Briefly describe what this PR does and why it is needed. Include any relevant background or context.


Fix journal image display after leaving the journal creation/edit page.

What is this PR trying to achieve?

Explain the main goal or problem this PR addresses. What functionality, bug fix, or improvement does it introduce?


Bug fix.

How did you achieve it?

Describe the approach, implementation details, or key changes made. Mention any libraries, refactors, or design choices involved.


Use mini magick.

Before:
image

After:
image

Checklist

  • Changes have been top-hatted locally
  • Tests have been added or updated
  • Documentation has been updated (if applicable)
  • Linked related issues

@hari-co hari-co requested review from CulmoneY, Copilot and pranavrao145 and removed request for Copilot March 16, 2026 23:55
Copy link
Contributor

@CulmoneY CulmoneY left a comment

Choose a reason for hiding this comment

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

Could we also use minimagick to resize/compress the images?

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