{"action":"created","issue":{"url":"https://api.github.com/repos/sillsdev/LfMerge/issues/41","repository_url":"https://api.github.com/repos/sillsdev/LfMerge","labels_url":"https://api.github.com/repos/sillsdev/LfMerge/issues/41/labels{/name}","comments_url":"https://api.github.com/repos/sillsdev/LfMerge/issues/41/comments","events_url":"https://api.github.com/repos/sillsdev/LfMerge/issues/41/events","html_url":"https://github.com/sillsdev/LfMerge/pull/41","id":237470194,"number":41,"title":"Send/Receive Chorus Notes with LF comments","user":{"login":"rmunn","id":90762,"avatar_url":"https://avatars1.githubusercontent.com/u/90762?v=3","gravatar_id":"","url":"https://api.github.com/users/rmunn","html_url":"https://github.com/rmunn","followers_url":"https://api.github.com/users/rmunn/followers","following_url":"https://api.github.com/users/rmunn/following{/other_user}","gists_url":"https://api.github.com/users/rmunn/gists{/gist_id}","starred_url":"https://api.github.com/users/rmunn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rmunn/subscriptions","organizations_url":"https://api.github.com/users/rmunn/orgs","repos_url":"https://api.github.com/users/rmunn/repos","events_url":"https://api.github.com/users/rmunn/events{/privacy}","received_events_url":"https://api.github.com/users/rmunn/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-06-21T09:26:18Z","updated_at":"2017-06-23T06:07:55Z","closed_at":"2017-06-23T06:07:55Z","pull_request":{"url":"https://api.github.com/repos/sillsdev/LfMerge/pulls/41","html_url":"https://github.com/sillsdev/LfMerge/pull/41","diff_url":"https://github.com/sillsdev/LfMerge/pull/41.diff","patch_url":"https://github.com/sillsdev/LfMerge/pull/41.patch"},"body":"The corresponding [LfMergeBridge PR](https://github.com/sillsdev/flexbridge/pull/149) should be merged in first, then built, before this PR is merged. That will allow the TeamCity build process to obtain an LfMergeBridge DLL that knows the new GetChorusNotes and WriteToChorusNotes actions. If this PR is merged too soon, the resulting LfMerge.exe will run, but will throw an exception when you try to do a Send/Receive and it tries to handle comments.\r\n\r\nThere are no unit tests yet, but I'm creating the PR now so that we can see how it builds on TeamCity, and so that the code can be reviewed while we wait for the LfMergeBridge PR to be merged.\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/sillsdev/lfmerge/41)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/sillsdev/LfMerge/issues/comments/310580468","html_url":"https://github.com/sillsdev/LfMerge/pull/41#issuecomment-310580468","issue_url":"https://api.github.com/repos/sillsdev/LfMerge/issues/41","id":310580468,"user":{"login":"rmunn","id":90762,"avatar_url":"https://avatars1.githubusercontent.com/u/90762?v=3","gravatar_id":"","url":"https://api.github.com/users/rmunn","html_url":"https://github.com/rmunn","followers_url":"https://api.github.com/users/rmunn/followers","following_url":"https://api.github.com/users/rmunn/following{/other_user}","gists_url":"https://api.github.com/users/rmunn/gists{/gist_id}","starred_url":"https://api.github.com/users/rmunn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rmunn/subscriptions","organizations_url":"https://api.github.com/users/rmunn/orgs","repos_url":"https://api.github.com/users/rmunn/repos","events_url":"https://api.github.com/users/rmunn/events{/privacy}","received_events_url":"https://api.github.com/users/rmunn/received_events","type":"User","site_admin":false},"created_at":"2017-06-23T06:07:55Z","updated_at":"2017-06-23T06:07:55Z","body":"Closing and reopening to trigger another build."}}
{"actor":{"display_login":"rmunn"}}