-
-
Notifications
You must be signed in to change notification settings - Fork 461
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
Renaming from update services #262
Comments
I think I see the issue. In Base::_get_updating_repos() I parse the $_REQUEST variable. https://github.com/afragen/github-updater/blob/develop/src/GitHub_Updater/Base.php#L525-L530 The $_REQUEST variable structure must be different for iThemes Sync, ManageWP, etc. and once I can get that info this should be fixed. |
iThemes Sync is now working again. Thanks @chrisbliss18 |
Just was listening to Apply Filters with @pippinsplugins and thought I should really just add a couple of hooks into the right places and figure out how to add support for remote update services via additional small classes. |
I've been testing with a local install of InfiniteWP and every seems to work. The only exceptions are that InfiniteWP doesn't seem to identify child themes for updates. |
It seems renaming from update services like iThemes Sync, ManageWP, and likely others is broken.
I'm reaching out to iThemes for help in figuring out the problem.
The text was updated successfully, but these errors were encountered: