{"action":"created","comment":{"url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/comments/120450810","pull_request_review_id":42414961,"id":120450810,"diff_hunk":"@@ -378,7 +378,6 @@ public long saveAsDraft(CMSUser user, Enrollment ticket) throws PortalServiceExc\n         }\n \n         if (ticket.getTicketId() == 0) {\n-            ticket.setTicketId(getSequence().getNextValue(Sequences.TICKET_SEQ));","path":"psm-app/cms-business-process/src/main/java/gov/medicaid/services/impl/ProviderEnrollmentServiceBean.java","position":4,"original_position":4,"commit_id":"b2ed85991988adab9f0858cd390cb7be4b9ade85","original_commit_id":"b2ed85991988adab9f0858cd390cb7be4b9ade85","user":{"login":"jasonaowen","id":1494855,"avatar_url":"https://avatars3.githubusercontent.com/u/1494855?v=3","gravatar_id":"","url":"https://api.github.com/users/jasonaowen","html_url":"https://github.com/jasonaowen","followers_url":"https://api.github.com/users/jasonaowen/followers","following_url":"https://api.github.com/users/jasonaowen/following{/other_user}","gists_url":"https://api.github.com/users/jasonaowen/gists{/gist_id}","starred_url":"https://api.github.com/users/jasonaowen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jasonaowen/subscriptions","organizations_url":"https://api.github.com/users/jasonaowen/orgs","repos_url":"https://api.github.com/users/jasonaowen/repos","events_url":"https://api.github.com/users/jasonaowen/events{/privacy}","received_events_url":"https://api.github.com/users/jasonaowen/received_events","type":"User","site_admin":false},"body":"The placeholder is `0`, and the `if` statement above verifies that the `ticketId` is already set to `0`.","created_at":"2017-06-06T19:00:37Z","updated_at":"2017-06-06T19:00:37Z","html_url":"https://github.com/OpenTechStrategies/psm/pull/83#discussion_r120450810","pull_request_url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/83","_links":{"self":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/comments/120450810"},"html":{"href":"https://github.com/OpenTechStrategies/psm/pull/83#discussion_r120450810"},"pull_request":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/83"}}},"pull_request":{"url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/83","id":124256200,"html_url":"https://github.com/OpenTechStrategies/psm/pull/83","diff_url":"https://github.com/OpenTechStrategies/psm/pull/83.diff","patch_url":"https://github.com/OpenTechStrategies/psm/pull/83.patch","issue_url":"https://api.github.com/repos/OpenTechStrategies/psm/issues/83","number":83,"state":"open","locked":false,"title":"Hibernate-5-ify Enrollment","user":{"login":"jasonaowen","id":1494855,"avatar_url":"https://avatars3.githubusercontent.com/u/1494855?v=3","gravatar_id":"","url":"https://api.github.com/users/jasonaowen","html_url":"https://github.com/jasonaowen","followers_url":"https://api.github.com/users/jasonaowen/followers","following_url":"https://api.github.com/users/jasonaowen/following{/other_user}","gists_url":"https://api.github.com/users/jasonaowen/gists{/gist_id}","starred_url":"https://api.github.com/users/jasonaowen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jasonaowen/subscriptions","organizations_url":"https://api.github.com/users/jasonaowen/orgs","repos_url":"https://api.github.com/users/jasonaowen/repos","events_url":"https://api.github.com/users/jasonaowen/events{/privacy}","received_events_url":"https://api.github.com/users/jasonaowen/received_events","type":"User","site_admin":false},"body":"Remove redundant comments, pluralize the table name, use a more descriptive column names, and generate primary key values using Hibernate natively.\r\n\r\nNote that the old table was named `TICKET`, and some of the fields (eg `ticketId`) and comments reflect that legacy name, as well. This is confusing and should be cleaned up later, but I argue it is out of scope at the moment and should be fixed as part of issue #57.\r\n\r\nIssue #36 Use Hibernate 5, instead of 4","created_at":"2017-06-06T18:22:13Z","updated_at":"2017-06-06T19:00:37Z","closed_at":null,"merged_at":null,"merge_commit_sha":"e6ccf559242467519bb6fd04edc40de86ba94bd1","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/83/commits","review_comments_url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/83/comments","review_comment_url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/comments{/number}","comments_url":"https://api.github.com/repos/OpenTechStrategies/psm/issues/83/comments","statuses_url":"https://api.github.com/repos/OpenTechStrategies/psm/statuses/b2ed85991988adab9f0858cd390cb7be4b9ade85","head":{"label":"OpenTechStrategies:36-hibernate-5-enrollment","ref":"36-hibernate-5-enrollment","sha":"b2ed85991988adab9f0858cd390cb7be4b9ade85","user":{"login":"OpenTechStrategies","id":1455458,"avatar_url":"https://avatars0.githubusercontent.com/u/1455458?v=3","gravatar_id":"","url":"https://api.github.com/users/OpenTechStrategies","html_url":"https://github.com/OpenTechStrategies","followers_url":"https://api.github.com/users/OpenTechStrategies/followers","following_url":"https://api.github.com/users/OpenTechStrategies/following{/other_user}","gists_url":"https://api.github.com/users/OpenTechStrategies/gists{/gist_id}","starred_url":"https://api.github.com/users/OpenTechStrategies/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OpenTechStrategies/subscriptions","organizations_url":"https://api.github.com/users/OpenTechStrategies/orgs","repos_url":"https://api.github.com/users/OpenTechStrategies/repos","events_url":"https://api.github.com/users/OpenTechStrategies/events{/privacy}","received_events_url":"https://api.github.com/users/OpenTechStrategies/received_events","type":"Organization","site_admin":false},"repo":{"id":66302616,"name":"psm","full_name":"OpenTechStrategies/psm","owner":{"login":"OpenTechStrategies","id":1455458,"avatar_url":"https://avatars0.githubusercontent.com/u/1455458?v=3","gravatar_id":"","url":"https://api.github.com/users/OpenTechStrategies","html_url":"https://github.com/OpenTechStrategies","followers_url":"https://api.github.com/users/OpenTechStrategies/followers","following_url":"https://api.github.com/users/OpenTechStrategies/following{/other_user}","gists_url":"https://api.github.com/users/OpenTechStrategies/gists{/gist_id}","starred_url":"https://api.github.com/users/OpenTechStrategies/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OpenTechStrategies/subscriptions","organizations_url":"https://api.github.com/users/OpenTechStrategies/orgs","repos_url":"https://api.github.com/users/OpenTechStrategies/repos","events_url":"https://api.github.com/users/OpenTechStrategies/events{/privacy}","received_events_url":"https://api.github.com/users/OpenTechStrategies/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/OpenTechStrategies/psm","description":"Welcome to the Medicare/Medicaid Provider Enrollment Screening Portal","fork":true,"url":"https://api.github.com/repos/OpenTechStrategies/psm","forks_url":"https://api.github.com/repos/OpenTechStrategies/psm/forks","keys_url":"https://api.github.com/repos/OpenTechStrategies/psm/keys{/key_id}","collaborators_url":"https://api.github.com/repos/OpenTechStrategies/psm/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/OpenTechStrategies/psm/teams","hooks_url":"https://api.github.com/repos/OpenTechStrategies/psm/hooks","issue_events_url":"https://api.github.com/repos/OpenTechStrategies/psm/issues/events{/number}","events_url":"https://api.github.com/repos/OpenTechStrategies/psm/events","assignees_url":"https://api.github.com/repos/OpenTechStrategies/psm/assignees{/user}","branches_url":"https://api.github.com/repos/OpenTechStrategies/psm/branches{/branch}","tags_url":"https://api.github.com/repos/OpenTechStrategies/psm/tags","blobs_url":"https://api.github.com/repos/OpenTechStrategies/psm/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/OpenTechStrategies/psm/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/OpenTechStrategies/psm/git/refs{/sha}","trees_url":"https://api.github.com/repos/OpenTechStrategies/psm/git/trees{/sha}","statuses_url":"https://api.github.com/repos/OpenTechStrategies/psm/statuses/{sha}","languages_url":"https://api.github.com/repos/OpenTechStrategies/psm/languages","stargazers_url":"https://api.github.com/repos/OpenTechStrategies/psm/stargazers","contributors_url":"https://api.github.com/repos/OpenTechStrategies/psm/contributors","subscribers_url":"https://api.github.com/repos/OpenTechStrategies/psm/subscribers","subscription_url":"https://api.github.com/repos/OpenTechStrategies/psm/subscription","commits_url":"https://api.github.com/repos/OpenTechStrategies/psm/commits{/sha}","git_commits_url":"https://api.github.com/repos/OpenTechStrategies/psm/git/commits{/sha}","comments_url":"https://api.github.com/repos/OpenTechStrategies/psm/comments{/number}","issue_comment_url":"https://api.github.com/repos/OpenTechStrategies/psm/issues/comments{/number}","contents_url":"https://api.github.com/repos/OpenTechStrategies/psm/contents/{+path}","compare_url":"https://api.github.com/repos/OpenTechStrategies/psm/compare/{base}...{head}","merges_url":"https://api.github.com/repos/OpenTechStrategies/psm/merges","archive_url":"https://api.github.com/repos/OpenTechStrategies/psm/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/OpenTechStrategies/psm/downloads","issues_url":"https://api.github.com/repos/OpenTechStrategies/psm/issues{/number}","pulls_url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls{/number}","milestones_url":"https://api.github.com/repos/OpenTechStrategies/psm/milestones{/number}","notifications_url":"https://api.github.com/repos/OpenTechStrategies/psm/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/OpenTechStrategies/psm/labels{/name}","releases_url":"https://api.github.com/repos/OpenTechStrategies/psm/releases{/id}","deployments_url":"https://api.github.com/repos/OpenTechStrategies/psm/deployments","created_at":"2016-08-22T19:40:02Z","updated_at":"2017-05-25T16:43:44Z","pushed_at":"2017-06-06T18:43:16Z","git_url":"git://github.com/OpenTechStrategies/psm.git","ssh_url":"git@github.com:OpenTechStrategies/psm.git","clone_url":"https://github.com/OpenTechStrategies/psm.git","svn_url":"https://github.com/OpenTechStrategies/psm","homepage":"","size":188848,"stargazers_count":1,"watchers_count":1,"language":"Java","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":1,"mirror_url":null,"open_issues_count":33,"forks":1,"open_issues":33,"watchers":1,"default_branch":"master"}},"base":{"label":"OpenTechStrategies:master","ref":"master","sha":"16861841d640e6c6ef4cb2cefeacc3ab5d21430d","user":{"login":"OpenTechStrategies","id":1455458,"avatar_url":"https://avatars0.githubusercontent.com/u/1455458?v=3","gravatar_id":"","url":"https://api.github.com/users/OpenTechStrategies","html_url":"https://github.com/OpenTechStrategies","followers_url":"https://api.github.com/users/OpenTechStrategies/followers","following_url":"https://api.github.com/users/OpenTechStrategies/following{/other_user}","gists_url":"https://api.github.com/users/OpenTechStrategies/gists{/gist_id}","starred_url":"https://api.github.com/users/OpenTechStrategies/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OpenTechStrategies/subscriptions","organizations_url":"https://api.github.com/users/OpenTechStrategies/orgs","repos_url":"https://api.github.com/users/OpenTechStrategies/repos","events_url":"https://api.github.com/users/OpenTechStrategies/events{/privacy}","received_events_url":"https://api.github.com/users/OpenTechStrategies/received_events","type":"Organization","site_admin":false},"repo":{"id":66302616,"name":"psm","full_name":"OpenTechStrategies/psm","owner":{"login":"OpenTechStrategies","id":1455458,"avatar_url":"https://avatars0.githubusercontent.com/u/1455458?v=3","gravatar_id":"","url":"https://api.github.com/users/OpenTechStrategies","html_url":"https://github.com/OpenTechStrategies","followers_url":"https://api.github.com/users/OpenTechStrategies/followers","following_url":"https://api.github.com/users/OpenTechStrategies/following{/other_user}","gists_url":"https://api.github.com/users/OpenTechStrategies/gists{/gist_id}","starred_url":"https://api.github.com/users/OpenTechStrategies/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OpenTechStrategies/subscriptions","organizations_url":"https://api.github.com/users/OpenTechStrategies/orgs","repos_url":"https://api.github.com/users/OpenTechStrategies/repos","events_url":"https://api.github.com/users/OpenTechStrategies/events{/privacy}","received_events_url":"https://api.github.com/users/OpenTechStrategies/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/OpenTechStrategies/psm","description":"Welcome to the Medicare/Medicaid Provider Enrollment Screening Portal","fork":true,"url":"https://api.github.com/repos/OpenTechStrategies/psm","forks_url":"https://api.github.com/repos/OpenTechStrategies/psm/forks","keys_url":"https://api.github.com/repos/OpenTechStrategies/psm/keys{/key_id}","collaborators_url":"https://api.github.com/repos/OpenTechStrategies/psm/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/OpenTechStrategies/psm/teams","hooks_url":"https://api.github.com/repos/OpenTechStrategies/psm/hooks","issue_events_url":"https://api.github.com/repos/OpenTechStrategies/psm/issues/events{/number}","events_url":"https://api.github.com/repos/OpenTechStrategies/psm/events","assignees_url":"https://api.github.com/repos/OpenTechStrategies/psm/assignees{/user}","branches_url":"https://api.github.com/repos/OpenTechStrategies/psm/branches{/branch}","tags_url":"https://api.github.com/repos/OpenTechStrategies/psm/tags","blobs_url":"https://api.github.com/repos/OpenTechStrategies/psm/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/OpenTechStrategies/psm/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/OpenTechStrategies/psm/git/refs{/sha}","trees_url":"https://api.github.com/repos/OpenTechStrategies/psm/git/trees{/sha}","statuses_url":"https://api.github.com/repos/OpenTechStrategies/psm/statuses/{sha}","languages_url":"https://api.github.com/repos/OpenTechStrategies/psm/languages","stargazers_url":"https://api.github.com/repos/OpenTechStrategies/psm/stargazers","contributors_url":"https://api.github.com/repos/OpenTechStrategies/psm/contributors","subscribers_url":"https://api.github.com/repos/OpenTechStrategies/psm/subscribers","subscription_url":"https://api.github.com/repos/OpenTechStrategies/psm/subscription","commits_url":"https://api.github.com/repos/OpenTechStrategies/psm/commits{/sha}","git_commits_url":"https://api.github.com/repos/OpenTechStrategies/psm/git/commits{/sha}","comments_url":"https://api.github.com/repos/OpenTechStrategies/psm/comments{/number}","issue_comment_url":"https://api.github.com/repos/OpenTechStrategies/psm/issues/comments{/number}","contents_url":"https://api.github.com/repos/OpenTechStrategies/psm/contents/{+path}","compare_url":"https://api.github.com/repos/OpenTechStrategies/psm/compare/{base}...{head}","merges_url":"https://api.github.com/repos/OpenTechStrategies/psm/merges","archive_url":"https://api.github.com/repos/OpenTechStrategies/psm/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/OpenTechStrategies/psm/downloads","issues_url":"https://api.github.com/repos/OpenTechStrategies/psm/issues{/number}","pulls_url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls{/number}","milestones_url":"https://api.github.com/repos/OpenTechStrategies/psm/milestones{/number}","notifications_url":"https://api.github.com/repos/OpenTechStrategies/psm/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/OpenTechStrategies/psm/labels{/name}","releases_url":"https://api.github.com/repos/OpenTechStrategies/psm/releases{/id}","deployments_url":"https://api.github.com/repos/OpenTechStrategies/psm/deployments","created_at":"2016-08-22T19:40:02Z","updated_at":"2017-05-25T16:43:44Z","pushed_at":"2017-06-06T18:43:16Z","git_url":"git://github.com/OpenTechStrategies/psm.git","ssh_url":"git@github.com:OpenTechStrategies/psm.git","clone_url":"https://github.com/OpenTechStrategies/psm.git","svn_url":"https://github.com/OpenTechStrategies/psm","homepage":"","size":188848,"stargazers_count":1,"watchers_count":1,"language":"Java","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":1,"mirror_url":null,"open_issues_count":33,"forks":1,"open_issues":33,"watchers":1,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/83"},"html":{"href":"https://github.com/OpenTechStrategies/psm/pull/83"},"issue":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/issues/83"},"comments":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/issues/83/comments"},"review_comments":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/83/comments"},"review_comment":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/83/commits"},"statuses":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/statuses/b2ed85991988adab9f0858cd390cb7be4b9ade85"}}}}
{"actor":{"display_login":"jasonaowen"}}