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

Fix validation failures of jpg/jpeg files #23

Merged
merged 1 commit into from
Dec 23, 2020
Merged

Fix validation failures of jpg/jpeg files #23

merged 1 commit into from
Dec 23, 2020

Conversation

felixradtke
Copy link
Contributor

An update of symfony/mime renders the current validator incompatible. This issue is identical to the one from flarum/core's avatar uploader (flarum/framework#2497) and can be fixed in the same way. Confirmed lokally that the fix works.

An update of symfony/mime renders the current validator incompatible. This issue is identical to the one from flarum/core's avatar uploader (flarum/framework#2497) and can be fixed in the same way. Tested locally
SychO9
SychO9 previously requested changes Dec 22, 2020
Copy link
Owner

@SychO9 SychO9 left a comment

Choose a reason for hiding this comment

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

Hello, Thank you for the Pull Request!

@SychO9 SychO9 dismissed their stale review December 23, 2020 18:35

Moving forward

@SychO9 SychO9 merged commit 50b0422 into SychO9:master Dec 23, 2020
@XstraAI
Copy link

XstraAI commented Dec 24, 2020

Great extension thank you.

I am not a developer so please excuse my ignorance. How would I apply this fix? Can I just reinstall via composer and the fix will pull through? Or do I manually drop in the new src/CoverValidator.php?

@SychO9
Copy link
Owner

SychO9 commented Dec 24, 2020

I released this fix in v1.2.2 so just update the extension by running composer update sycho/flarum-profile-cover

@XstraAI
Copy link

XstraAI commented Dec 24, 2020

Perfect, thank you.

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.

3 participants