{"action":"created","issue":{"url":"https://api.github.com/repos/phenotips/phenotips/issues/1738","labels_url":"https://api.github.com/repos/phenotips/phenotips/issues/1738/labels{/name}","comments_url":"https://api.github.com/repos/phenotips/phenotips/issues/1738/comments","events_url":"https://api.github.com/repos/phenotips/phenotips/issues/1738/events","html_url":"https://github.com/phenotips/phenotips/pull/1738","id":86233923,"number":1738,"title":"Develop","user":{"login":"jakennedy","id":8642627,"avatar_url":"https://avatars.githubusercontent.com/u/8642627?v=3","gravatar_id":"","url":"https://api.github.com/users/jakennedy","html_url":"https://github.com/jakennedy","followers_url":"https://api.github.com/users/jakennedy/followers","following_url":"https://api.github.com/users/jakennedy/following{/other_user}","gists_url":"https://api.github.com/users/jakennedy/gists{/gist_id}","starred_url":"https://api.github.com/users/jakennedy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jakennedy/subscriptions","organizations_url":"https://api.github.com/users/jakennedy/orgs","repos_url":"https://api.github.com/users/jakennedy/repos","events_url":"https://api.github.com/users/jakennedy/events{/privacy}","received_events_url":"https://api.github.com/users/jakennedy/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-06-08T16:00:56Z","updated_at":"2015-06-23T13:03:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/phenotips/phenotips/pulls/1738","html_url":"https://github.com/phenotips/phenotips/pull/1738","diff_url":"https://github.com/phenotips/phenotips/pull/1738.diff","patch_url":"https://github.com/phenotips/phenotips/pull/1738.patch"},"body":"Hi Sergiu, et. al., \r\n\r\nI am creating this pull request so you can look at the changes that were made to support storing attachments in S3 for Harvard's UDN project. We reused an existing XWiki contrib module [xwiki-platform-cloud](https://github.com/xwiki-contrib/xwiki-platform-cloud) and made some changes to make it work with 5.4.x. \r\n\r\nThe module was added as a subtree to our fork of the Phenotips project because we are only building one component. However, for full integration it would be better if the module itself was forked, improved, and added as a Phenotips dependency in Maven. \r\n\r\nWe would like the Phenotips team to add cloud integration to the project so we don't have to maintain this fork going forward. "},"comment":{"url":"https://api.github.com/repos/phenotips/phenotips/issues/comments/114493383","html_url":"https://github.com/phenotips/phenotips/pull/1738#issuecomment-114493383","issue_url":"https://api.github.com/repos/phenotips/phenotips/issues/1738","id":114493383,"user":{"login":"marta-","id":651980,"avatar_url":"https://avatars.githubusercontent.com/u/651980?v=3","gravatar_id":"","url":"https://api.github.com/users/marta-","html_url":"https://github.com/marta-","followers_url":"https://api.github.com/users/marta-/followers","following_url":"https://api.github.com/users/marta-/following{/other_user}","gists_url":"https://api.github.com/users/marta-/gists{/gist_id}","starred_url":"https://api.github.com/users/marta-/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marta-/subscriptions","organizations_url":"https://api.github.com/users/marta-/orgs","repos_url":"https://api.github.com/users/marta-/repos","events_url":"https://api.github.com/users/marta-/events{/privacy}","received_events_url":"https://api.github.com/users/marta-/received_events","type":"User","site_admin":false},"created_at":"2015-06-23T13:03:13Z","updated_at":"2015-06-23T13:03:13Z","body":"Yes. What will happen is:\r\n- we'll squash the changes to reflect cleanly what was modified wrt the code developed at xwiki-contrib\r\n- we will commit the code to xwiki-contrib so that the fix be there in the original code\r\n- we will then fork it, mentain it on our side and contribute back any relevant fixes and developments to stay in sync with the origin"}}