payload: {"action":"created","comment":{"url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/comments/321915598","pull_request_review_id":285106880,"id":321915598,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDMyMTkxNTU5OA==","diff_hunk":"@@ -158,43 +161,40 @@ class EditAndSyncFeaturesViewController: UIViewController {\n         areaOfInterest = self.extentViewFrameToEnvelope()\n         \n         geodatabaseSyncTask.defaultGenerateGeodatabaseParameters(withExtent: areaOfInterest) { [weak self] (params: AGSGenerateGeodatabaseParameters?, error: Error?) in\n-            if let params = params,\n-                let self = self {\n-                // Don't include attachments to minimize the geodatabase size.\n-                params.returnAttachments = false\n-                \n-                // Create a temporary file for the geodatabase.\n-                let dateFormatter = ISO8601DateFormatter()\n-                \n-                let documentDirectoryURL = FileManager.default.urls(for: .documentDirectory, in: .userDomainMask).first!\n-                let downloadFileURL = documentDirectoryURL\n-                    .appendingPathComponent(dateFormatter.string(from: Date()))\n-                    .appendingPathExtension(\"geodatabase\")\n-               \n-                // Request a job to generate the geodatabase.\n-                let generateGeodatabaseJob = self.geodatabaseSyncTask.generateJob(with: params, downloadFileURL: downloadFileURL)\n-                self.generateJob = generateGeodatabaseJob\n-                generateGeodatabaseJob.start(\n-                    statusHandler: { (status: AGSJobStatus) in\n-                        SVProgressHUD.show(withStatus: status.statusString()) //Show job status.\n-                    },\n-                    completion: { [weak self] (_, error: Error?) in\n-                        SVProgressHUD.dismiss()\n-                        \n-                        if let error = error {\n-                            self?.presentAlert(error: error)\n-                        } else {\n-                            // Load the geodatabase when the job is done.\n-                            self?.geodatabase = generateGeodatabaseJob.result\n-                            self?.geodatabase.load { [weak self] (error: Error?) in\n-                                self?.geodatabaseDidLoad()\n-                            }\n+            guard let self = self else { return }\n+            \n+            guard let params = params else { return }","path":"arcgis-ios-sdk-samples/Edit data/Edit and sync features/EditAndSyncFeaturesViewController.swift","position":166,"original_position":70,"commit_id":"0ba4bd6dfd8f638951cceb44112aaa13b4d214c0","original_commit_id":"0ba4bd6dfd8f638951cceb44112aaa13b4d214c0","user":{"login":"sbaskaran","id":4185324,"node_id":"MDQ6VXNlcjQxODUzMjQ=","avatar_url":"https://avatars1.githubusercontent.com/u/4185324?v=4","gravatar_id":"","url":"https://api.github.com/users/sbaskaran","html_url":"https://github.com/sbaskaran","followers_url":"https://api.github.com/users/sbaskaran/followers","following_url":"https://api.github.com/users/sbaskaran/following{/other_user}","gists_url":"https://api.github.com/users/sbaskaran/gists{/gist_id}","starred_url":"https://api.github.com/users/sbaskaran/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sbaskaran/subscriptions","organizations_url":"https://api.github.com/users/sbaskaran/orgs","repos_url":"https://api.github.com/users/sbaskaran/repos","events_url":"https://api.github.com/users/sbaskaran/events{/privacy}","received_events_url":"https://api.github.com/users/sbaskaran/received_events","type":"User","site_admin":false},"body":"I think you'd want to present the alert here?\r\n","created_at":"2019-09-06T21:24:37Z","updated_at":"2019-09-06T21:24:37Z","html_url":"https://github.com/Esri/arcgis-runtime-samples-ios/pull/750#discussion_r321915598","pull_request_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/750","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/comments/321915598"},"html":{"href":"https://github.com/Esri/arcgis-runtime-samples-ios/pull/750#discussion_r321915598"},"pull_request":{"href":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/750"}}},"pull_request":{"url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/750","id":306617812,"node_id":"MDExOlB1bGxSZXF1ZXN0MzA2NjE3ODEy","html_url":"https://github.com/Esri/arcgis-runtime-samples-ios/pull/750","diff_url":"https://github.com/Esri/arcgis-runtime-samples-ios/pull/750.diff","patch_url":"https://github.com/Esri/arcgis-runtime-samples-ios/pull/750.patch","issue_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/issues/750","number":750,"state":"open","locked":false,"title":"Vivian edit sync v next","user":{"login":"vquach2404","id":52215269,"node_id":"MDQ6VXNlcjUyMjE1MjY5","avatar_url":"https://avatars2.githubusercontent.com/u/52215269?v=4","gravatar_id":"","url":"https://api.github.com/users/vquach2404","html_url":"https://github.com/vquach2404","followers_url":"https://api.github.com/users/vquach2404/followers","following_url":"https://api.github.com/users/vquach2404/following{/other_user}","gists_url":"https://api.github.com/users/vquach2404/gists{/gist_id}","starred_url":"https://api.github.com/users/vquach2404/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vquach2404/subscriptions","organizations_url":"https://api.github.com/users/vquach2404/orgs","repos_url":"https://api.github.com/users/vquach2404/repos","events_url":"https://api.github.com/users/vquach2404/events{/privacy}","received_events_url":"https://api.github.com/users/vquach2404/received_events","type":"User","site_admin":false},"body":"","created_at":"2019-08-12T18:54:17Z","updated_at":"2019-09-06T21:24:37Z","closed_at":null,"merged_at":null,"merge_commit_sha":"5e32c651e5245fd9e044211ee2cad704b2e66533","assignee":{"login":"vquach2404","id":52215269,"node_id":"MDQ6VXNlcjUyMjE1MjY5","avatar_url":"https://avatars2.githubusercontent.com/u/52215269?v=4","gravatar_id":"","url":"https://api.github.com/users/vquach2404","html_url":"https://github.com/vquach2404","followers_url":"https://api.github.com/users/vquach2404/followers","following_url":"https://api.github.com/users/vquach2404/following{/other_user}","gists_url":"https://api.github.com/users/vquach2404/gists{/gist_id}","starred_url":"https://api.github.com/users/vquach2404/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vquach2404/subscriptions","organizations_url":"https://api.github.com/users/vquach2404/orgs","repos_url":"https://api.github.com/users/vquach2404/repos","events_url":"https://api.github.com/users/vquach2404/events{/privacy}","received_events_url":"https://api.github.com/users/vquach2404/received_events","type":"User","site_admin":false},"assignees":[{"login":"vquach2404","id":52215269,"node_id":"MDQ6VXNlcjUyMjE1MjY5","avatar_url":"https://avatars2.githubusercontent.com/u/52215269?v=4","gravatar_id":"","url":"https://api.github.com/users/vquach2404","html_url":"https://github.com/vquach2404","followers_url":"https://api.github.com/users/vquach2404/followers","following_url":"https://api.github.com/users/vquach2404/following{/other_user}","gists_url":"https://api.github.com/users/vquach2404/gists{/gist_id}","starred_url":"https://api.github.com/users/vquach2404/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vquach2404/subscriptions","organizations_url":"https://api.github.com/users/vquach2404/orgs","repos_url":"https://api.github.com/users/vquach2404/repos","events_url":"https://api.github.com/users/vquach2404/events{/privacy}","received_events_url":"https://api.github.com/users/vquach2404/received_events","type":"User","site_admin":false}],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/750/commits","review_comments_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/750/comments","review_comment_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/issues/750/comments","statuses_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/statuses/0ba4bd6dfd8f638951cceb44112aaa13b4d214c0","head":{"label":"Esri:vivian_editSync_vNext","ref":"vivian_editSync_vNext","sha":"0ba4bd6dfd8f638951cceb44112aaa13b4d214c0","user":{"login":"Esri","id":628795,"node_id":"MDEyOk9yZ2FuaXphdGlvbjYyODc5NQ==","avatar_url":"https://avatars1.githubusercontent.com/u/628795?v=4","gravatar_id":"","url":"https://api.github.com/users/Esri","html_url":"https://github.com/Esri","followers_url":"https://api.github.com/users/Esri/followers","following_url":"https://api.github.com/users/Esri/following{/other_user}","gists_url":"https://api.github.com/users/Esri/gists{/gist_id}","starred_url":"https://api.github.com/users/Esri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Esri/subscriptions","organizations_url":"https://api.github.com/users/Esri/orgs","repos_url":"https://api.github.com/users/Esri/repos","events_url":"https://api.github.com/users/Esri/events{/privacy}","received_events_url":"https://api.github.com/users/Esri/received_events","type":"Organization","site_admin":false},"repo":{"id":7922900,"node_id":"MDEwOlJlcG9zaXRvcnk3OTIyOTAw","name":"arcgis-runtime-samples-ios","full_name":"Esri/arcgis-runtime-samples-ios","private":false,"owner":{"login":"Esri","id":628795,"node_id":"MDEyOk9yZ2FuaXphdGlvbjYyODc5NQ==","avatar_url":"https://avatars1.githubusercontent.com/u/628795?v=4","gravatar_id":"","url":"https://api.github.com/users/Esri","html_url":"https://github.com/Esri","followers_url":"https://api.github.com/users/Esri/followers","following_url":"https://api.github.com/users/Esri/following{/other_user}","gists_url":"https://api.github.com/users/Esri/gists{/gist_id}","starred_url":"https://api.github.com/users/Esri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Esri/subscriptions","organizations_url":"https://api.github.com/users/Esri/orgs","repos_url":"https://api.github.com/users/Esri/repos","events_url":"https://api.github.com/users/Esri/events{/privacy}","received_events_url":"https://api.github.com/users/Esri/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/Esri/arcgis-runtime-samples-ios","description":"Swift samples demonstrating various capabilities of ArcGIS Runtime SDK for iOS","fork":false,"url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios","forks_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/forks","keys_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/teams","hooks_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/hooks","issue_events_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/issues/events{/number}","events_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/events","assignees_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/assignees{/user}","branches_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/branches{/branch}","tags_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/tags","blobs_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/git/refs{/sha}","trees_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/statuses/{sha}","languages_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/languages","stargazers_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/stargazers","contributors_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/contributors","subscribers_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/subscribers","subscription_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/subscription","commits_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/commits{/sha}","git_commits_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/git/commits{/sha}","comments_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/comments{/number}","issue_comment_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/issues/comments{/number}","contents_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/contents/{+path}","compare_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/merges","archive_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/downloads","issues_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/issues{/number}","pulls_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls{/number}","milestones_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/milestones{/number}","notifications_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/labels{/name}","releases_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/releases{/id}","deployments_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/deployments","created_at":"2013-01-30T20:18:40Z","updated_at":"2019-09-04T20:21:58Z","pushed_at":"2019-09-06T21:16:44Z","git_url":"git://github.com/Esri/arcgis-runtime-samples-ios.git","ssh_url":"git@github.com:Esri/arcgis-runtime-samples-ios.git","clone_url":"https://github.com/Esri/arcgis-runtime-samples-ios.git","svn_url":"https://github.com/Esri/arcgis-runtime-samples-ios","homepage":"http://developers.arcgis.com/en/ios/","size":484191,"stargazers_count":244,"watchers_count":244,"language":"Swift","has_issues":true,"has_projects":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":364,"mirror_url":null,"archived":false,"disabled":false,"open_issues_count":17,"license":{"key":"apache-2.0","name":"Apache License 2.0","spdx_id":"Apache-2.0","url":"https://api.github.com/licenses/apache-2.0","node_id":"MDc6TGljZW5zZTI="},"forks":364,"open_issues":17,"watchers":244,"default_branch":"master"}},"base":{"label":"Esri:v.next","ref":"v.next","sha":"d5ab5ce8388bfa71bf6c9a00239e220139de9fea","user":{"login":"Esri","id":628795,"node_id":"MDEyOk9yZ2FuaXphdGlvbjYyODc5NQ==","avatar_url":"https://avatars1.githubusercontent.com/u/628795?v=4","gravatar_id":"","url":"https://api.github.com/users/Esri","html_url":"https://github.com/Esri","followers_url":"https://api.github.com/users/Esri/followers","following_url":"https://api.github.com/users/Esri/following{/other_user}","gists_url":"https://api.github.com/users/Esri/gists{/gist_id}","starred_url":"https://api.github.com/users/Esri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Esri/subscriptions","organizations_url":"https://api.github.com/users/Esri/orgs","repos_url":"https://api.github.com/users/Esri/repos","events_url":"https://api.github.com/users/Esri/events{/privacy}","received_events_url":"https://api.github.com/users/Esri/received_events","type":"Organization","site_admin":false},"repo":{"id":7922900,"node_id":"MDEwOlJlcG9zaXRvcnk3OTIyOTAw","name":"arcgis-runtime-samples-ios","full_name":"Esri/arcgis-runtime-samples-ios","private":false,"owner":{"login":"Esri","id":628795,"node_id":"MDEyOk9yZ2FuaXphdGlvbjYyODc5NQ==","avatar_url":"https://avatars1.githubusercontent.com/u/628795?v=4","gravatar_id":"","url":"https://api.github.com/users/Esri","html_url":"https://github.com/Esri","followers_url":"https://api.github.com/users/Esri/followers","following_url":"https://api.github.com/users/Esri/following{/other_user}","gists_url":"https://api.github.com/users/Esri/gists{/gist_id}","starred_url":"https://api.github.com/users/Esri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Esri/subscriptions","organizations_url":"https://api.github.com/users/Esri/orgs","repos_url":"https://api.github.com/users/Esri/repos","events_url":"https://api.github.com/users/Esri/events{/privacy}","received_events_url":"https://api.github.com/users/Esri/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/Esri/arcgis-runtime-samples-ios","description":"Swift samples demonstrating various capabilities of ArcGIS Runtime SDK for iOS","fork":false,"url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios","forks_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/forks","keys_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/teams","hooks_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/hooks","issue_events_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/issues/events{/number}","events_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/events","assignees_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/assignees{/user}","branches_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/branches{/branch}","tags_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/tags","blobs_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/git/refs{/sha}","trees_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/statuses/{sha}","languages_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/languages","stargazers_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/stargazers","contributors_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/contributors","subscribers_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/subscribers","subscription_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/subscription","commits_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/commits{/sha}","git_commits_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/git/commits{/sha}","comments_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/comments{/number}","issue_comment_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/issues/comments{/number}","contents_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/contents/{+path}","compare_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/merges","archive_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/downloads","issues_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/issues{/number}","pulls_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls{/number}","milestones_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/milestones{/number}","notifications_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/labels{/name}","releases_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/releases{/id}","deployments_url":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/deployments","created_at":"2013-01-30T20:18:40Z","updated_at":"2019-09-04T20:21:58Z","pushed_at":"2019-09-06T21:16:44Z","git_url":"git://github.com/Esri/arcgis-runtime-samples-ios.git","ssh_url":"git@github.com:Esri/arcgis-runtime-samples-ios.git","clone_url":"https://github.com/Esri/arcgis-runtime-samples-ios.git","svn_url":"https://github.com/Esri/arcgis-runtime-samples-ios","homepage":"http://developers.arcgis.com/en/ios/","size":484191,"stargazers_count":244,"watchers_count":244,"language":"Swift","has_issues":true,"has_projects":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":364,"mirror_url":null,"archived":false,"disabled":false,"open_issues_count":17,"license":{"key":"apache-2.0","name":"Apache License 2.0","spdx_id":"Apache-2.0","url":"https://api.github.com/licenses/apache-2.0","node_id":"MDc6TGljZW5zZTI="},"forks":364,"open_issues":17,"watchers":244,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/750"},"html":{"href":"https://github.com/Esri/arcgis-runtime-samples-ios/pull/750"},"issue":{"href":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/issues/750"},"comments":{"href":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/issues/750/comments"},"review_comments":{"href":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/750/comments"},"review_comment":{"href":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/pulls/750/commits"},"statuses":{"href":"https://api.github.com/repos/Esri/arcgis-runtime-samples-ios/statuses/0ba4bd6dfd8f638951cceb44112aaa13b4d214c0"}},"author_association":"COLLABORATOR"}}
other: {"actor":{"display_login":"sbaskaran"}}