{"action":"created","issue":{"url":"https://api.github.com/repos/roots/bedrock/issues/232","repository_url":"https://api.github.com/repos/roots/bedrock","labels_url":"https://api.github.com/repos/roots/bedrock/issues/232/labels{/name}","comments_url":"https://api.github.com/repos/roots/bedrock/issues/232/comments","events_url":"https://api.github.com/repos/roots/bedrock/issues/232/events","html_url":"https://github.com/roots/bedrock/issues/232","id":129085056,"number":232,"title":"Content in 'app'?","user":{"login":"bitwombat","id":3937658,"avatar_url":"https://avatars.githubusercontent.com/u/3937658?v=3","gravatar_id":"","url":"https://api.github.com/users/bitwombat","html_url":"https://github.com/bitwombat","followers_url":"https://api.github.com/users/bitwombat/followers","following_url":"https://api.github.com/users/bitwombat/following{/other_user}","gists_url":"https://api.github.com/users/bitwombat/gists{/gist_id}","starred_url":"https://api.github.com/users/bitwombat/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bitwombat/subscriptions","organizations_url":"https://api.github.com/users/bitwombat/orgs","repos_url":"https://api.github.com/users/bitwombat/repos","events_url":"https://api.github.com/users/bitwombat/events{/privacy}","received_events_url":"https://api.github.com/users/bitwombat/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2016-01-27T09:34:34Z","updated_at":"2016-04-06T03:02:54Z","closed_at":"2016-01-27T22:04:06Z","body":"I hope this doesn't come off as a troll, but does it make sense to have content in the 'app' dir?  I've always hated how wp-content has code (plugins) AND content (uploads) mixed together in wp-content... renaming it to 'app' doesn't fix this... just now we have content in the 'app' directory, instead of code in the 'content' directory!\r\n\r\nOn my own here?"},"comment":{"url":"https://api.github.com/repos/roots/bedrock/issues/comments/206095306","html_url":"https://github.com/roots/bedrock/issues/232#issuecomment-206095306","issue_url":"https://api.github.com/repos/roots/bedrock/issues/232","id":206095306,"user":{"login":"retlehs","id":115911,"avatar_url":"https://avatars.githubusercontent.com/u/115911?v=3","gravatar_id":"","url":"https://api.github.com/users/retlehs","html_url":"https://github.com/retlehs","followers_url":"https://api.github.com/users/retlehs/followers","following_url":"https://api.github.com/users/retlehs/following{/other_user}","gists_url":"https://api.github.com/users/retlehs/gists{/gist_id}","starred_url":"https://api.github.com/users/retlehs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/retlehs/subscriptions","organizations_url":"https://api.github.com/users/retlehs/orgs","repos_url":"https://api.github.com/users/retlehs/repos","events_url":"https://api.github.com/users/retlehs/events{/privacy}","received_events_url":"https://api.github.com/users/retlehs/received_events","type":"User","site_admin":false},"created_at":"2016-04-06T03:02:54Z","updated_at":"2016-04-06T03:02:54Z","body":"> I don't see how having an uploads folder into an app folder is semantically correct.\r\n\r\nit's not really semantically correct in `wp-content` either. just use the `UPLOADS` constant\r\n\r\n> Is there any other reason that would explain the choice not to move to the uploads folder to its parent folder?\r\n\r\nyou can already do that pretty easily if you want. as @kalenjohnson mentioned:\r\n\r\n```define('UPLOADS', 'web/uploads/');```\r\n\r\ni completely agree that it should be `web/uploads/` (the old roots theme used to use `/assets/` in the webroot), but, we likely won't do this in bedrock as it's considered non-standard and usually frowned upon. by renaming `wp-content` to `app` we're not doing any harm, but the `uploads` directory naturally lives in the content directory on a wordpress installation. i guarantee there's plugins 'doing it wrong' and referencing the uploads directory relative to the content directory.\r\n"}}