{"action":"created","issue":{"url":"https://api.github.com/repos/diydrones/ardupilot/issues/2640","labels_url":"https://api.github.com/repos/diydrones/ardupilot/issues/2640/labels{/name}","comments_url":"https://api.github.com/repos/diydrones/ardupilot/issues/2640/comments","events_url":"https://api.github.com/repos/diydrones/ardupilot/issues/2640/events","html_url":"https://github.com/diydrones/ardupilot/pull/2640","id":98540263,"number":2640,"title":"DataFlash_test:make it works with pixhawk2","user":{"login":"rentt","id":2141741,"avatar_url":"https://avatars.githubusercontent.com/u/2141741?v=3","gravatar_id":"","url":"https://api.github.com/users/rentt","html_url":"https://github.com/rentt","followers_url":"https://api.github.com/users/rentt/followers","following_url":"https://api.github.com/users/rentt/following{/other_user}","gists_url":"https://api.github.com/users/rentt/gists{/gist_id}","starred_url":"https://api.github.com/users/rentt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rentt/subscriptions","organizations_url":"https://api.github.com/users/rentt/orgs","repos_url":"https://api.github.com/users/rentt/repos","events_url":"https://api.github.com/users/rentt/events{/privacy}","received_events_url":"https://api.github.com/users/rentt/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-08-01T14:21:35Z","updated_at":"2015-08-02T23:36:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/diydrones/ardupilot/pulls/2640","html_url":"https://github.com/diydrones/ardupilot/pull/2640","diff_url":"https://github.com/diydrones/ardupilot/pull/2640.diff","patch_url":"https://github.com/diydrones/ardupilot/pull/2640.patch"},"body":"I am new to APM code and still learning the examples of APM. The DataFlash_test app seems outdated and has compiling errors (make px4-v2). I have changed the code and made it work on PixhawkV2. I have not tested it on other platforms for I don't have the boards. Hope the developers can review the source and make it workable on other platforms.\r\n\r\nPS: For such a case, is it right to open a pull request instead of reporting an issue?"},"comment":{"url":"https://api.github.com/repos/diydrones/ardupilot/issues/comments/127083878","html_url":"https://github.com/diydrones/ardupilot/pull/2640#issuecomment-127083878","issue_url":"https://api.github.com/repos/diydrones/ardupilot/issues/2640","id":127083878,"user":{"login":"gmorph","id":1016642,"avatar_url":"https://avatars.githubusercontent.com/u/1016642?v=3","gravatar_id":"","url":"https://api.github.com/users/gmorph","html_url":"https://github.com/gmorph","followers_url":"https://api.github.com/users/gmorph/followers","following_url":"https://api.github.com/users/gmorph/following{/other_user}","gists_url":"https://api.github.com/users/gmorph/gists{/gist_id}","starred_url":"https://api.github.com/users/gmorph/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gmorph/subscriptions","organizations_url":"https://api.github.com/users/gmorph/orgs","repos_url":"https://api.github.com/users/gmorph/repos","events_url":"https://api.github.com/users/gmorph/events{/privacy}","received_events_url":"https://api.github.com/users/gmorph/received_events","type":"User","site_admin":false},"created_at":"2015-08-02T23:36:38Z","updated_at":"2015-08-02T23:36:38Z","body":"We can't include this PR as it explicitly breaks all other boardtypes except Pixhawk V2.  It need's to be fixed to work with all board types."}}