{"action":"created","issue":{"url":"https://api.github.com/repos/hyperledger/fabric/issues/1433","repository_url":"https://api.github.com/repos/hyperledger/fabric","labels_url":"https://api.github.com/repos/hyperledger/fabric/issues/1433/labels{/name}","comments_url":"https://api.github.com/repos/hyperledger/fabric/issues/1433/comments","events_url":"https://api.github.com/repos/hyperledger/fabric/issues/1433/events","html_url":"https://github.com/hyperledger/fabric/pull/1433","id":154089620,"number":1433,"title":"Update chaincode APIs","user":{"login":"binhn","id":3342057,"avatar_url":"https://avatars.githubusercontent.com/u/3342057?v=3","gravatar_id":"","url":"https://api.github.com/users/binhn","html_url":"https://github.com/binhn","followers_url":"https://api.github.com/users/binhn/followers","following_url":"https://api.github.com/users/binhn/following{/other_user}","gists_url":"https://api.github.com/users/binhn/gists{/gist_id}","starred_url":"https://api.github.com/users/binhn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/binhn/subscriptions","organizations_url":"https://api.github.com/users/binhn/orgs","repos_url":"https://api.github.com/users/binhn/repos","events_url":"https://api.github.com/users/binhn/events{/privacy}","received_events_url":"https://api.github.com/users/binhn/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-05-10T19:41:08Z","updated_at":"2016-05-11T17:37:58Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hyperledger/fabric/pulls/1433","html_url":"https://github.com/hyperledger/fabric/pull/1433","diff_url":"https://github.com/hyperledger/fabric/pull/1433.diff","patch_url":"https://github.com/hyperledger/fabric/pull/1433.patch"},"body":"<!--- Provide a general summary of your changes in the Title above -->\r\nPassing transaction timestamp to the chaincode context and update the APIs\r\n\r\n## Description\r\n<!--- Describe your changes in detail. -->\r\nChaincode needs the transaction creation timestamp. The fabric includes the timestamp in the context passed to the chaincode on init, invoke, and query calls. This change also includes update to the chaincode API document and godoc comments. Unit-test for the timestamp is added to the chaincode_example01, which is run as part of the test-suite. \r\n\r\n## Motivation and Context\r\n<!--- Why is this change required? What problem does it solve? -->\r\n<!--- If it fixes an open issue, please link to the issue here. -->\r\nFixes #875\r\n\r\n## How Has This Been Tested?\r\n<!--- Please describe in detail how you tested your changes. -->\r\n<!--- If this PR does not contain a new test case, explain why. -->\r\nRun the unit-tests or the chaincode_example01\r\n\r\n## Checklist:\r\n<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->\r\n<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->\r\n- [x ] I have added a [Signed-off-by](https://github.com/hyperledger/fabric/blob/master/CONTRIBUTING.md#legal-stuff)\r\n- [x ] Either no new documentation is required by this change, OR I added new documentation\r\n- [ x] Either no new tests are required by this change, OR I added new tests\r\n- [x ] I have run [goimports](https://godoc.org/golang.org/x/tools/cmd/goimports), [go vet](https://golang.org/cmd/vet/), and [golint](https://github.com/golang/lint). I have cleaned up all valid errors and warnings in code I have added or modified. These tools may generate false positives. Don't be worried about ignoring some errors or warnings. The goal is clean, consistent, and readable code.\r\n\r\nSigned-off-by: binhn@us.ibm.com\r\n"},"comment":{"url":"https://api.github.com/repos/hyperledger/fabric/issues/comments/218533276","html_url":"https://github.com/hyperledger/fabric/pull/1433#issuecomment-218533276","issue_url":"https://api.github.com/repos/hyperledger/fabric/issues/1433","id":218533276,"user":{"login":"binhn","id":3342057,"avatar_url":"https://avatars.githubusercontent.com/u/3342057?v=3","gravatar_id":"","url":"https://api.github.com/users/binhn","html_url":"https://github.com/binhn","followers_url":"https://api.github.com/users/binhn/followers","following_url":"https://api.github.com/users/binhn/following{/other_user}","gists_url":"https://api.github.com/users/binhn/gists{/gist_id}","starred_url":"https://api.github.com/users/binhn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/binhn/subscriptions","organizations_url":"https://api.github.com/users/binhn/orgs","repos_url":"https://api.github.com/users/binhn/repos","events_url":"https://api.github.com/users/binhn/events{/privacy}","received_events_url":"https://api.github.com/users/binhn/received_events","type":"User","site_admin":false},"created_at":"2016-05-11T17:37:58Z","updated_at":"2016-05-11T17:37:58Z","body":"@srderson i rebased and force push to update the pr.  Please merge\r\n"}}