{"action":"created","comment":{"url":"https://api.github.com/repos/orgzly/orgzly-android/pulls/comments/137393981","pull_request_review_id":61055159,"id":137393981,"diff_hunk":"@@ -412,7 +483,29 @@ private PendingIntent markNoteAsDonePendingIntent(\n \n         return PendingIntent.getService(\n                 context,\n-                Long.valueOf(noteId).intValue(),\n+                Long.valueOf(2 * noteId).intValue(),\n+                intent,\n+                PendingIntent.FLAG_UPDATE_CURRENT);\n+    }\n+\n+    private PendingIntent reminderSnoozePendingIntent(Context context,\n+                                                      long noteId,\n+                                                      long timestamp,\n+                                                      String notificationTag,\n+                                                      int notificationId) {\n+        if (BuildConfig.LOG_DEBUG) LogUtils.d(TAG, noteId, timestamp);\n+        Intent intent = new Intent(context, ActionService.class);\n+\n+        intent.setAction(AppIntent.ACTION_REMINDER_SNOOZE_REQUEST);\n+\n+        intent.putExtra(ActionService.EXTRA_NOTE_ID, noteId);\n+        intent.putExtra(ActionService.EXTRA_SNOOZE_TIMESTAMP, timestamp);\n+        intent.putExtra(ActionService.EXTRA_NOTIFICATION_TAG, notificationTag);\n+        intent.putExtra(ActionService.EXTRA_NOTIFICATION_ID, notificationId);\n+\n+        return PendingIntent.getService(\n+                context,\n+                Long.valueOf(2 * noteId + 1).intValue(),","path":"app/src/main/java/com/orgzly/android/reminders/ReminderService.java","position":162,"original_position":133,"commit_id":"815c0a823d7b747ca6e452754907e7df1a0c3414","original_commit_id":"108ef8f80092335ba07567ab91d8e99724d69c6b","user":{"login":"nevenz","id":10671476,"avatar_url":"https://avatars0.githubusercontent.com/u/10671476?v=4","gravatar_id":"","url":"https://api.github.com/users/nevenz","html_url":"https://github.com/nevenz","followers_url":"https://api.github.com/users/nevenz/followers","following_url":"https://api.github.com/users/nevenz/following{/other_user}","gists_url":"https://api.github.com/users/nevenz/gists{/gist_id}","starred_url":"https://api.github.com/users/nevenz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nevenz/subscriptions","organizations_url":"https://api.github.com/users/nevenz/orgs","repos_url":"https://api.github.com/users/nevenz/repos","events_url":"https://api.github.com/users/nevenz/events{/privacy}","received_events_url":"https://api.github.com/users/nevenz/received_events","type":"User","site_admin":false},"body":"Did you have issues with this being just `noteId` and leaving the same above in `markNoteAsDonePendingIntent`?\r\n\r\nThis shouldn't need to be unique if intent is different (even by having only a different action).","created_at":"2017-09-06T21:23:36Z","updated_at":"2017-09-06T21:23:36Z","html_url":"https://github.com/orgzly/orgzly-android/pull/160#discussion_r137393981","pull_request_url":"https://api.github.com/repos/orgzly/orgzly-android/pulls/160","author_association":"OWNER","_links":{"self":{"href":"https://api.github.com/repos/orgzly/orgzly-android/pulls/comments/137393981"},"html":{"href":"https://github.com/orgzly/orgzly-android/pull/160#discussion_r137393981"},"pull_request":{"href":"https://api.github.com/repos/orgzly/orgzly-android/pulls/160"}}},"pull_request":{"url":"https://api.github.com/repos/orgzly/orgzly-android/pulls/160","id":137855119,"html_url":"https://github.com/orgzly/orgzly-android/pull/160","diff_url":"https://github.com/orgzly/orgzly-android/pull/160.diff","patch_url":"https://github.com/orgzly/orgzly-android/pull/160.patch","issue_url":"https://api.github.com/repos/orgzly/orgzly-android/issues/160","number":160,"state":"open","locked":false,"title":"Feat/snooze","user":{"login":"tulth","id":11246388,"avatar_url":"https://avatars0.githubusercontent.com/u/11246388?v=4","gravatar_id":"","url":"https://api.github.com/users/tulth","html_url":"https://github.com/tulth","followers_url":"https://api.github.com/users/tulth/followers","following_url":"https://api.github.com/users/tulth/following{/other_user}","gists_url":"https://api.github.com/users/tulth/gists{/gist_id}","starred_url":"https://api.github.com/users/tulth/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tulth/subscriptions","organizations_url":"https://api.github.com/users/tulth/orgs","repos_url":"https://api.github.com/users/tulth/repos","events_url":"https://api.github.com/users/tulth/events{/privacy}","received_events_url":"https://api.github.com/users/tulth/received_events","type":"User","site_admin":false},"body":"Features:\r\n* Notification action support for android wearables.  Tested to work with pebble fw 4.3.\r\n* snooze feature for reminder pop ups.\r\n* setting for snooze time in preferences, defaults to 10 minutes.\r\n\r\nLooking for feedback.\r\nThe lifecycle for the snooze was much more complex than I expected. See diagram:\r\n\r\n![snooze_diagram](https://cdn.rawgit.com/tulth/orgzly-android/feat/snooze/doc/snooze_diagram.svg)\r\n\r\n\r\nIf it can be simpler, let me know.\r\n\r\n","created_at":"2017-08-28T00:36:51Z","updated_at":"2017-09-06T21:23:36Z","closed_at":null,"merged_at":null,"merge_commit_sha":"6a0e4ee92fd6104713f76ab4722f0b943681fc66","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/orgzly/orgzly-android/pulls/160/commits","review_comments_url":"https://api.github.com/repos/orgzly/orgzly-android/pulls/160/comments","review_comment_url":"https://api.github.com/repos/orgzly/orgzly-android/pulls/comments{/number}","comments_url":"https://api.github.com/repos/orgzly/orgzly-android/issues/160/comments","statuses_url":"https://api.github.com/repos/orgzly/orgzly-android/statuses/815c0a823d7b747ca6e452754907e7df1a0c3414","head":{"label":"tulth:feat/snooze","ref":"feat/snooze","sha":"815c0a823d7b747ca6e452754907e7df1a0c3414","user":{"login":"tulth","id":11246388,"avatar_url":"https://avatars0.githubusercontent.com/u/11246388?v=4","gravatar_id":"","url":"https://api.github.com/users/tulth","html_url":"https://github.com/tulth","followers_url":"https://api.github.com/users/tulth/followers","following_url":"https://api.github.com/users/tulth/following{/other_user}","gists_url":"https://api.github.com/users/tulth/gists{/gist_id}","starred_url":"https://api.github.com/users/tulth/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tulth/subscriptions","organizations_url":"https://api.github.com/users/tulth/orgs","repos_url":"https://api.github.com/users/tulth/repos","events_url":"https://api.github.com/users/tulth/events{/privacy}","received_events_url":"https://api.github.com/users/tulth/received_events","type":"User","site_admin":false},"repo":{"id":98828166,"name":"orgzly-android","full_name":"tulth/orgzly-android","owner":{"login":"tulth","id":11246388,"avatar_url":"https://avatars0.githubusercontent.com/u/11246388?v=4","gravatar_id":"","url":"https://api.github.com/users/tulth","html_url":"https://github.com/tulth","followers_url":"https://api.github.com/users/tulth/followers","following_url":"https://api.github.com/users/tulth/following{/other_user}","gists_url":"https://api.github.com/users/tulth/gists{/gist_id}","starred_url":"https://api.github.com/users/tulth/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tulth/subscriptions","organizations_url":"https://api.github.com/users/tulth/orgs","repos_url":"https://api.github.com/users/tulth/repos","events_url":"https://api.github.com/users/tulth/events{/privacy}","received_events_url":"https://api.github.com/users/tulth/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/tulth/orgzly-android","description":"Outliner for taking notes and managing to-do lists","fork":true,"url":"https://api.github.com/repos/tulth/orgzly-android","forks_url":"https://api.github.com/repos/tulth/orgzly-android/forks","keys_url":"https://api.github.com/repos/tulth/orgzly-android/keys{/key_id}","collaborators_url":"https://api.github.com/repos/tulth/orgzly-android/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/tulth/orgzly-android/teams","hooks_url":"https://api.github.com/repos/tulth/orgzly-android/hooks","issue_events_url":"https://api.github.com/repos/tulth/orgzly-android/issues/events{/number}","events_url":"https://api.github.com/repos/tulth/orgzly-android/events","assignees_url":"https://api.github.com/repos/tulth/orgzly-android/assignees{/user}","branches_url":"https://api.github.com/repos/tulth/orgzly-android/branches{/branch}","tags_url":"https://api.github.com/repos/tulth/orgzly-android/tags","blobs_url":"https://api.github.com/repos/tulth/orgzly-android/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/tulth/orgzly-android/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/tulth/orgzly-android/git/refs{/sha}","trees_url":"https://api.github.com/repos/tulth/orgzly-android/git/trees{/sha}","statuses_url":"https://api.github.com/repos/tulth/orgzly-android/statuses/{sha}","languages_url":"https://api.github.com/repos/tulth/orgzly-android/languages","stargazers_url":"https://api.github.com/repos/tulth/orgzly-android/stargazers","contributors_url":"https://api.github.com/repos/tulth/orgzly-android/contributors","subscribers_url":"https://api.github.com/repos/tulth/orgzly-android/subscribers","subscription_url":"https://api.github.com/repos/tulth/orgzly-android/subscription","commits_url":"https://api.github.com/repos/tulth/orgzly-android/commits{/sha}","git_commits_url":"https://api.github.com/repos/tulth/orgzly-android/git/commits{/sha}","comments_url":"https://api.github.com/repos/tulth/orgzly-android/comments{/number}","issue_comment_url":"https://api.github.com/repos/tulth/orgzly-android/issues/comments{/number}","contents_url":"https://api.github.com/repos/tulth/orgzly-android/contents/{+path}","compare_url":"https://api.github.com/repos/tulth/orgzly-android/compare/{base}...{head}","merges_url":"https://api.github.com/repos/tulth/orgzly-android/merges","archive_url":"https://api.github.com/repos/tulth/orgzly-android/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/tulth/orgzly-android/downloads","issues_url":"https://api.github.com/repos/tulth/orgzly-android/issues{/number}","pulls_url":"https://api.github.com/repos/tulth/orgzly-android/pulls{/number}","milestones_url":"https://api.github.com/repos/tulth/orgzly-android/milestones{/number}","notifications_url":"https://api.github.com/repos/tulth/orgzly-android/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/tulth/orgzly-android/labels{/name}","releases_url":"https://api.github.com/repos/tulth/orgzly-android/releases{/id}","deployments_url":"https://api.github.com/repos/tulth/orgzly-android/deployments","created_at":"2017-07-30T22:23:32Z","updated_at":"2017-07-30T22:23:34Z","pushed_at":"2017-09-06T18:29:58Z","git_url":"git://github.com/tulth/orgzly-android.git","ssh_url":"git@github.com:tulth/orgzly-android.git","clone_url":"https://github.com/tulth/orgzly-android.git","svn_url":"https://github.com/tulth/orgzly-android","homepage":"http://www.orgzly.com","size":26671,"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,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"orgzly:master","ref":"master","sha":"5f06c2831f4c6fd80abcb0a9f63f8624109f06de","user":{"login":"orgzly","id":10672741,"avatar_url":"https://avatars0.githubusercontent.com/u/10672741?v=4","gravatar_id":"","url":"https://api.github.com/users/orgzly","html_url":"https://github.com/orgzly","followers_url":"https://api.github.com/users/orgzly/followers","following_url":"https://api.github.com/users/orgzly/following{/other_user}","gists_url":"https://api.github.com/users/orgzly/gists{/gist_id}","starred_url":"https://api.github.com/users/orgzly/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/orgzly/subscriptions","organizations_url":"https://api.github.com/users/orgzly/orgs","repos_url":"https://api.github.com/users/orgzly/repos","events_url":"https://api.github.com/users/orgzly/events{/privacy}","received_events_url":"https://api.github.com/users/orgzly/received_events","type":"Organization","site_admin":false},"repo":{"id":80837945,"name":"orgzly-android","full_name":"orgzly/orgzly-android","owner":{"login":"orgzly","id":10672741,"avatar_url":"https://avatars0.githubusercontent.com/u/10672741?v=4","gravatar_id":"","url":"https://api.github.com/users/orgzly","html_url":"https://github.com/orgzly","followers_url":"https://api.github.com/users/orgzly/followers","following_url":"https://api.github.com/users/orgzly/following{/other_user}","gists_url":"https://api.github.com/users/orgzly/gists{/gist_id}","starred_url":"https://api.github.com/users/orgzly/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/orgzly/subscriptions","organizations_url":"https://api.github.com/users/orgzly/orgs","repos_url":"https://api.github.com/users/orgzly/repos","events_url":"https://api.github.com/users/orgzly/events{/privacy}","received_events_url":"https://api.github.com/users/orgzly/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/orgzly/orgzly-android","description":"Outliner for taking notes and managing to-do lists","fork":false,"url":"https://api.github.com/repos/orgzly/orgzly-android","forks_url":"https://api.github.com/repos/orgzly/orgzly-android/forks","keys_url":"https://api.github.com/repos/orgzly/orgzly-android/keys{/key_id}","collaborators_url":"https://api.github.com/repos/orgzly/orgzly-android/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/orgzly/orgzly-android/teams","hooks_url":"https://api.github.com/repos/orgzly/orgzly-android/hooks","issue_events_url":"https://api.github.com/repos/orgzly/orgzly-android/issues/events{/number}","events_url":"https://api.github.com/repos/orgzly/orgzly-android/events","assignees_url":"https://api.github.com/repos/orgzly/orgzly-android/assignees{/user}","branches_url":"https://api.github.com/repos/orgzly/orgzly-android/branches{/branch}","tags_url":"https://api.github.com/repos/orgzly/orgzly-android/tags","blobs_url":"https://api.github.com/repos/orgzly/orgzly-android/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/orgzly/orgzly-android/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/orgzly/orgzly-android/git/refs{/sha}","trees_url":"https://api.github.com/repos/orgzly/orgzly-android/git/trees{/sha}","statuses_url":"https://api.github.com/repos/orgzly/orgzly-android/statuses/{sha}","languages_url":"https://api.github.com/repos/orgzly/orgzly-android/languages","stargazers_url":"https://api.github.com/repos/orgzly/orgzly-android/stargazers","contributors_url":"https://api.github.com/repos/orgzly/orgzly-android/contributors","subscribers_url":"https://api.github.com/repos/orgzly/orgzly-android/subscribers","subscription_url":"https://api.github.com/repos/orgzly/orgzly-android/subscription","commits_url":"https://api.github.com/repos/orgzly/orgzly-android/commits{/sha}","git_commits_url":"https://api.github.com/repos/orgzly/orgzly-android/git/commits{/sha}","comments_url":"https://api.github.com/repos/orgzly/orgzly-android/comments{/number}","issue_comment_url":"https://api.github.com/repos/orgzly/orgzly-android/issues/comments{/number}","contents_url":"https://api.github.com/repos/orgzly/orgzly-android/contents/{+path}","compare_url":"https://api.github.com/repos/orgzly/orgzly-android/compare/{base}...{head}","merges_url":"https://api.github.com/repos/orgzly/orgzly-android/merges","archive_url":"https://api.github.com/repos/orgzly/orgzly-android/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/orgzly/orgzly-android/downloads","issues_url":"https://api.github.com/repos/orgzly/orgzly-android/issues{/number}","pulls_url":"https://api.github.com/repos/orgzly/orgzly-android/pulls{/number}","milestones_url":"https://api.github.com/repos/orgzly/orgzly-android/milestones{/number}","notifications_url":"https://api.github.com/repos/orgzly/orgzly-android/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/orgzly/orgzly-android/labels{/name}","releases_url":"https://api.github.com/repos/orgzly/orgzly-android/releases{/id}","deployments_url":"https://api.github.com/repos/orgzly/orgzly-android/deployments","created_at":"2017-02-03T14:48:34Z","updated_at":"2017-09-06T06:58:43Z","pushed_at":"2017-09-06T21:01:36Z","git_url":"git://github.com/orgzly/orgzly-android.git","ssh_url":"git@github.com:orgzly/orgzly-android.git","clone_url":"https://github.com/orgzly/orgzly-android.git","svn_url":"https://github.com/orgzly/orgzly-android","homepage":"http://www.orgzly.com","size":26057,"stargazers_count":474,"watchers_count":474,"language":"Java","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":59,"mirror_url":null,"open_issues_count":89,"forks":59,"open_issues":89,"watchers":474,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/orgzly/orgzly-android/pulls/160"},"html":{"href":"https://github.com/orgzly/orgzly-android/pull/160"},"issue":{"href":"https://api.github.com/repos/orgzly/orgzly-android/issues/160"},"comments":{"href":"https://api.github.com/repos/orgzly/orgzly-android/issues/160/comments"},"review_comments":{"href":"https://api.github.com/repos/orgzly/orgzly-android/pulls/160/comments"},"review_comment":{"href":"https://api.github.com/repos/orgzly/orgzly-android/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/orgzly/orgzly-android/pulls/160/commits"},"statuses":{"href":"https://api.github.com/repos/orgzly/orgzly-android/statuses/815c0a823d7b747ca6e452754907e7df1a0c3414"}},"author_association":"NONE"}}
{"actor":{"display_login":"nevenz"}}