{"action":"opened","issue":{"url":"https://api.github.com/repos/WordPoints/wordpoints/issues/372","repository_url":"https://api.github.com/repos/WordPoints/wordpoints","labels_url":"https://api.github.com/repos/WordPoints/wordpoints/issues/372/labels{/name}","comments_url":"https://api.github.com/repos/WordPoints/wordpoints/issues/372/comments","events_url":"https://api.github.com/repos/WordPoints/wordpoints/issues/372/events","html_url":"https://github.com/WordPoints/wordpoints/issues/372","id":161708700,"number":372,"title":"Autoload all classes","user":{"login":"JDGrimes","id":4005415,"avatar_url":"https://avatars.githubusercontent.com/u/4005415?v=3","gravatar_id":"","url":"https://api.github.com/users/JDGrimes","html_url":"https://github.com/JDGrimes","followers_url":"https://api.github.com/users/JDGrimes/followers","following_url":"https://api.github.com/users/JDGrimes/following{/other_user}","gists_url":"https://api.github.com/users/JDGrimes/gists{/gist_id}","starred_url":"https://api.github.com/users/JDGrimes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JDGrimes/subscriptions","organizations_url":"https://api.github.com/users/JDGrimes/orgs","repos_url":"https://api.github.com/users/JDGrimes/repos","events_url":"https://api.github.com/users/JDGrimes/events{/privacy}","received_events_url":"https://api.github.com/users/JDGrimes/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/WordPoints/wordpoints/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-06-22T15:09:40Z","updated_at":"2016-06-22T15:09:40Z","closed_at":null,"body":"After the Hooks API was merged in #321, we now have a built-in autoloader. We should move all of our classes so that they can take advantage of this. This is a potentially breaking change, though it doesn't have to be, so we need to decide exactly how we want to do it. Probably not in 2.1.0.\r\n\r\nWhile merging the code I happened to think that we might as well put our admin classes in with the other classes in a single directory, since they'll be loaded only as needed. However, then I happened to think that our fallback will always load them them then if autoloading is disabled, so we might at least want to wait to do this until WordPress bumps its version requirement to 5.3. Also, our directory structure relating to the admin is currently such that everything for the admin is grouped together: classes, functions, assets, etc. Unless we change that, it would make sense to keep the admin classes with the other admin code, despite the fact that this adds an extra autoload directory."}}
{"actor":{"display_login":"JDGrimes"}}