Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix function name typo in store private data command #1901

Merged
merged 1 commit into from
Sep 18, 2020
Merged

fix function name typo in store private data command #1901

merged 1 commit into from
Sep 18, 2020

Conversation

Salmandabbakuti
Copy link

Type of change

  • Documentation update

Description

fix function name typo in store private data command

Signed-off-by: Salman Dabbakuti dabbakutisalman@gmail.com

Signed-off-by: Salmandabbakuti <Salmandabbakuti@github.com>
Copy link
Contributor

@ale-linux ale-linux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution!

@ale-linux ale-linux merged commit 3ee47a7 into hyperledger:release-2.2 Sep 18, 2020
@Salmandabbakuti Salmandabbakuti deleted the patch-1 branch September 18, 2020 12:30
@@ -535,7 +535,7 @@ problematic newline characters that linux base64 command adds.
.. code:: bash

export MARBLE=$(echo -n "{\"name\":\"marble1\",\"color\":\"blue\",\"size\":35,\"owner\":\"tom\",\"price\":99}" | base64 | tr -d \\n)
peer chaincode invoke -o localhost:7050 --ordererTLSHostnameOverride orderer.example.com --tls --cafile ${PWD}/organizations/ordererOrganizations/example.com/orderers/orderer.example.com/msp/tlscacerts/tlsca.example.com-cert.pem -C mychannel -n marblesp -c '{"Args":["InitMarble"]}' --transient "{\"marble\":\"$MARBLE\"}"
peer chaincode invoke -o localhost:7050 --ordererTLSHostnameOverride orderer.example.com --tls --cafile ${PWD}/organizations/ordererOrganizations/example.com/orderers/orderer.example.com/msp/tlscacerts/tlsca.example.com-cert.pem -C mychannel -n marblesp -c '{"Args":["initMarble"]}' --transient "{\"marble\":\"$MARBLE\"}"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fyi The Marble PD sample is soon getting deprecated. See the change to this tutorial at #1875

@denyeart
Copy link
Contributor

@ale-linux @Salmandabbakuti actually it should remain InitMarble. The samples were updated a couple months ago to capitalize function names.
But it doesn't really matter, as it appears the contract api layer is case insensitive on calls.
Plus the tutorial will switch to basic asset transfer soon when the PR is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants