{"action":"created","issue":{"url":"https://api.github.com/repos/Jbarget/handlebars-morning-challenge/issues/16","repository_url":"https://api.github.com/repos/Jbarget/handlebars-morning-challenge","labels_url":"https://api.github.com/repos/Jbarget/handlebars-morning-challenge/issues/16/labels{/name}","comments_url":"https://api.github.com/repos/Jbarget/handlebars-morning-challenge/issues/16/comments","events_url":"https://api.github.com/repos/Jbarget/handlebars-morning-challenge/issues/16/events","html_url":"https://github.com/Jbarget/handlebars-morning-challenge/issues/16","id":218761447,"number":16,"title":"demonstration of \"best practices\"","user":{"login":"jsms90","id":15656538,"avatar_url":"https://avatars1.githubusercontent.com/u/15656538?v=3","gravatar_id":"","url":"https://api.github.com/users/jsms90","html_url":"https://github.com/jsms90","followers_url":"https://api.github.com/users/jsms90/followers","following_url":"https://api.github.com/users/jsms90/following{/other_user}","gists_url":"https://api.github.com/users/jsms90/gists{/gist_id}","starred_url":"https://api.github.com/users/jsms90/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jsms90/subscriptions","organizations_url":"https://api.github.com/users/jsms90/orgs","repos_url":"https://api.github.com/users/jsms90/repos","events_url":"https://api.github.com/users/jsms90/events{/privacy}","received_events_url":"https://api.github.com/users/jsms90/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-04-02T12:28:18Z","updated_at":"2017-04-03T14:55:45Z","closed_at":null,"body":"You mention that this isn't the way you would expect students' code to look:\r\n> NB. I know the code isn't very dry but this is just for demonstration purposes!\r\n\r\nBut maybe the explanation could be a bit clearer, about why you have chosen not to make it dry? And/or maybe you could add a DRY branch, just to show what this would look like?\r\n\r\nThis is particularly timely, since there have been similar references in other workshops, and FAC10 have expressed frustration around being told \"this is not the way you should do it\", with no offering of how it should/could be done, or why mentors are doing it differently.\r\n\r\nSee number 5 under Go in [their latest stop go continue](https://github.com/foundersandcoders/london-curriculum/blob/d314f755712bed05c561c8ffa380255387519cec/stop-go-continue/fac-10/cohort-week-6.md#go), which was particularly in reference to [these lines](https://github.com/FAC9/postgres-workshop/blob/master/data.sql#L60-L61) in a postgres workshop, but apparently there have been others."},"comment":{"url":"https://api.github.com/repos/Jbarget/handlebars-morning-challenge/issues/comments/291168371","html_url":"https://github.com/Jbarget/handlebars-morning-challenge/issues/16#issuecomment-291168371","issue_url":"https://api.github.com/repos/Jbarget/handlebars-morning-challenge/issues/16","id":291168371,"user":{"login":"Jbarget","id":11725595,"avatar_url":"https://avatars0.githubusercontent.com/u/11725595?v=3","gravatar_id":"","url":"https://api.github.com/users/Jbarget","html_url":"https://github.com/Jbarget","followers_url":"https://api.github.com/users/Jbarget/followers","following_url":"https://api.github.com/users/Jbarget/following{/other_user}","gists_url":"https://api.github.com/users/Jbarget/gists{/gist_id}","starred_url":"https://api.github.com/users/Jbarget/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Jbarget/subscriptions","organizations_url":"https://api.github.com/users/Jbarget/orgs","repos_url":"https://api.github.com/users/Jbarget/repos","events_url":"https://api.github.com/users/Jbarget/events{/privacy}","received_events_url":"https://api.github.com/users/Jbarget/received_events","type":"User","site_admin":false},"created_at":"2017-04-03T14:55:45Z","updated_at":"2017-04-03T14:55:45Z","body":"@jsms90 ive added a branch called `best-practice`. Let me know if you can spot anything there to fix"}}
{"actor":{"display_login":"Jbarget"}}