{"action":"created","comment":{"url":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/comments/188783579","pull_request_review_id":120833360,"id":188783579,"diff_hunk":"@@ -88,13 +88,20 @@ Add the application key provided to you by Placed in the application tag of your\n <meta-data android:name=\"placed_app_key\" android:value=\"YOUR_APP_KEY\" />\n ```\n \n-#### Prompting for location permission\n+#### Opt-in Dialogue and Prompting for Location Permission\n \n-If you don't already prompt for fine location permission, you should do so before registering the user with the Placed SDK.\n+As outlined in Section 5 of the [Placed Affiliate Agreement](https://affiliate.placed.com/placed-affiliate-agreement/), you must satisfy two requirements prior to registering a user with the Placed SDK:  \n+1. *Gather Express Consent for User Data Collection via Opt-in Dialog*  \n+In addition having a legally compliant privacy policy describing Placed’s collection of location and device information, you must include a discrete opt-in dialog which gathers express consent for data collection. This dialog appears prior to the fine location permission prompt, and includes:\n+    - The language: “*Aggregated device data, including location and apps, is measured for the purposes of market research by Placed, Inc.*”\n+    - Links to the Placed [Terms of Service](https://www.placed.com/terms-of-service) and [Privacy Policy](https://www.placed.com/privacy-policy)\n+    - Buttons to “Accept” or “Cancel”\n \n-For an example, see the [main activity in the sample app](https://github.com/placed/android-placed-sdk/blob/master/SampleApp/app/src/main/java/com/placed/android/sampleapp/MainActivity.java).\n \n-Notice in this case the fine location permission is requested before registering the user and acceptance triggers app registion as described below.\n+1. *Prompt for Fine Location Permission*  \n+Eligible users must allow fine location permission, which subsequently triggers app registration as described in the [Registering a User](https://github.com/placed/android-placed-sdk#registering-a-user) section.\n+\n+For an example of the opt-in dialog and location permission prompt, please refer to the [main activity of the sample app](https://github.com/placed/android-placed-sdk/blob/master/SampleApp/app/src/main/java/com/placed/android/sampleapp/MainActivity.java). We have also provided a [gallery for inspiration](./gallery) on how you can better integrate the opt-in experience into your app.","path":"README.md","position":30,"original_position":30,"commit_id":"961989ddb1f18486f922cf76dc4cd7a308bc9b4a","original_commit_id":"961989ddb1f18486f922cf76dc4cd7a308bc9b4a","user":{"login":"dgranahan","id":8977153,"avatar_url":"https://avatars3.githubusercontent.com/u/8977153?v=4","gravatar_id":"","url":"https://api.github.com/users/dgranahan","html_url":"https://github.com/dgranahan","followers_url":"https://api.github.com/users/dgranahan/followers","following_url":"https://api.github.com/users/dgranahan/following{/other_user}","gists_url":"https://api.github.com/users/dgranahan/gists{/gist_id}","starred_url":"https://api.github.com/users/dgranahan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dgranahan/subscriptions","organizations_url":"https://api.github.com/users/dgranahan/orgs","repos_url":"https://api.github.com/users/dgranahan/repos","events_url":"https://api.github.com/users/dgranahan/events{/privacy}","received_events_url":"https://api.github.com/users/dgranahan/received_events","type":"User","site_admin":false},"body":"Minor: it might be nice to make the 'main activity' link be relative.","created_at":"2018-05-16T21:50:34Z","updated_at":"2018-05-16T21:50:34Z","html_url":"https://github.com/sewichi/android-placed-sdk/pull/13#discussion_r188783579","pull_request_url":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/13","author_association":"NONE","_links":{"self":{"href":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/comments/188783579"},"html":{"href":"https://github.com/sewichi/android-placed-sdk/pull/13#discussion_r188783579"},"pull_request":{"href":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/13"}}},"pull_request":{"url":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/13","id":187570221,"html_url":"https://github.com/sewichi/android-placed-sdk/pull/13","diff_url":"https://github.com/sewichi/android-placed-sdk/pull/13.diff","patch_url":"https://github.com/sewichi/android-placed-sdk/pull/13.patch","issue_url":"https://api.github.com/repos/sewichi/android-placed-sdk/issues/13","number":13,"state":"open","locked":false,"title":"[MOBILE-2292] Ensure Sample App includes Opt-In dialog, update documentation","user":{"login":"sstasiuk","id":36459897,"avatar_url":"https://avatars2.githubusercontent.com/u/36459897?v=4","gravatar_id":"","url":"https://api.github.com/users/sstasiuk","html_url":"https://github.com/sstasiuk","followers_url":"https://api.github.com/users/sstasiuk/followers","following_url":"https://api.github.com/users/sstasiuk/following{/other_user}","gists_url":"https://api.github.com/users/sstasiuk/gists{/gist_id}","starred_url":"https://api.github.com/users/sstasiuk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sstasiuk/subscriptions","organizations_url":"https://api.github.com/users/sstasiuk/orgs","repos_url":"https://api.github.com/users/sstasiuk/repos","events_url":"https://api.github.com/users/sstasiuk/events{/privacy}","received_events_url":"https://api.github.com/users/sstasiuk/received_events","type":"User","site_admin":false},"body":"* Documentation updated\r\n* Sample alert images added\r\n* Added sample alert to project\r\n\r\nLink to branch with updated readme: https://github.com/sewichi/android-placed-sdk/tree/sample-app-includes-opt-in-dialog\r\n\r\n<img width=\"589\" alt=\"screen shot 2018-05-11 at 3 13 10 pm\" src=\"https://user-images.githubusercontent.com/36459897/39949277-c1d2bb06-552e-11e8-9b28-6e68708b1c74.png\">\r\n","created_at":"2018-05-11T21:07:55Z","updated_at":"2018-05-16T21:50:34Z","closed_at":null,"merged_at":null,"merge_commit_sha":"1b136337205a3b33882123c19ca7327c38324e81","assignee":{"login":"dgranahan","id":8977153,"avatar_url":"https://avatars3.githubusercontent.com/u/8977153?v=4","gravatar_id":"","url":"https://api.github.com/users/dgranahan","html_url":"https://github.com/dgranahan","followers_url":"https://api.github.com/users/dgranahan/followers","following_url":"https://api.github.com/users/dgranahan/following{/other_user}","gists_url":"https://api.github.com/users/dgranahan/gists{/gist_id}","starred_url":"https://api.github.com/users/dgranahan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dgranahan/subscriptions","organizations_url":"https://api.github.com/users/dgranahan/orgs","repos_url":"https://api.github.com/users/dgranahan/repos","events_url":"https://api.github.com/users/dgranahan/events{/privacy}","received_events_url":"https://api.github.com/users/dgranahan/received_events","type":"User","site_admin":false},"assignees":[{"login":"dgranahan","id":8977153,"avatar_url":"https://avatars3.githubusercontent.com/u/8977153?v=4","gravatar_id":"","url":"https://api.github.com/users/dgranahan","html_url":"https://github.com/dgranahan","followers_url":"https://api.github.com/users/dgranahan/followers","following_url":"https://api.github.com/users/dgranahan/following{/other_user}","gists_url":"https://api.github.com/users/dgranahan/gists{/gist_id}","starred_url":"https://api.github.com/users/dgranahan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dgranahan/subscriptions","organizations_url":"https://api.github.com/users/dgranahan/orgs","repos_url":"https://api.github.com/users/dgranahan/repos","events_url":"https://api.github.com/users/dgranahan/events{/privacy}","received_events_url":"https://api.github.com/users/dgranahan/received_events","type":"User","site_admin":false}],"requested_reviewers":[{"login":"nickneiman","id":5332546,"avatar_url":"https://avatars3.githubusercontent.com/u/5332546?v=4","gravatar_id":"","url":"https://api.github.com/users/nickneiman","html_url":"https://github.com/nickneiman","followers_url":"https://api.github.com/users/nickneiman/followers","following_url":"https://api.github.com/users/nickneiman/following{/other_user}","gists_url":"https://api.github.com/users/nickneiman/gists{/gist_id}","starred_url":"https://api.github.com/users/nickneiman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nickneiman/subscriptions","organizations_url":"https://api.github.com/users/nickneiman/orgs","repos_url":"https://api.github.com/users/nickneiman/repos","events_url":"https://api.github.com/users/nickneiman/events{/privacy}","received_events_url":"https://api.github.com/users/nickneiman/received_events","type":"User","site_admin":false}],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/13/commits","review_comments_url":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/13/comments","review_comment_url":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/comments{/number}","comments_url":"https://api.github.com/repos/sewichi/android-placed-sdk/issues/13/comments","statuses_url":"https://api.github.com/repos/sewichi/android-placed-sdk/statuses/961989ddb1f18486f922cf76dc4cd7a308bc9b4a","head":{"label":"sewichi:sample-app-includes-opt-in-dialog","ref":"sample-app-includes-opt-in-dialog","sha":"961989ddb1f18486f922cf76dc4cd7a308bc9b4a","user":{"login":"sewichi","id":924111,"avatar_url":"https://avatars0.githubusercontent.com/u/924111?v=4","gravatar_id":"","url":"https://api.github.com/users/sewichi","html_url":"https://github.com/sewichi","followers_url":"https://api.github.com/users/sewichi/followers","following_url":"https://api.github.com/users/sewichi/following{/other_user}","gists_url":"https://api.github.com/users/sewichi/gists{/gist_id}","starred_url":"https://api.github.com/users/sewichi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sewichi/subscriptions","organizations_url":"https://api.github.com/users/sewichi/orgs","repos_url":"https://api.github.com/users/sewichi/repos","events_url":"https://api.github.com/users/sewichi/events{/privacy}","received_events_url":"https://api.github.com/users/sewichi/received_events","type":"Organization","site_admin":false},"repo":{"id":66582005,"name":"android-placed-sdk","full_name":"sewichi/android-placed-sdk","owner":{"login":"sewichi","id":924111,"avatar_url":"https://avatars0.githubusercontent.com/u/924111?v=4","gravatar_id":"","url":"https://api.github.com/users/sewichi","html_url":"https://github.com/sewichi","followers_url":"https://api.github.com/users/sewichi/followers","following_url":"https://api.github.com/users/sewichi/following{/other_user}","gists_url":"https://api.github.com/users/sewichi/gists{/gist_id}","starred_url":"https://api.github.com/users/sewichi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sewichi/subscriptions","organizations_url":"https://api.github.com/users/sewichi/orgs","repos_url":"https://api.github.com/users/sewichi/repos","events_url":"https://api.github.com/users/sewichi/events{/privacy}","received_events_url":"https://api.github.com/users/sewichi/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/sewichi/android-placed-sdk","description":null,"fork":true,"url":"https://api.github.com/repos/sewichi/android-placed-sdk","forks_url":"https://api.github.com/repos/sewichi/android-placed-sdk/forks","keys_url":"https://api.github.com/repos/sewichi/android-placed-sdk/keys{/key_id}","collaborators_url":"https://api.github.com/repos/sewichi/android-placed-sdk/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/sewichi/android-placed-sdk/teams","hooks_url":"https://api.github.com/repos/sewichi/android-placed-sdk/hooks","issue_events_url":"https://api.github.com/repos/sewichi/android-placed-sdk/issues/events{/number}","events_url":"https://api.github.com/repos/sewichi/android-placed-sdk/events","assignees_url":"https://api.github.com/repos/sewichi/android-placed-sdk/assignees{/user}","branches_url":"https://api.github.com/repos/sewichi/android-placed-sdk/branches{/branch}","tags_url":"https://api.github.com/repos/sewichi/android-placed-sdk/tags","blobs_url":"https://api.github.com/repos/sewichi/android-placed-sdk/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/sewichi/android-placed-sdk/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/sewichi/android-placed-sdk/git/refs{/sha}","trees_url":"https://api.github.com/repos/sewichi/android-placed-sdk/git/trees{/sha}","statuses_url":"https://api.github.com/repos/sewichi/android-placed-sdk/statuses/{sha}","languages_url":"https://api.github.com/repos/sewichi/android-placed-sdk/languages","stargazers_url":"https://api.github.com/repos/sewichi/android-placed-sdk/stargazers","contributors_url":"https://api.github.com/repos/sewichi/android-placed-sdk/contributors","subscribers_url":"https://api.github.com/repos/sewichi/android-placed-sdk/subscribers","subscription_url":"https://api.github.com/repos/sewichi/android-placed-sdk/subscription","commits_url":"https://api.github.com/repos/sewichi/android-placed-sdk/commits{/sha}","git_commits_url":"https://api.github.com/repos/sewichi/android-placed-sdk/git/commits{/sha}","comments_url":"https://api.github.com/repos/sewichi/android-placed-sdk/comments{/number}","issue_comment_url":"https://api.github.com/repos/sewichi/android-placed-sdk/issues/comments{/number}","contents_url":"https://api.github.com/repos/sewichi/android-placed-sdk/contents/{+path}","compare_url":"https://api.github.com/repos/sewichi/android-placed-sdk/compare/{base}...{head}","merges_url":"https://api.github.com/repos/sewichi/android-placed-sdk/merges","archive_url":"https://api.github.com/repos/sewichi/android-placed-sdk/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/sewichi/android-placed-sdk/downloads","issues_url":"https://api.github.com/repos/sewichi/android-placed-sdk/issues{/number}","pulls_url":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls{/number}","milestones_url":"https://api.github.com/repos/sewichi/android-placed-sdk/milestones{/number}","notifications_url":"https://api.github.com/repos/sewichi/android-placed-sdk/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/sewichi/android-placed-sdk/labels{/name}","releases_url":"https://api.github.com/repos/sewichi/android-placed-sdk/releases{/id}","deployments_url":"https://api.github.com/repos/sewichi/android-placed-sdk/deployments","created_at":"2016-08-25T18:13:08Z","updated_at":"2016-08-30T00:00:45Z","pushed_at":"2018-05-15T22:47:13Z","git_url":"git://github.com/sewichi/android-placed-sdk.git","ssh_url":"git@github.com:sewichi/android-placed-sdk.git","clone_url":"https://github.com/sewichi/android-placed-sdk.git","svn_url":"https://github.com/sewichi/android-placed-sdk","homepage":null,"size":6606,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":1,"license":null,"forks":0,"open_issues":1,"watchers":0,"default_branch":"master"}},"base":{"label":"sewichi:master","ref":"master","sha":"505357fc9a68e5495e6593d289eb60838dd5620f","user":{"login":"sewichi","id":924111,"avatar_url":"https://avatars0.githubusercontent.com/u/924111?v=4","gravatar_id":"","url":"https://api.github.com/users/sewichi","html_url":"https://github.com/sewichi","followers_url":"https://api.github.com/users/sewichi/followers","following_url":"https://api.github.com/users/sewichi/following{/other_user}","gists_url":"https://api.github.com/users/sewichi/gists{/gist_id}","starred_url":"https://api.github.com/users/sewichi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sewichi/subscriptions","organizations_url":"https://api.github.com/users/sewichi/orgs","repos_url":"https://api.github.com/users/sewichi/repos","events_url":"https://api.github.com/users/sewichi/events{/privacy}","received_events_url":"https://api.github.com/users/sewichi/received_events","type":"Organization","site_admin":false},"repo":{"id":66582005,"name":"android-placed-sdk","full_name":"sewichi/android-placed-sdk","owner":{"login":"sewichi","id":924111,"avatar_url":"https://avatars0.githubusercontent.com/u/924111?v=4","gravatar_id":"","url":"https://api.github.com/users/sewichi","html_url":"https://github.com/sewichi","followers_url":"https://api.github.com/users/sewichi/followers","following_url":"https://api.github.com/users/sewichi/following{/other_user}","gists_url":"https://api.github.com/users/sewichi/gists{/gist_id}","starred_url":"https://api.github.com/users/sewichi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sewichi/subscriptions","organizations_url":"https://api.github.com/users/sewichi/orgs","repos_url":"https://api.github.com/users/sewichi/repos","events_url":"https://api.github.com/users/sewichi/events{/privacy}","received_events_url":"https://api.github.com/users/sewichi/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/sewichi/android-placed-sdk","description":null,"fork":true,"url":"https://api.github.com/repos/sewichi/android-placed-sdk","forks_url":"https://api.github.com/repos/sewichi/android-placed-sdk/forks","keys_url":"https://api.github.com/repos/sewichi/android-placed-sdk/keys{/key_id}","collaborators_url":"https://api.github.com/repos/sewichi/android-placed-sdk/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/sewichi/android-placed-sdk/teams","hooks_url":"https://api.github.com/repos/sewichi/android-placed-sdk/hooks","issue_events_url":"https://api.github.com/repos/sewichi/android-placed-sdk/issues/events{/number}","events_url":"https://api.github.com/repos/sewichi/android-placed-sdk/events","assignees_url":"https://api.github.com/repos/sewichi/android-placed-sdk/assignees{/user}","branches_url":"https://api.github.com/repos/sewichi/android-placed-sdk/branches{/branch}","tags_url":"https://api.github.com/repos/sewichi/android-placed-sdk/tags","blobs_url":"https://api.github.com/repos/sewichi/android-placed-sdk/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/sewichi/android-placed-sdk/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/sewichi/android-placed-sdk/git/refs{/sha}","trees_url":"https://api.github.com/repos/sewichi/android-placed-sdk/git/trees{/sha}","statuses_url":"https://api.github.com/repos/sewichi/android-placed-sdk/statuses/{sha}","languages_url":"https://api.github.com/repos/sewichi/android-placed-sdk/languages","stargazers_url":"https://api.github.com/repos/sewichi/android-placed-sdk/stargazers","contributors_url":"https://api.github.com/repos/sewichi/android-placed-sdk/contributors","subscribers_url":"https://api.github.com/repos/sewichi/android-placed-sdk/subscribers","subscription_url":"https://api.github.com/repos/sewichi/android-placed-sdk/subscription","commits_url":"https://api.github.com/repos/sewichi/android-placed-sdk/commits{/sha}","git_commits_url":"https://api.github.com/repos/sewichi/android-placed-sdk/git/commits{/sha}","comments_url":"https://api.github.com/repos/sewichi/android-placed-sdk/comments{/number}","issue_comment_url":"https://api.github.com/repos/sewichi/android-placed-sdk/issues/comments{/number}","contents_url":"https://api.github.com/repos/sewichi/android-placed-sdk/contents/{+path}","compare_url":"https://api.github.com/repos/sewichi/android-placed-sdk/compare/{base}...{head}","merges_url":"https://api.github.com/repos/sewichi/android-placed-sdk/merges","archive_url":"https://api.github.com/repos/sewichi/android-placed-sdk/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/sewichi/android-placed-sdk/downloads","issues_url":"https://api.github.com/repos/sewichi/android-placed-sdk/issues{/number}","pulls_url":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls{/number}","milestones_url":"https://api.github.com/repos/sewichi/android-placed-sdk/milestones{/number}","notifications_url":"https://api.github.com/repos/sewichi/android-placed-sdk/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/sewichi/android-placed-sdk/labels{/name}","releases_url":"https://api.github.com/repos/sewichi/android-placed-sdk/releases{/id}","deployments_url":"https://api.github.com/repos/sewichi/android-placed-sdk/deployments","created_at":"2016-08-25T18:13:08Z","updated_at":"2016-08-30T00:00:45Z","pushed_at":"2018-05-15T22:47:13Z","git_url":"git://github.com/sewichi/android-placed-sdk.git","ssh_url":"git@github.com:sewichi/android-placed-sdk.git","clone_url":"https://github.com/sewichi/android-placed-sdk.git","svn_url":"https://github.com/sewichi/android-placed-sdk","homepage":null,"size":6606,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":1,"license":null,"forks":0,"open_issues":1,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/13"},"html":{"href":"https://github.com/sewichi/android-placed-sdk/pull/13"},"issue":{"href":"https://api.github.com/repos/sewichi/android-placed-sdk/issues/13"},"comments":{"href":"https://api.github.com/repos/sewichi/android-placed-sdk/issues/13/comments"},"review_comments":{"href":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/13/comments"},"review_comment":{"href":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/sewichi/android-placed-sdk/pulls/13/commits"},"statuses":{"href":"https://api.github.com/repos/sewichi/android-placed-sdk/statuses/961989ddb1f18486f922cf76dc4cd7a308bc9b4a"}},"author_association":"NONE"}}
{"actor":{"display_login":"dgranahan"}}