{"action":"created","comment":{"url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/comments/120410285","pull_request_review_id":42370864,"id":120410285,"diff_hunk":"@@ -30,20 +44,29 @@\n     /**\n      * Document data.\n      */\n+    @Column(name = \"body\")\n     private String text;\n \n+    @Column(name = \"created_by\")\n     private String createdBy;\n \n-    /**\n-     * Timestamp.\n-     */\n+    @Column(name = \"created_at\")\n     private Date createdOn;\n \n     /**\n      * If the agreement can be deleted.\n      */\n+    @Transient","path":"psm-app/services/src/main/java/gov/medicaid/entities/AgreementDocument.java","position":70,"original_position":44,"commit_id":"51765705c7eee15faec75bbd19f945bcb4524961","original_commit_id":"51765705c7eee15faec75bbd19f945bcb4524961","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 guideline I'm using so far is when the `Medicaid.hbm.xml` description of the table doesn't include that field.\r\n\r\nIn this specific case, deleting doesn't seem to be fully implemented - this is the same as with `ProviderType`s - see #41.","created_at":"2017-06-06T16:26:46Z","updated_at":"2017-06-06T16:26:46Z","html_url":"https://github.com/OpenTechStrategies/psm/pull/75#discussion_r120410285","pull_request_url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/75","_links":{"self":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/comments/120410285"},"html":{"href":"https://github.com/OpenTechStrategies/psm/pull/75#discussion_r120410285"},"pull_request":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/75"}}},"pull_request":{"url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/75","id":123984896,"html_url":"https://github.com/OpenTechStrategies/psm/pull/75","diff_url":"https://github.com/OpenTechStrategies/psm/pull/75.diff","patch_url":"https://github.com/OpenTechStrategies/psm/pull/75.patch","issue_url":"https://api.github.com/repos/OpenTechStrategies/psm/issues/75","number":75,"state":"open","locked":false,"title":"Hibernate-5-ify AgreementDocuments","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, remove the empty constructor, pluralize the table name, use a more descriptive primary key column name, and generate primary key values using Hibernate natively.\r\n\r\nI tested by creating an Agreement and creating an Agreement Addendum. Note that logging in as `admin` appears to fail because of a missing table (`ticket`), but it does give you a cookie, and you can enter the URL of the agreements page directly: https://localhost:8443/cms/admin/viewAgreementDocuments . Editing agreements is still blocked by provider settings. Deleting agreements appears not to have been fully implemented; the UI is present but disabled.\r\n\r\nIssue #36 Use Hibernate 5, instead of 4","created_at":"2017-06-05T14:12:23Z","updated_at":"2017-06-06T16:26:46Z","closed_at":null,"merged_at":null,"merge_commit_sha":"db36b24aedc46fb0e6640eb2bf71be5c5721e183","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/75/commits","review_comments_url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/75/comments","review_comment_url":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/comments{/number}","comments_url":"https://api.github.com/repos/OpenTechStrategies/psm/issues/75/comments","statuses_url":"https://api.github.com/repos/OpenTechStrategies/psm/statuses/51765705c7eee15faec75bbd19f945bcb4524961","head":{"label":"OpenTechStrategies:36-hibernate-5-agreement-document","ref":"36-hibernate-5-agreement-document","sha":"51765705c7eee15faec75bbd19f945bcb4524961","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-06T15:32:47Z","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":188829,"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":34,"forks":1,"open_issues":34,"watchers":1,"default_branch":"master"}},"base":{"label":"OpenTechStrategies:master","ref":"master","sha":"a5c4a2d21091bb401dec8feae11fa4cd08b4ab89","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-06T15:32:47Z","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":188829,"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":34,"forks":1,"open_issues":34,"watchers":1,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/75"},"html":{"href":"https://github.com/OpenTechStrategies/psm/pull/75"},"issue":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/issues/75"},"comments":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/issues/75/comments"},"review_comments":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/75/comments"},"review_comment":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/pulls/75/commits"},"statuses":{"href":"https://api.github.com/repos/OpenTechStrategies/psm/statuses/51765705c7eee15faec75bbd19f945bcb4524961"}}}}
{"actor":{"display_login":"jasonaowen"}}