{"action":"created","comment":{"url":"https://api.github.com/repos/garnryang/Team8/pulls/comments/32795511","id":32795511,"diff_hunk":"@@ -106,12 +123,23 @@ public void doRedo() {\n \tpublic void enterNumber(CellCoordinates cellCoordinates, int number) {\r\n \t\t\r\n \t\tCell currentCell = board.getCell(cellCoordinates.getRowIndex(), cellCoordinates.getColumnIndex());\r\n-\t\tSudokuAction sudokuAction = new SudokuAction(cellCoordinates, currentCell.getNumber(), number);\r\n \t\t\r\n-\t\t/* TODO Actual Board Update to be implemented */\r\n+\t\t/* TODO - come back after PencilMarkMaanger is implement\r\n+\t\t * current assumption is that PecilMarkManager will be a snapshot of the current pencil mark status \r","path":"src/edu/psu/sweng500/team8/play/GameSession.java","position":92,"original_position":92,"commit_id":"e20a47cd8320305eae97ac2807caa6dc7048effc","original_commit_id":"682357c7db839a83b795148e76e749f01504c5b4","user":{"login":"garnryang","id":12421716,"avatar_url":"https://avatars.githubusercontent.com/u/12421716?v=3","gravatar_id":"","url":"https://api.github.com/users/garnryang","html_url":"https://github.com/garnryang","followers_url":"https://api.github.com/users/garnryang/followers","following_url":"https://api.github.com/users/garnryang/following{/other_user}","gists_url":"https://api.github.com/users/garnryang/gists{/gist_id}","starred_url":"https://api.github.com/users/garnryang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garnryang/subscriptions","organizations_url":"https://api.github.com/users/garnryang/orgs","repos_url":"https://api.github.com/users/garnryang/repos","events_url":"https://api.github.com/users/garnryang/events{/privacy}","received_events_url":"https://api.github.com/users/garnryang/received_events","type":"User","site_admin":false},"body":"PencilMarkManager being a singleton and it controlling updates is fine.\r\nSo if Cell.pencilMarks is used to keep track of pencil mark per cell, is PencilMarkManager going to iterate through each cell to wipeout when a number is entered?\r\nRegardless, I will update the code to get a snapshot of the board's pencil mark using Cell.pencilMarks.","created_at":"2015-06-19T01:35:27Z","updated_at":"2015-06-19T01:35:27Z","html_url":"https://github.com/garnryang/Team8/pull/3#discussion_r32795511","pull_request_url":"https://api.github.com/repos/garnryang/Team8/pulls/3","_links":{"self":{"href":"https://api.github.com/repos/garnryang/Team8/pulls/comments/32795511"},"html":{"href":"https://github.com/garnryang/Team8/pull/3#discussion_r32795511"},"pull_request":{"href":"https://api.github.com/repos/garnryang/Team8/pulls/3"}}},"pull_request":{"url":"https://api.github.com/repos/garnryang/Team8/pulls/3","id":37943109,"html_url":"https://github.com/garnryang/Team8/pull/3","diff_url":"https://github.com/garnryang/Team8/pull/3.diff","patch_url":"https://github.com/garnryang/Team8/pull/3.patch","issue_url":"https://api.github.com/repos/garnryang/Team8/issues/3","number":3,"state":"open","locked":false,"title":"Scott","user":{"login":"garnryang","id":12421716,"avatar_url":"https://avatars.githubusercontent.com/u/12421716?v=3","gravatar_id":"","url":"https://api.github.com/users/garnryang","html_url":"https://github.com/garnryang","followers_url":"https://api.github.com/users/garnryang/followers","following_url":"https://api.github.com/users/garnryang/following{/other_user}","gists_url":"https://api.github.com/users/garnryang/gists{/gist_id}","starred_url":"https://api.github.com/users/garnryang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garnryang/subscriptions","organizations_url":"https://api.github.com/users/garnryang/orgs","repos_url":"https://api.github.com/users/garnryang/repos","events_url":"https://api.github.com/users/garnryang/events{/privacy}","received_events_url":"https://api.github.com/users/garnryang/received_events","type":"User","site_admin":false},"body":"basic redo/undo implemented\r\nplaceholder for pencil mark is added (assuming pencilMarkManager will hold the snapshot of pencil mark state)","created_at":"2015-06-17T20:29:43Z","updated_at":"2015-06-19T01:35:27Z","closed_at":null,"merged_at":null,"merge_commit_sha":"e93b834d1b8be6a9fd68ddb90cd37cbd61e10a05","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/garnryang/Team8/pulls/3/commits","review_comments_url":"https://api.github.com/repos/garnryang/Team8/pulls/3/comments","review_comment_url":"https://api.github.com/repos/garnryang/Team8/pulls/comments{/number}","comments_url":"https://api.github.com/repos/garnryang/Team8/issues/3/comments","statuses_url":"https://api.github.com/repos/garnryang/Team8/statuses/e20a47cd8320305eae97ac2807caa6dc7048effc","head":{"label":"garnryang:ScottL","ref":"ScottL","sha":"e20a47cd8320305eae97ac2807caa6dc7048effc","user":{"login":"garnryang","id":12421716,"avatar_url":"https://avatars.githubusercontent.com/u/12421716?v=3","gravatar_id":"","url":"https://api.github.com/users/garnryang","html_url":"https://github.com/garnryang","followers_url":"https://api.github.com/users/garnryang/followers","following_url":"https://api.github.com/users/garnryang/following{/other_user}","gists_url":"https://api.github.com/users/garnryang/gists{/gist_id}","starred_url":"https://api.github.com/users/garnryang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garnryang/subscriptions","organizations_url":"https://api.github.com/users/garnryang/orgs","repos_url":"https://api.github.com/users/garnryang/repos","events_url":"https://api.github.com/users/garnryang/events{/privacy}","received_events_url":"https://api.github.com/users/garnryang/received_events","type":"User","site_admin":false},"repo":{"id":35524748,"name":"Team8","full_name":"garnryang/Team8","owner":{"login":"garnryang","id":12421716,"avatar_url":"https://avatars.githubusercontent.com/u/12421716?v=3","gravatar_id":"","url":"https://api.github.com/users/garnryang","html_url":"https://github.com/garnryang","followers_url":"https://api.github.com/users/garnryang/followers","following_url":"https://api.github.com/users/garnryang/following{/other_user}","gists_url":"https://api.github.com/users/garnryang/gists{/gist_id}","starred_url":"https://api.github.com/users/garnryang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garnryang/subscriptions","organizations_url":"https://api.github.com/users/garnryang/orgs","repos_url":"https://api.github.com/users/garnryang/repos","events_url":"https://api.github.com/users/garnryang/events{/privacy}","received_events_url":"https://api.github.com/users/garnryang/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/garnryang/Team8","description":"SWENG500 - Team8","fork":false,"url":"https://api.github.com/repos/garnryang/Team8","forks_url":"https://api.github.com/repos/garnryang/Team8/forks","keys_url":"https://api.github.com/repos/garnryang/Team8/keys{/key_id}","collaborators_url":"https://api.github.com/repos/garnryang/Team8/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/garnryang/Team8/teams","hooks_url":"https://api.github.com/repos/garnryang/Team8/hooks","issue_events_url":"https://api.github.com/repos/garnryang/Team8/issues/events{/number}","events_url":"https://api.github.com/repos/garnryang/Team8/events","assignees_url":"https://api.github.com/repos/garnryang/Team8/assignees{/user}","branches_url":"https://api.github.com/repos/garnryang/Team8/branches{/branch}","tags_url":"https://api.github.com/repos/garnryang/Team8/tags","blobs_url":"https://api.github.com/repos/garnryang/Team8/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/garnryang/Team8/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/garnryang/Team8/git/refs{/sha}","trees_url":"https://api.github.com/repos/garnryang/Team8/git/trees{/sha}","statuses_url":"https://api.github.com/repos/garnryang/Team8/statuses/{sha}","languages_url":"https://api.github.com/repos/garnryang/Team8/languages","stargazers_url":"https://api.github.com/repos/garnryang/Team8/stargazers","contributors_url":"https://api.github.com/repos/garnryang/Team8/contributors","subscribers_url":"https://api.github.com/repos/garnryang/Team8/subscribers","subscription_url":"https://api.github.com/repos/garnryang/Team8/subscription","commits_url":"https://api.github.com/repos/garnryang/Team8/commits{/sha}","git_commits_url":"https://api.github.com/repos/garnryang/Team8/git/commits{/sha}","comments_url":"https://api.github.com/repos/garnryang/Team8/comments{/number}","issue_comment_url":"https://api.github.com/repos/garnryang/Team8/issues/comments{/number}","contents_url":"https://api.github.com/repos/garnryang/Team8/contents/{+path}","compare_url":"https://api.github.com/repos/garnryang/Team8/compare/{base}...{head}","merges_url":"https://api.github.com/repos/garnryang/Team8/merges","archive_url":"https://api.github.com/repos/garnryang/Team8/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/garnryang/Team8/downloads","issues_url":"https://api.github.com/repos/garnryang/Team8/issues{/number}","pulls_url":"https://api.github.com/repos/garnryang/Team8/pulls{/number}","milestones_url":"https://api.github.com/repos/garnryang/Team8/milestones{/number}","notifications_url":"https://api.github.com/repos/garnryang/Team8/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/garnryang/Team8/labels{/name}","releases_url":"https://api.github.com/repos/garnryang/Team8/releases{/id}","created_at":"2015-05-13T02:51:29Z","updated_at":"2015-05-26T01:39:48Z","pushed_at":"2015-06-19T01:20:24Z","git_url":"git://github.com/garnryang/Team8.git","ssh_url":"git@github.com:garnryang/Team8.git","clone_url":"https://github.com/garnryang/Team8.git","svn_url":"https://github.com/garnryang/Team8","homepage":null,"size":2063,"stargazers_count":1,"watchers_count":1,"language":"Java","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":1,"mirror_url":null,"open_issues_count":2,"forks":1,"open_issues":2,"watchers":1,"default_branch":"master"}},"base":{"label":"garnryang:master","ref":"master","sha":"b8c3994fa5ad8dd56ec414d9a6c62ca3e9b261e8","user":{"login":"garnryang","id":12421716,"avatar_url":"https://avatars.githubusercontent.com/u/12421716?v=3","gravatar_id":"","url":"https://api.github.com/users/garnryang","html_url":"https://github.com/garnryang","followers_url":"https://api.github.com/users/garnryang/followers","following_url":"https://api.github.com/users/garnryang/following{/other_user}","gists_url":"https://api.github.com/users/garnryang/gists{/gist_id}","starred_url":"https://api.github.com/users/garnryang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garnryang/subscriptions","organizations_url":"https://api.github.com/users/garnryang/orgs","repos_url":"https://api.github.com/users/garnryang/repos","events_url":"https://api.github.com/users/garnryang/events{/privacy}","received_events_url":"https://api.github.com/users/garnryang/received_events","type":"User","site_admin":false},"repo":{"id":35524748,"name":"Team8","full_name":"garnryang/Team8","owner":{"login":"garnryang","id":12421716,"avatar_url":"https://avatars.githubusercontent.com/u/12421716?v=3","gravatar_id":"","url":"https://api.github.com/users/garnryang","html_url":"https://github.com/garnryang","followers_url":"https://api.github.com/users/garnryang/followers","following_url":"https://api.github.com/users/garnryang/following{/other_user}","gists_url":"https://api.github.com/users/garnryang/gists{/gist_id}","starred_url":"https://api.github.com/users/garnryang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garnryang/subscriptions","organizations_url":"https://api.github.com/users/garnryang/orgs","repos_url":"https://api.github.com/users/garnryang/repos","events_url":"https://api.github.com/users/garnryang/events{/privacy}","received_events_url":"https://api.github.com/users/garnryang/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/garnryang/Team8","description":"SWENG500 - Team8","fork":false,"url":"https://api.github.com/repos/garnryang/Team8","forks_url":"https://api.github.com/repos/garnryang/Team8/forks","keys_url":"https://api.github.com/repos/garnryang/Team8/keys{/key_id}","collaborators_url":"https://api.github.com/repos/garnryang/Team8/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/garnryang/Team8/teams","hooks_url":"https://api.github.com/repos/garnryang/Team8/hooks","issue_events_url":"https://api.github.com/repos/garnryang/Team8/issues/events{/number}","events_url":"https://api.github.com/repos/garnryang/Team8/events","assignees_url":"https://api.github.com/repos/garnryang/Team8/assignees{/user}","branches_url":"https://api.github.com/repos/garnryang/Team8/branches{/branch}","tags_url":"https://api.github.com/repos/garnryang/Team8/tags","blobs_url":"https://api.github.com/repos/garnryang/Team8/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/garnryang/Team8/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/garnryang/Team8/git/refs{/sha}","trees_url":"https://api.github.com/repos/garnryang/Team8/git/trees{/sha}","statuses_url":"https://api.github.com/repos/garnryang/Team8/statuses/{sha}","languages_url":"https://api.github.com/repos/garnryang/Team8/languages","stargazers_url":"https://api.github.com/repos/garnryang/Team8/stargazers","contributors_url":"https://api.github.com/repos/garnryang/Team8/contributors","subscribers_url":"https://api.github.com/repos/garnryang/Team8/subscribers","subscription_url":"https://api.github.com/repos/garnryang/Team8/subscription","commits_url":"https://api.github.com/repos/garnryang/Team8/commits{/sha}","git_commits_url":"https://api.github.com/repos/garnryang/Team8/git/commits{/sha}","comments_url":"https://api.github.com/repos/garnryang/Team8/comments{/number}","issue_comment_url":"https://api.github.com/repos/garnryang/Team8/issues/comments{/number}","contents_url":"https://api.github.com/repos/garnryang/Team8/contents/{+path}","compare_url":"https://api.github.com/repos/garnryang/Team8/compare/{base}...{head}","merges_url":"https://api.github.com/repos/garnryang/Team8/merges","archive_url":"https://api.github.com/repos/garnryang/Team8/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/garnryang/Team8/downloads","issues_url":"https://api.github.com/repos/garnryang/Team8/issues{/number}","pulls_url":"https://api.github.com/repos/garnryang/Team8/pulls{/number}","milestones_url":"https://api.github.com/repos/garnryang/Team8/milestones{/number}","notifications_url":"https://api.github.com/repos/garnryang/Team8/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/garnryang/Team8/labels{/name}","releases_url":"https://api.github.com/repos/garnryang/Team8/releases{/id}","created_at":"2015-05-13T02:51:29Z","updated_at":"2015-05-26T01:39:48Z","pushed_at":"2015-06-19T01:20:24Z","git_url":"git://github.com/garnryang/Team8.git","ssh_url":"git@github.com:garnryang/Team8.git","clone_url":"https://github.com/garnryang/Team8.git","svn_url":"https://github.com/garnryang/Team8","homepage":null,"size":2063,"stargazers_count":1,"watchers_count":1,"language":"Java","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":1,"mirror_url":null,"open_issues_count":2,"forks":1,"open_issues":2,"watchers":1,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/garnryang/Team8/pulls/3"},"html":{"href":"https://github.com/garnryang/Team8/pull/3"},"issue":{"href":"https://api.github.com/repos/garnryang/Team8/issues/3"},"comments":{"href":"https://api.github.com/repos/garnryang/Team8/issues/3/comments"},"review_comments":{"href":"https://api.github.com/repos/garnryang/Team8/pulls/3/comments"},"review_comment":{"href":"https://api.github.com/repos/garnryang/Team8/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/garnryang/Team8/pulls/3/commits"},"statuses":{"href":"https://api.github.com/repos/garnryang/Team8/statuses/e20a47cd8320305eae97ac2807caa6dc7048effc"}}}}