{"action":"created","comment":{"url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/comments/83157767","pull_request_review_id":4007233,"id":83157767,"diff_hunk":"@@ -98,63 +128,49 @@ export class Compose extends React.Component {\n           </button>\n         </div>\n         <form\n-            id=\"messaging-compose\"\n+            id=\"msg-compose\"\n             onSubmit={(domEvent) => { domEvent.preventDefault(); }}>\n           <MessageFrom\n-              cssClass=\"messaging-from\"\n+              cssClass=\"msg-from\"\n               lastName={message.sender.lastName}\n               firstName={message.sender.firstName}\n               middleName={message.sender.middleName}/>\n           <MessageRecipient\n-              errorMessage={composeMessage.errors.recipient}\n-              cssClass=\"messaging-recipient\"\n+              errorMessage={validations.isValidRecipient(message.recipient) ? '' : composeMessage.errors.recipient}","path":"src/js/messaging/containers/Compose.jsx","position":93,"original_position":93,"commit_id":"d48cc90a167d89f111f5ab8f27381268cbf1e648","original_commit_id":"d48cc90a167d89f111f5ab8f27381268cbf1e648","user":{"login":"U-DON","id":1067024,"avatar_url":"https://avatars.githubusercontent.com/u/1067024?v=3","gravatar_id":"","url":"https://api.github.com/users/U-DON","html_url":"https://github.com/U-DON","followers_url":"https://api.github.com/users/U-DON/followers","following_url":"https://api.github.com/users/U-DON/following{/other_user}","gists_url":"https://api.github.com/users/U-DON/gists{/gist_id}","starred_url":"https://api.github.com/users/U-DON/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/U-DON/subscriptions","organizations_url":"https://api.github.com/users/U-DON/orgs","repos_url":"https://api.github.com/users/U-DON/repos","events_url":"https://api.github.com/users/U-DON/events{/privacy}","received_events_url":"https://api.github.com/users/U-DON/received_events","type":"User","site_admin":false},"body":"This is something that can be done separately from this PR, but I'm considering the idea of passing in fewer props if the components can self-contain them. For example, validation handling, classes that are likely to always be associated with the component, etc.\r\n\r\nSpecifically for this case, since `message.recipient` is already passed in the form of another prop (`recipient`), the validation can happen within the component itself.","created_at":"2016-10-13T07:33:51Z","updated_at":"2016-10-13T07:38:13Z","html_url":"https://github.com/department-of-veterans-affairs/vets-website/pull/3335#discussion_r83157767","pull_request_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/3335","_links":{"self":{"href":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/comments/83157767"},"html":{"href":"https://github.com/department-of-veterans-affairs/vets-website/pull/3335#discussion_r83157767"},"pull_request":{"href":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/3335"}}},"pull_request":{"url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/3335","id":89112391,"html_url":"https://github.com/department-of-veterans-affairs/vets-website/pull/3335","diff_url":"https://github.com/department-of-veterans-affairs/vets-website/pull/3335.diff","patch_url":"https://github.com/department-of-veterans-affairs/vets-website/pull/3335.patch","issue_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/issues/3335","number":3335,"state":"open","locked":false,"title":"Added error checking for Compose message feature","user":{"login":"webinista","id":354277,"avatar_url":"https://avatars.githubusercontent.com/u/354277?v=3","gravatar_id":"","url":"https://api.github.com/users/webinista","html_url":"https://github.com/webinista","followers_url":"https://api.github.com/users/webinista/followers","following_url":"https://api.github.com/users/webinista/following{/other_user}","gists_url":"https://api.github.com/users/webinista/gists{/gist_id}","starred_url":"https://api.github.com/users/webinista/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/webinista/subscriptions","organizations_url":"https://api.github.com/users/webinista/orgs","repos_url":"https://api.github.com/users/webinista/repos","events_url":"https://api.github.com/users/webinista/events{/privacy}","received_events_url":"https://api.github.com/users/webinista/received_events","type":"User","site_admin":false},"body":"Closes department-of-veterans-affairs/messaging-fe#55\r\n\r\n- Changed messaging/message-* class names to use msg-* prefix\r\n- Added conditional classes to MessageRecipient.\r\n- Added MessageSubjectGroup, MessageWriteGroup components.\r\n    - Moving MessageSubject, MessageCategory from Compose to MessageSubjectGroup\r\n    - Moving MessageWrite, MessageAttachments, MessageSend to MessageWriteGroup\r\n- Removed subjectRequired variable from Compose.\r\n- Consolidated composeMessage.errors.category and composeMessage.errors.subject into a composeMessage.subjectLine object.\r\n- Changing `DELETE_ATTACHMENT` to `DELETE_COMPOSE_ATTACHMENT`","created_at":"2016-10-12T23:25:37Z","updated_at":"2016-10-13T07:38:13Z","closed_at":null,"merged_at":null,"merge_commit_sha":"75d054db836b348991aabecf3bc2163fdbcae7de","assignee":{"login":"U-DON","id":1067024,"avatar_url":"https://avatars.githubusercontent.com/u/1067024?v=3","gravatar_id":"","url":"https://api.github.com/users/U-DON","html_url":"https://github.com/U-DON","followers_url":"https://api.github.com/users/U-DON/followers","following_url":"https://api.github.com/users/U-DON/following{/other_user}","gists_url":"https://api.github.com/users/U-DON/gists{/gist_id}","starred_url":"https://api.github.com/users/U-DON/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/U-DON/subscriptions","organizations_url":"https://api.github.com/users/U-DON/orgs","repos_url":"https://api.github.com/users/U-DON/repos","events_url":"https://api.github.com/users/U-DON/events{/privacy}","received_events_url":"https://api.github.com/users/U-DON/received_events","type":"User","site_admin":false},"assignees":[{"login":"U-DON","id":1067024,"avatar_url":"https://avatars.githubusercontent.com/u/1067024?v=3","gravatar_id":"","url":"https://api.github.com/users/U-DON","html_url":"https://github.com/U-DON","followers_url":"https://api.github.com/users/U-DON/followers","following_url":"https://api.github.com/users/U-DON/following{/other_user}","gists_url":"https://api.github.com/users/U-DON/gists{/gist_id}","starred_url":"https://api.github.com/users/U-DON/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/U-DON/subscriptions","organizations_url":"https://api.github.com/users/U-DON/orgs","repos_url":"https://api.github.com/users/U-DON/repos","events_url":"https://api.github.com/users/U-DON/events{/privacy}","received_events_url":"https://api.github.com/users/U-DON/received_events","type":"User","site_admin":false}],"milestone":null,"commits_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/3335/commits","review_comments_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/3335/comments","review_comment_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/comments{/number}","comments_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/issues/3335/comments","statuses_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/statuses/d48cc90a167d89f111f5ab8f27381268cbf1e648","head":{"label":"department-of-veterans-affairs:department-of-veterans-affairs/messaging-fe#55","ref":"department-of-veterans-affairs/messaging-fe#55","sha":"d48cc90a167d89f111f5ab8f27381268cbf1e648","user":{"login":"department-of-veterans-affairs","id":5421563,"avatar_url":"https://avatars.githubusercontent.com/u/5421563?v=3","gravatar_id":"","url":"https://api.github.com/users/department-of-veterans-affairs","html_url":"https://github.com/department-of-veterans-affairs","followers_url":"https://api.github.com/users/department-of-veterans-affairs/followers","following_url":"https://api.github.com/users/department-of-veterans-affairs/following{/other_user}","gists_url":"https://api.github.com/users/department-of-veterans-affairs/gists{/gist_id}","starred_url":"https://api.github.com/users/department-of-veterans-affairs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/department-of-veterans-affairs/subscriptions","organizations_url":"https://api.github.com/users/department-of-veterans-affairs/orgs","repos_url":"https://api.github.com/users/department-of-veterans-affairs/repos","events_url":"https://api.github.com/users/department-of-veterans-affairs/events{/privacy}","received_events_url":"https://api.github.com/users/department-of-veterans-affairs/received_events","type":"Organization","site_admin":false},"repo":{"id":33202667,"name":"vets-website","full_name":"department-of-veterans-affairs/vets-website","owner":{"login":"department-of-veterans-affairs","id":5421563,"avatar_url":"https://avatars.githubusercontent.com/u/5421563?v=3","gravatar_id":"","url":"https://api.github.com/users/department-of-veterans-affairs","html_url":"https://github.com/department-of-veterans-affairs","followers_url":"https://api.github.com/users/department-of-veterans-affairs/followers","following_url":"https://api.github.com/users/department-of-veterans-affairs/following{/other_user}","gists_url":"https://api.github.com/users/department-of-veterans-affairs/gists{/gist_id}","starred_url":"https://api.github.com/users/department-of-veterans-affairs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/department-of-veterans-affairs/subscriptions","organizations_url":"https://api.github.com/users/department-of-veterans-affairs/orgs","repos_url":"https://api.github.com/users/department-of-veterans-affairs/repos","events_url":"https://api.github.com/users/department-of-veterans-affairs/events{/privacy}","received_events_url":"https://api.github.com/users/department-of-veterans-affairs/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/department-of-veterans-affairs/vets-website","description":"Beta version of Vets.gov","fork":false,"url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website","forks_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/forks","keys_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/keys{/key_id}","collaborators_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/teams","hooks_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/hooks","issue_events_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/issues/events{/number}","events_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/events","assignees_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/assignees{/user}","branches_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/branches{/branch}","tags_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/tags","blobs_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/git/refs{/sha}","trees_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/git/trees{/sha}","statuses_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/statuses/{sha}","languages_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/languages","stargazers_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/stargazers","contributors_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/contributors","subscribers_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/subscribers","subscription_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/subscription","commits_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/commits{/sha}","git_commits_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/git/commits{/sha}","comments_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/comments{/number}","issue_comment_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/issues/comments{/number}","contents_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/contents/{+path}","compare_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/compare/{base}...{head}","merges_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/merges","archive_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/downloads","issues_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/issues{/number}","pulls_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls{/number}","milestones_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/milestones{/number}","notifications_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/labels{/name}","releases_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/releases{/id}","deployments_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/deployments","created_at":"2015-03-31T18:34:05Z","updated_at":"2016-10-08T11:22:57Z","pushed_at":"2016-10-13T04:45:21Z","git_url":"git://github.com/department-of-veterans-affairs/vets-website.git","ssh_url":"git@github.com:department-of-veterans-affairs/vets-website.git","clone_url":"https://github.com/department-of-veterans-affairs/vets-website.git","svn_url":"https://github.com/department-of-veterans-affairs/vets-website","homepage":"","size":105144,"stargazers_count":57,"watchers_count":57,"language":"JavaScript","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":31,"mirror_url":null,"open_issues_count":178,"forks":31,"open_issues":178,"watchers":57,"default_branch":"master"}},"base":{"label":"department-of-veterans-affairs:master","ref":"master","sha":"05aaedc20d4ca82ce5b87890e48ed6e055a773f7","user":{"login":"department-of-veterans-affairs","id":5421563,"avatar_url":"https://avatars.githubusercontent.com/u/5421563?v=3","gravatar_id":"","url":"https://api.github.com/users/department-of-veterans-affairs","html_url":"https://github.com/department-of-veterans-affairs","followers_url":"https://api.github.com/users/department-of-veterans-affairs/followers","following_url":"https://api.github.com/users/department-of-veterans-affairs/following{/other_user}","gists_url":"https://api.github.com/users/department-of-veterans-affairs/gists{/gist_id}","starred_url":"https://api.github.com/users/department-of-veterans-affairs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/department-of-veterans-affairs/subscriptions","organizations_url":"https://api.github.com/users/department-of-veterans-affairs/orgs","repos_url":"https://api.github.com/users/department-of-veterans-affairs/repos","events_url":"https://api.github.com/users/department-of-veterans-affairs/events{/privacy}","received_events_url":"https://api.github.com/users/department-of-veterans-affairs/received_events","type":"Organization","site_admin":false},"repo":{"id":33202667,"name":"vets-website","full_name":"department-of-veterans-affairs/vets-website","owner":{"login":"department-of-veterans-affairs","id":5421563,"avatar_url":"https://avatars.githubusercontent.com/u/5421563?v=3","gravatar_id":"","url":"https://api.github.com/users/department-of-veterans-affairs","html_url":"https://github.com/department-of-veterans-affairs","followers_url":"https://api.github.com/users/department-of-veterans-affairs/followers","following_url":"https://api.github.com/users/department-of-veterans-affairs/following{/other_user}","gists_url":"https://api.github.com/users/department-of-veterans-affairs/gists{/gist_id}","starred_url":"https://api.github.com/users/department-of-veterans-affairs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/department-of-veterans-affairs/subscriptions","organizations_url":"https://api.github.com/users/department-of-veterans-affairs/orgs","repos_url":"https://api.github.com/users/department-of-veterans-affairs/repos","events_url":"https://api.github.com/users/department-of-veterans-affairs/events{/privacy}","received_events_url":"https://api.github.com/users/department-of-veterans-affairs/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/department-of-veterans-affairs/vets-website","description":"Beta version of Vets.gov","fork":false,"url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website","forks_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/forks","keys_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/keys{/key_id}","collaborators_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/teams","hooks_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/hooks","issue_events_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/issues/events{/number}","events_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/events","assignees_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/assignees{/user}","branches_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/branches{/branch}","tags_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/tags","blobs_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/git/refs{/sha}","trees_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/git/trees{/sha}","statuses_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/statuses/{sha}","languages_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/languages","stargazers_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/stargazers","contributors_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/contributors","subscribers_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/subscribers","subscription_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/subscription","commits_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/commits{/sha}","git_commits_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/git/commits{/sha}","comments_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/comments{/number}","issue_comment_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/issues/comments{/number}","contents_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/contents/{+path}","compare_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/compare/{base}...{head}","merges_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/merges","archive_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/downloads","issues_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/issues{/number}","pulls_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls{/number}","milestones_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/milestones{/number}","notifications_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/labels{/name}","releases_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/releases{/id}","deployments_url":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/deployments","created_at":"2015-03-31T18:34:05Z","updated_at":"2016-10-08T11:22:57Z","pushed_at":"2016-10-13T04:45:21Z","git_url":"git://github.com/department-of-veterans-affairs/vets-website.git","ssh_url":"git@github.com:department-of-veterans-affairs/vets-website.git","clone_url":"https://github.com/department-of-veterans-affairs/vets-website.git","svn_url":"https://github.com/department-of-veterans-affairs/vets-website","homepage":"","size":105144,"stargazers_count":57,"watchers_count":57,"language":"JavaScript","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":31,"mirror_url":null,"open_issues_count":178,"forks":31,"open_issues":178,"watchers":57,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/3335"},"html":{"href":"https://github.com/department-of-veterans-affairs/vets-website/pull/3335"},"issue":{"href":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/issues/3335"},"comments":{"href":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/issues/3335/comments"},"review_comments":{"href":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/3335/comments"},"review_comment":{"href":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/pulls/3335/commits"},"statuses":{"href":"https://api.github.com/repos/department-of-veterans-affairs/vets-website/statuses/d48cc90a167d89f111f5ab8f27381268cbf1e648"}}}}
{"actor":{"display_login":"U-DON"}}