{"action":"created","issue":{"url":"https://api.github.com/repos/codice/ddf/issues/3818","repository_url":"https://api.github.com/repos/codice/ddf","labels_url":"https://api.github.com/repos/codice/ddf/issues/3818/labels{/name}","comments_url":"https://api.github.com/repos/codice/ddf/issues/3818/comments","events_url":"https://api.github.com/repos/codice/ddf/issues/3818/events","html_url":"https://github.com/codice/ddf/pull/3818","id":364967427,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE5MDI4NTM4","number":3818,"title":"DDF-4186 Modifying ACL to incorporate R/W permissions on Metacards","user":{"login":"Schachte","id":7055226,"node_id":"MDQ6VXNlcjcwNTUyMjY=","avatar_url":"https://avatars1.githubusercontent.com/u/7055226?v=4","gravatar_id":"","url":"https://api.github.com/users/Schachte","html_url":"https://github.com/Schachte","followers_url":"https://api.github.com/users/Schachte/followers","following_url":"https://api.github.com/users/Schachte/following{/other_user}","gists_url":"https://api.github.com/users/Schachte/gists{/gist_id}","starred_url":"https://api.github.com/users/Schachte/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Schachte/subscriptions","organizations_url":"https://api.github.com/users/Schachte/orgs","repos_url":"https://api.github.com/users/Schachte/repos","events_url":"https://api.github.com/users/Schachte/events{/privacy}","received_events_url":"https://api.github.com/users/Schachte/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"djblue","id":1986211,"node_id":"MDQ6VXNlcjE5ODYyMTE=","avatar_url":"https://avatars2.githubusercontent.com/u/1986211?v=4","gravatar_id":"","url":"https://api.github.com/users/djblue","html_url":"https://github.com/djblue","followers_url":"https://api.github.com/users/djblue/followers","following_url":"https://api.github.com/users/djblue/following{/other_user}","gists_url":"https://api.github.com/users/djblue/gists{/gist_id}","starred_url":"https://api.github.com/users/djblue/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/djblue/subscriptions","organizations_url":"https://api.github.com/users/djblue/orgs","repos_url":"https://api.github.com/users/djblue/repos","events_url":"https://api.github.com/users/djblue/events{/privacy}","received_events_url":"https://api.github.com/users/djblue/received_events","type":"User","site_admin":false},"assignees":[{"login":"djblue","id":1986211,"node_id":"MDQ6VXNlcjE5ODYyMTE=","avatar_url":"https://avatars2.githubusercontent.com/u/1986211?v=4","gravatar_id":"","url":"https://api.github.com/users/djblue","html_url":"https://github.com/djblue","followers_url":"https://api.github.com/users/djblue/followers","following_url":"https://api.github.com/users/djblue/following{/other_user}","gists_url":"https://api.github.com/users/djblue/gists{/gist_id}","starred_url":"https://api.github.com/users/djblue/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/djblue/subscriptions","organizations_url":"https://api.github.com/users/djblue/orgs","repos_url":"https://api.github.com/users/djblue/repos","events_url":"https://api.github.com/users/djblue/events{/privacy}","received_events_url":"https://api.github.com/users/djblue/received_events","type":"User","site_admin":false},{"login":"bdeining","id":6551038,"node_id":"MDQ6VXNlcjY1NTEwMzg=","avatar_url":"https://avatars2.githubusercontent.com/u/6551038?v=4","gravatar_id":"","url":"https://api.github.com/users/bdeining","html_url":"https://github.com/bdeining","followers_url":"https://api.github.com/users/bdeining/followers","following_url":"https://api.github.com/users/bdeining/following{/other_user}","gists_url":"https://api.github.com/users/bdeining/gists{/gist_id}","starred_url":"https://api.github.com/users/bdeining/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bdeining/subscriptions","organizations_url":"https://api.github.com/users/bdeining/orgs","repos_url":"https://api.github.com/users/bdeining/repos","events_url":"https://api.github.com/users/bdeining/events{/privacy}","received_events_url":"https://api.github.com/users/bdeining/received_events","type":"User","site_admin":false}],"milestone":null,"comments":22,"created_at":"2018-09-28T16:56:26Z","updated_at":"2018-10-11T05:47:24Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/codice/ddf/pulls/3818","html_url":"https://github.com/codice/ddf/pull/3818","diff_url":"https://github.com/codice/ddf/pull/3818.diff","patch_url":"https://github.com/codice/ddf/pull/3818.patch"},"body":"#### What does this PR do?\r\nAdds an additional level of granularity to the ACL data model to incorporate read/write perms. \r\n#### Who is reviewing it? \r\n<!--(please choose AT LEAST two reviewers that need to approve the PR before it can get merged)-->\r\n#### Select relevant component teams: \r\nhttps://github.com/orgs/codice/teams\r\n#### Ask 2 committers to review/merge the PR and tag them here.\r\n<!--\r\nIf you don't know who to ask, you can request reviews in https://groups.google.com/forum/#!forum/ddf-developers .\r\n(please choose ONLY two committers from below, delete the rest)\r\n-->\r\n@bdeining \r\n@djblue \r\n@middlets719 \r\n@cantstoptheunk \r\n@Bdthomson \r\n@adimka \r\n@gordocanchola \r\n\r\n\r\n#### How should this be tested?\r\nEnsure no regressions have occurred for _sharing_ as a whole in search forms, workspaces.\r\n\r\nEnsure that the forms can't be modified once shared unless you're an owner or access individual. \r\n<!--(List steps with links to updated documentation)-->\r\n#### Any background context you want to provide?\r\n#### What are the relevant tickets?\r\n[](https://codice.atlassian.net/browse/)\r\n#### Screenshots\r\n<!--(if appropriate)-->\r\n#### Checklist:\r\n- [ ] Documentation Updated\r\n- [ ] Update / Add Unit Tests\r\n- [ ] Update / Add Integration Tests\r\n\r\n#### Notes on Review Process\r\nPlease see [Notes on Review Process](https://codice.atlassian.net/wiki/spaces/DDF/pages/71946981/Pull+Request+Guidelines) for further guidance on requirements for merging and abbreviated reviews. \r\n\r\n#### Review Comment Legend:\r\n- ✏️ (Pencil) This comment is a nitpick or style suggestion, no action required for approval. This comment should provide a suggestion either as an in line code snippet or a gist. \r\n- ❓ (Question Mark) This comment is to gain a clearer understanding of design or code choices, clarification is required but action may not be necessary for approval.\r\n- ❗ (Exclamation Mark) This comment is critical and requires clarification or action before approval.\r\n"},"comment":{"url":"https://api.github.com/repos/codice/ddf/issues/comments/428828560","html_url":"https://github.com/codice/ddf/pull/3818#issuecomment-428828560","issue_url":"https://api.github.com/repos/codice/ddf/issues/3818","id":428828560,"node_id":"MDEyOklzc3VlQ29tbWVudDQyODgyODU2MA==","user":{"login":"cxbot","id":21088360,"node_id":"MDQ6VXNlcjIxMDg4MzYw","avatar_url":"https://avatars2.githubusercontent.com/u/21088360?v=4","gravatar_id":"","url":"https://api.github.com/users/cxbot","html_url":"https://github.com/cxbot","followers_url":"https://api.github.com/users/cxbot/followers","following_url":"https://api.github.com/users/cxbot/following{/other_user}","gists_url":"https://api.github.com/users/cxbot/gists{/gist_id}","starred_url":"https://api.github.com/users/cxbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cxbot/subscriptions","organizations_url":"https://api.github.com/users/cxbot/orgs","repos_url":"https://api.github.com/users/cxbot/repos","events_url":"https://api.github.com/users/cxbot/events{/privacy}","received_events_url":"https://api.github.com/users/cxbot/received_events","type":"User","site_admin":false},"created_at":"2018-10-11T05:47:24Z","updated_at":"2018-10-11T05:47:24Z","author_association":"COLLABORATOR","body":"\nRefer to this link for build results (access rights to CI server needed): \nhttps://jenkins.codice.org/job/DDF-Jobs/job/pr/job/Linux/5484/<h2>Failed Tests: <span class='status-failure'>2</span></h2><h3><a name='DDF-Jobs/pr/Linux/ddf.test.itests:test-itests-ddf' /><a href='https://jenkins.codice.org/job/DDF-Jobs/job/pr/job/Linux/5484/ddf.test.itests$test-itests-ddf/testReport'>DDF-Jobs/pr/Linux/ddf.test.itests:test-itests-ddf</a>: <span class='status-failure'>2</span></h3><ul><li><a href='https://jenkins.codice.org/job/DDF-Jobs/job/pr/job/Linux/5484/ddf.test.itests$test-itests-ddf/testReport/ddf.test.itests.platform/TestSecurity/testGuestRestAccess/'><strong>ddf.test.itests.platform.TestSecurity.testGuestRestAccess</strong></a></li><li><a href='https://jenkins.codice.org/job/DDF-Jobs/job/pr/job/Linux/5484/ddf.test.itests$test-itests-ddf/testReport/ddf.test.itests.platform/TestSecurity/testBadHokSamlAssertionInHeaders/'><strong>ddf.test.itests.platform.TestSecurity.testBadHokSamlAssertionInHeaders</strong></a></li></ul>\n:x: JOB FAILURE\n"}}
{"actor":{"display_login":"cxbot"}}