Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 Fixed image resizing when sharp is not installed #10415

Closed
wants to merge 1 commit into from
Closed

🐛 Fixed image resizing when sharp is not installed #10415

wants to merge 1 commit into from

Conversation

allouis
Copy link
Contributor

@allouis allouis commented Jan 23, 2019

refs #10181

The ghost-ignition library overrides any passed props such as code
when passing an error, this fix removes the original err.code so that
we can be explicit in which errors we are throwing and catching.

refs #10181

The ghost-ignition library overrides any passed props such as `code`
when passing an error, this fix removes the original `err.code` so that
we can be explicit in which errors we are throwing and catching.
@allouis
Copy link
Contributor Author

allouis commented Jan 23, 2019

@kirrg001 What do you think of this - rather than fixing in ghost-ignition and bumping it? I think it would be a breaking change to ignition

@kirrg001
Copy link
Contributor

@allouis I think it can wait for Monday? What do you think? I remember image manipulation can be disabled temporary or?

@kirrg001
Copy link
Contributor

The ghost-ignition library overrides any passed props such as code
when passing an error, this fix removes the original err.code so that
we can be explicit in which errors we are throwing and catching.

IMO Ignition should respect code if you want to override it.

@allouis
Copy link
Contributor Author

allouis commented Jan 23, 2019

IMO Ignition should respect code if you want to override it.

I agree, the only problem is that it's a major change, and we'll have to check that nothing is relying on current behaviour!

I think it can wait for Monday?

Not sure what you mean by this? You mean fix ignition on Monday?

@kirrg001
Copy link
Contributor

Not sure what you mean by this? You mean fix ignition on Monday?

This report is not critical IMO. Something to work on bug fix Monday?

@allouis
Copy link
Contributor Author

allouis commented Jan 23, 2019

Yeah ☺️ But close this PR and fix ignition Monday, that's what you mean?

@kirrg001
Copy link
Contributor

Yeah i guess we close this PR for now and raise a proper bug fix issue, which get's the correct priority on the project board :)

@kirrg001 kirrg001 removed their assignment Jan 23, 2019
allouis added a commit to TryGhost/Ignition that referenced this pull request Jan 28, 2019
refs TryGhost/Ghost#10415
refs TryGhost/Ghost#10181
refs TryGhost/Ghost#10421

When supplying a code to an ignition error, if you pass an error with a code, the error's code takes precedence rather than the explicit code you pass.
@allouis
Copy link
Contributor Author

allouis commented Jan 28, 2019

Closing as this will be fixed in Ignition

@allouis allouis closed this Jan 28, 2019
@allouis allouis deleted the image-resizing-issue branch January 28, 2019 10:23
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