The tutorial should eventually contain the asset transfer flow that is contained in the readme here: https://github.com/hyperledger/fabric-samples/tree/master/asset-transfer-private-data/chaincode-go. Eventually we should aim to remove that readme and link to the tutorial. If the tutorial works from e2e, I am ok merging it and iterating from there, so that we are away from the dependency on the old private data smart contract.

I think we can also replace the commands to use the chaincode lifecycle with the command to deploy using the test network script to shorten the tutorial, but that can also follow later.

The tutorial should eventually contain the asset transfer flow that is contained in the readme here: https://github.com/hyperledger/fabric-samples/tree/master/asset-transfer-private-data/chaincode-go. Eventually we should aim to remove that readme and link to the tutorial. If the tutorial works from e2e, I am ok merging it and iterating from there, so that we are away from the dependency on the old private data smart contract.

I think we can also replace the commands to use the chaincode lifecycle with the command to deploy using the test network script to shorten the tutorial, but that can also follow later.

I added an asset transfer header, to specifically call this out. The original direction was to integrate the private data README into the existing tutorial which I took to mean that we wanted to keep the flow of the tutorial. We can certainly just adopt the README flow as well and drop the peer chaincode lifecycle piece in favor of the script approach to make it shorter, but I agree we should merge for now and do that in another PR.

The tutorial should eventually contain the asset transfer flow that is contained in the readme here: https://github.com/hyperledger/fabric-samples/tree/master/asset-transfer-private-data/chaincode-go. Eventually we should aim to remove that readme and link to the tutorial. If the tutorial works from e2e, I am ok merging it and iterating from there, so that we are away from the dependency on the old private data smart contract.
I think we can also replace the commands to use the chaincode lifecycle with the command to deploy using the test network script to shorten the tutorial, but that can also follow later.

I added an asset transfer header, to specifically call this out. The original direction was to integrate the private data README into the existing tutorial which I took to mean that we wanted to keep the flow of the tutorial. We can certainly just adopt the README flow as well and drop the peer chaincode lifecycle piece in favor of the script approach to make it shorter, but I agree we should merge for now and do that in another PR.

I agree that we need to maintain the current flow of the tutorial, at least in terms of keeping the technical sections. we can work to integrate and improve the flow until the readme is redundant.

Why the addition of the two Pipfile's? Especially the one in the root of the repo?

@btl5037 this has been addressed for some time, please review so we can get merged.

@Mergifyio backport release-2.2

Command backport release-2.2: failure

No backport have been created

• Backport to branch release-2.2 failed

Cherry-pick of 17858a5 has failed:

On branch mergify/bp/release-2.2/pr-1875
Your branch is up to date with 'origin/release-2.2'.

You are currently cherry-picking commit 17858a570.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:

	modified:   docs/source/images/SideDB-org1-org2.png
	modified:   docs/source/images/SideDBTutorialImages.pptx

Unmerged paths:
  (use "git add <file>..." to mark resolution)

	both modified:   docs/source/private_data_tutorial.rst