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

Logic with 1533 doesn't seem correct #2

Open
mark-netalico opened this issue Oct 20, 2016 · 2 comments
Open

Logic with 1533 doesn't seem correct #2

mark-netalico opened this issue Oct 20, 2016 · 2 comments

Comments

@mark-netalico
Copy link

Thanks for your work on this. The more turnkey we can make patching, the more likely stores will install it.

I've tried it on 2 sites so far (1.9.2.4 and 1.9.1.0) and it tried to revert 1533 when it wasn't actually necessary, which then broke the actual patch. I'm wondering if the latest versions of the patch have the 1533 issue fixed.

@mark-netalico
Copy link
Author

This script did work for me on a 1.9.0.1 and reverting 1533 was necessary.

@makanaashish
Copy link

The script doesn't work for me on 1.9.2.4

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

No branches or pull requests

2 participants