{"action":"created","issue":{"url":"https://api.github.com/repos/DevinVinson/WordPress-Plugin-Boilerplate/issues/397","repository_url":"https://api.github.com/repos/DevinVinson/WordPress-Plugin-Boilerplate","labels_url":"https://api.github.com/repos/DevinVinson/WordPress-Plugin-Boilerplate/issues/397/labels{/name}","comments_url":"https://api.github.com/repos/DevinVinson/WordPress-Plugin-Boilerplate/issues/397/comments","events_url":"https://api.github.com/repos/DevinVinson/WordPress-Plugin-Boilerplate/issues/397/events","html_url":"https://github.com/DevinVinson/WordPress-Plugin-Boilerplate/pull/397","id":175410221,"number":397,"title":"refactor using native add_action and add_filter functions","user":{"login":"yaronguez","id":384451,"avatar_url":"https://avatars.githubusercontent.com/u/384451?v=3","gravatar_id":"","url":"https://api.github.com/users/yaronguez","html_url":"https://github.com/yaronguez","followers_url":"https://api.github.com/users/yaronguez/followers","following_url":"https://api.github.com/users/yaronguez/following{/other_user}","gists_url":"https://api.github.com/users/yaronguez/gists{/gist_id}","starred_url":"https://api.github.com/users/yaronguez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yaronguez/subscriptions","organizations_url":"https://api.github.com/users/yaronguez/orgs","repos_url":"https://api.github.com/users/yaronguez/repos","events_url":"https://api.github.com/users/yaronguez/events{/privacy}","received_events_url":"https://api.github.com/users/yaronguez/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2016-09-07T04:27:16Z","updated_at":"2016-09-16T20:42:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/DevinVinson/WordPress-Plugin-Boilerplate/pulls/397","html_url":"https://github.com/DevinVinson/WordPress-Plugin-Boilerplate/pull/397","diff_url":"https://github.com/DevinVinson/WordPress-Plugin-Boilerplate/pull/397.diff","patch_url":"https://github.com/DevinVinson/WordPress-Plugin-Boilerplate/pull/397.patch"},"body":"See issue: https://github.com/DevinVinson/WordPress-Plugin-Boilerplate/issues/396\r\n\r\nBy using native add_action and add_filter functions you're able to use IDE benefits and stick to WordPress conventions, decreasing the learning curve to new plugin developers."},"comment":{"url":"https://api.github.com/repos/DevinVinson/WordPress-Plugin-Boilerplate/issues/comments/247704204","html_url":"https://github.com/DevinVinson/WordPress-Plugin-Boilerplate/pull/397#issuecomment-247704204","issue_url":"https://api.github.com/repos/DevinVinson/WordPress-Plugin-Boilerplate/issues/397","id":247704204,"user":{"login":"yaronguez","id":384451,"avatar_url":"https://avatars.githubusercontent.com/u/384451?v=3","gravatar_id":"","url":"https://api.github.com/users/yaronguez","html_url":"https://github.com/yaronguez","followers_url":"https://api.github.com/users/yaronguez/followers","following_url":"https://api.github.com/users/yaronguez/following{/other_user}","gists_url":"https://api.github.com/users/yaronguez/gists{/gist_id}","starred_url":"https://api.github.com/users/yaronguez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yaronguez/subscriptions","organizations_url":"https://api.github.com/users/yaronguez/orgs","repos_url":"https://api.github.com/users/yaronguez/repos","events_url":"https://api.github.com/users/yaronguez/events{/privacy}","received_events_url":"https://api.github.com/users/yaronguez/received_events","type":"User","site_admin":false},"created_at":"2016-09-16T20:42:16Z","updated_at":"2016-09-16T20:42:16Z","body":"After talking this over with Carl and other devs last night, I'd like to revisit the need to define all of your hooks at one time when the Run command is launched. The hook system doesn't care when a hook is created so why should the boilerplate go out of its way to group these events together?\r\n\r\nThe question of whether to define all of your hooks in one class (be it a Plugin_Name_Hooks class or the core Plugin_Name class) or to define them in the classes where each method is defined (e.g. Plugin_Name_Public->register_hooks()) is really a matter of personal preference. But at the very least I think we can do away with the custom Loader method.\r\n\r\nThoughts?\r\n\r\np.s. if you did try to watch the video stream last night, it got cut off when the videographer's phone died and was continued on another phone here: \r\nhttps://www.facebook.com/groups/advancedwp/permalink/1246482798747265/"}}
{"actor":{"display_login":"yaronguez"}}