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 remove alpha for iOS #464

Merged
merged 1 commit into from
Apr 7, 2023
Merged

Fix remove alpha for iOS #464

merged 1 commit into from
Apr 7, 2023

Conversation

mike-500
Copy link
Contributor

@mike-500 mike-500 commented Mar 2, 2023

Fix for #462.

After the dependency upgrade from image 3.x.x to image 4.0.x the remove alpha for iOS no longer works and builds are rejected from the App Store.
The fix is from the example here: brendan-duncan/image#447 (comment).

All tests are passing.

@lcsvcn
Copy link

lcsvcn commented Mar 9, 2023

I tested and it is working! Thanks @mike-500

Copy link

@tnaseem tnaseem left a comment

Choose a reason for hiding this comment

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

This fixes the problem for me.

@TheFabbiusCorp
Copy link

Hey @RatakondalaArun, @bradintheusa, can you please take a look? 🙏🏼

@vlazdra
Copy link
Contributor

vlazdra commented Mar 30, 2023

Is there really no one available to checkout these 2 lines of code? To get them merged and distributed to the community? 👀

Copy link
Contributor

@OutdatedGuy OutdatedGuy left a comment

Choose a reason for hiding this comment

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

LGTM!

@OutdatedGuy
Copy link
Contributor

@MarkOSullivan94, can you quickly take a look at this, cuz lots of apps are on hold due to app store restrictions.

@MarkOSullivan94
Copy link
Collaborator

Thanks for the ping @OutdatedGuy! Merging this in now and will have a release shortly available

@MarkOSullivan94 MarkOSullivan94 merged commit e57229d into fluttercommunity:master Apr 7, 2023
@MarkOSullivan94
Copy link
Collaborator

Thank you for submitting this change @mike-500 ❤️

@MarkOSullivan94
Copy link
Collaborator

Also big thank you to everyone who tested the change - makes it a lot quicker for me to ship this change ❤️

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.

9 participants