{"action":"created","issue":{"url":"https://api.github.com/repos/oscaraya/PT1_SCM_Workshop2/issues/1","repository_url":"https://api.github.com/repos/oscaraya/PT1_SCM_Workshop2","labels_url":"https://api.github.com/repos/oscaraya/PT1_SCM_Workshop2/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/oscaraya/PT1_SCM_Workshop2/issues/1/comments","events_url":"https://api.github.com/repos/oscaraya/PT1_SCM_Workshop2/issues/1/events","html_url":"https://github.com/oscaraya/PT1_SCM_Workshop2/issues/1","id":176666576,"number":1,"title":"[CR] Improve \"Show Total Cash Held\" function","user":{"login":"rongxin","id":1207819,"avatar_url":"https://avatars.githubusercontent.com/u/1207819?v=3","gravatar_id":"","url":"https://api.github.com/users/rongxin","html_url":"https://github.com/rongxin","followers_url":"https://api.github.com/users/rongxin/followers","following_url":"https://api.github.com/users/rongxin/following{/other_user}","gists_url":"https://api.github.com/users/rongxin/gists{/gist_id}","starred_url":"https://api.github.com/users/rongxin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rongxin/subscriptions","organizations_url":"https://api.github.com/users/rongxin/orgs","repos_url":"https://api.github.com/users/rongxin/repos","events_url":"https://api.github.com/users/rongxin/events{/privacy}","received_events_url":"https://api.github.com/users/rongxin/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/oscaraya/PT1_SCM_Workshop2/labels/Change%20Request","name":"Change Request","color":"5319e7"}],"state":"open","locked":false,"assignee":{"login":"LANXIANGMING","id":11241235,"avatar_url":"https://avatars.githubusercontent.com/u/11241235?v=3","gravatar_id":"","url":"https://api.github.com/users/LANXIANGMING","html_url":"https://github.com/LANXIANGMING","followers_url":"https://api.github.com/users/LANXIANGMING/followers","following_url":"https://api.github.com/users/LANXIANGMING/following{/other_user}","gists_url":"https://api.github.com/users/LANXIANGMING/gists{/gist_id}","starred_url":"https://api.github.com/users/LANXIANGMING/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LANXIANGMING/subscriptions","organizations_url":"https://api.github.com/users/LANXIANGMING/orgs","repos_url":"https://api.github.com/users/LANXIANGMING/repos","events_url":"https://api.github.com/users/LANXIANGMING/events{/privacy}","received_events_url":"https://api.github.com/users/LANXIANGMING/received_events","type":"User","site_admin":false},"assignees":[{"login":"LANXIANGMING","id":11241235,"avatar_url":"https://avatars.githubusercontent.com/u/11241235?v=3","gravatar_id":"","url":"https://api.github.com/users/LANXIANGMING","html_url":"https://github.com/LANXIANGMING","followers_url":"https://api.github.com/users/LANXIANGMING/followers","following_url":"https://api.github.com/users/LANXIANGMING/following{/other_user}","gists_url":"https://api.github.com/users/LANXIANGMING/gists{/gist_id}","starred_url":"https://api.github.com/users/LANXIANGMING/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LANXIANGMING/subscriptions","organizations_url":"https://api.github.com/users/LANXIANGMING/orgs","repos_url":"https://api.github.com/users/LANXIANGMING/repos","events_url":"https://api.github.com/users/LANXIANGMING/events{/privacy}","received_events_url":"https://api.github.com/users/LANXIANGMING/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/oscaraya/PT1_SCM_Workshop2/milestones/1","html_url":"https://github.com/oscaraya/PT1_SCM_Workshop2/milestones/1.0.9","labels_url":"https://api.github.com/repos/oscaraya/PT1_SCM_Workshop2/milestones/1/labels","id":2001341,"number":1,"title":"1.0.9","description":"Planned release for Change request and bug fixes.","creator":{"login":"rongxin","id":1207819,"avatar_url":"https://avatars.githubusercontent.com/u/1207819?v=3","gravatar_id":"","url":"https://api.github.com/users/rongxin","html_url":"https://github.com/rongxin","followers_url":"https://api.github.com/users/rongxin/followers","following_url":"https://api.github.com/users/rongxin/following{/other_user}","gists_url":"https://api.github.com/users/rongxin/gists{/gist_id}","starred_url":"https://api.github.com/users/rongxin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rongxin/subscriptions","organizations_url":"https://api.github.com/users/rongxin/orgs","repos_url":"https://api.github.com/users/rongxin/repos","events_url":"https://api.github.com/users/rongxin/events{/privacy}","received_events_url":"https://api.github.com/users/rongxin/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":0,"state":"open","created_at":"2016-09-13T14:52:18Z","updated_at":"2016-09-13T15:06:29Z","due_on":"2016-09-24T00:00:00Z","closed_at":null},"comments":5,"created_at":"2016-09-13T15:06:19Z","updated_at":"2016-09-15T09:15:06Z","closed_at":null,"body":"## Background\r\n### Current\r\nOn “Maintenance Panel”, the user (“Maintainer”) needs to click on each denomination to see the number of coins to be collected for each denomination. \r\n\r\nThe user then needs to click on “Show Total Cash Held” to see the total cash amount to be collected. \r\n\r\n### Problem\r\nIt has been reported that it is possible that what the user collected does not match with the amount recorded in the system. The current screen is causing inconvenience and confusion to the user by forcing him to click buttons one by one for each denomination and manually calculate each amount. Moreover, the user needs to calculate the total number of coins collected, which becomes another source of error. \r\n\r\n## Changes needed\r\nAfter pressing on “Show Total Cash Held” by the Maintainer, \r\n- the specific quantity of coins per denomination together with total cash amount for each denomination should be displayed. \r\n- the total number of coins should be displayed beside the total cash amount\r\n\r\n## Development Notes\r\n\r\n- Changes to be pushed to following *feature branch*, please check out the branch when start the task.\r\n  `feature/CR1-TotalCashHeld`\r\n- Specify the git issue number in the commit message:\r\n\r\n   E.g.  `git commit -m \"#1 | code changes details\"`\r\n\r\n- Perform the internal test and attach test result to the issue before pass to QA testing.\r\n- Assign to tech lead when the implementation and internal testing are done.\r\n\r\n### MaintenanceController.java \r\n- Add functions to get amount for each denomination. \r\n\r\n### MaintenancePanel.java\r\n- Change text \"Quantity of Coins Available\" to \"Quantity\" to leave more space. \r\n- Add new label \"Coin Value\" for indicating the coin value for each denomination.  \r\n- Add new label \"Amount\" for indication the amount of coin for each denomination. \r\n- Add new fields on the right side of coin quantity for each denomination to display amount of coins. \r\n- Remove buttons which is used to display quantity of coins for each denomination after pressing. \r\n- Display quantity of coins for each denomination automatically. \r\n- Add the new field on the right side of total cash amount to display total quantity of collected coins.  \r\n\r\n### TotalCashButtonListener.java \r\n- Add functions to get total quantity of coins to collect."},"comment":{"url":"https://api.github.com/repos/oscaraya/PT1_SCM_Workshop2/issues/comments/247277557","html_url":"https://github.com/oscaraya/PT1_SCM_Workshop2/issues/1#issuecomment-247277557","issue_url":"https://api.github.com/repos/oscaraya/PT1_SCM_Workshop2/issues/1","id":247277557,"user":{"login":"rongxin","id":1207819,"avatar_url":"https://avatars.githubusercontent.com/u/1207819?v=3","gravatar_id":"","url":"https://api.github.com/users/rongxin","html_url":"https://github.com/rongxin","followers_url":"https://api.github.com/users/rongxin/followers","following_url":"https://api.github.com/users/rongxin/following{/other_user}","gists_url":"https://api.github.com/users/rongxin/gists{/gist_id}","starred_url":"https://api.github.com/users/rongxin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rongxin/subscriptions","organizations_url":"https://api.github.com/users/rongxin/orgs","repos_url":"https://api.github.com/users/rongxin/repos","events_url":"https://api.github.com/users/rongxin/events{/privacy}","received_events_url":"https://api.github.com/users/rongxin/received_events","type":"User","site_admin":false},"created_at":"2016-09-15T09:15:06Z","updated_at":"2016-09-15T09:15:06Z","body":"@LANXIANGMING Thanks!\r\nI have merged the codes from feature branch to develop branch.\r\n\r\n@suhhgd I am passing this issue to you for QA testing.\r\nPlease pull latest codes from **develop** branch for the testing."}}
{"actor":{"display_login":"rongxin"}}