-
Notifications
You must be signed in to change notification settings - Fork 35
Rewritten - fixed all notices and errors - reference only #20
Comments
Very cool! So by the subject line I assume you mean that you fixed all the errors we were finding in #16, #18 and #19 ? And when you say "theme inclusion" do you mean that you are directly adding this into a theme versus requiring it be used as a plugin? In a quick look at the code I do see where you've hard-coded various aspects and in particular where you are focusing this on only one theme. It will be interesting to see if what you've done can fix the more generic case of looking at all the themes. Many thanks for providing this. (P.S. I'm not one of the core developers of this plugin but am a very interested user as I need it for some projects.) |
I'm not really sure if it fixes those issues. I found this in Guthub, noted that it failed miserably and then ported it directly into the theme. Yes, it now works from within the theme and updates via WordPress from Github using the Github API to compare tag versions. You can see it in action here... when we update of course! https://github.com/scarstens/whitelabel-framework Easier install with the download from the official website: |
Thanks for sharing. I picked through your stuff and think I snagged the correct pieces to try to apply the updater to my theme but it doesn't appear to be working. Here's my theme - if you notice any obvious pieces that I didn't apply, please let me know: https://github.com/ncsuwebdev/WordPress-theme--OITFlex.git. Appreciate it! |
@ncsumarit it looks like you converted it into a class... its hard to tell there is a lot going on in there. |
I'm working to incorporate these changes into this repo. Expect the plugin to be updated on the WordPress plugin page by the end of the day. |
Incorporated the changes while preserving the style.css directives. |
We used your code to build our own updater function. Biggest difference being we were doing it for theme inclusion instead of plugin inclusion. Several items were removed and several other items were hard coded. Please not this gist is the codes current state within our theme but hopefully it may have some references on how to fix your plugin to work correctly on WordPress again.
https://gist.github.com/3133020
The text was updated successfully, but these errors were encountered: