{"action":"created","issue":{"url":"https://api.github.com/repos/SAP/openui5/issues/555","labels_url":"https://api.github.com/repos/SAP/openui5/issues/555/labels{/name}","comments_url":"https://api.github.com/repos/SAP/openui5/issues/555/comments","events_url":"https://api.github.com/repos/SAP/openui5/issues/555/events","html_url":"https://github.com/SAP/openui5/pull/555","id":98349585,"number":555,"title":"[Internal] - sap/m/demokit/helloworld using AMD style","user":{"login":"ikiw","id":4917563,"avatar_url":"https://avatars.githubusercontent.com/u/4917563?v=3","gravatar_id":"","url":"https://api.github.com/users/ikiw","html_url":"https://github.com/ikiw","followers_url":"https://api.github.com/users/ikiw/followers","following_url":"https://api.github.com/users/ikiw/following{/other_user}","gists_url":"https://api.github.com/users/ikiw/gists{/gist_id}","starred_url":"https://api.github.com/users/ikiw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ikiw/subscriptions","organizations_url":"https://api.github.com/users/ikiw/orgs","repos_url":"https://api.github.com/users/ikiw/repos","events_url":"https://api.github.com/users/ikiw/events{/privacy}","received_events_url":"https://api.github.com/users/ikiw/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-07-31T09:12:35Z","updated_at":"2015-07-31T11:22:05Z","closed_at":"2015-07-31T09:16:43Z","pull_request":{"url":"https://api.github.com/repos/SAP/openui5/pulls/555","html_url":"https://github.com/SAP/openui5/pull/555","diff_url":"https://github.com/SAP/openui5/pull/555.diff","patch_url":"https://github.com/SAP/openui5/pull/555.patch"},"body":"Adapting the demokit helloworld to AMD style using sap.ui.define"},"comment":{"url":"https://api.github.com/repos/SAP/openui5/issues/comments/126658271","html_url":"https://github.com/SAP/openui5/pull/555#issuecomment-126658271","issue_url":"https://api.github.com/repos/SAP/openui5/issues/555","id":126658271,"user":{"login":"akudev","id":4866568,"avatar_url":"https://avatars.githubusercontent.com/u/4866568?v=3","gravatar_id":"","url":"https://api.github.com/users/akudev","html_url":"https://github.com/akudev","followers_url":"https://api.github.com/users/akudev/followers","following_url":"https://api.github.com/users/akudev/following{/other_user}","gists_url":"https://api.github.com/users/akudev/gists{/gist_id}","starred_url":"https://api.github.com/users/akudev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akudev/subscriptions","organizations_url":"https://api.github.com/users/akudev/orgs","repos_url":"https://api.github.com/users/akudev/repos","events_url":"https://api.github.com/users/akudev/events{/privacy}","received_events_url":"https://api.github.com/users/akudev/received_events","type":"User","site_admin":false},"created_at":"2015-07-31T11:22:05Z","updated_at":"2015-07-31T11:22:05Z","body":"Ummm... I thought I had commented before, but I haven't. \r\nI'd think that in a Hello World example simplicity wins over being clean and best-practice compliant. Shouldn't a \"Hello World\" show about the minimum code required to display this text within a typical environment of the framework? Without distracting constructs that do not serve this goal?\r\n\r\nI'm totally for AMD syntax and showing it in our very next example, but maybe not in the very first?\r\n\r\nRegards\r\nAndreas"}}