The following statement in Customer Needs section is interesting.

"The Blockchain implementation needs to be secure and fully under the control of Northwind Traders; while simultaneously integrating with partners and third-parties when managing the supply chain."

It says the blockchain ledger needs to be "fully under the control of Northwind Traders". The USP of blockchain is that it is shared ledger among the participants. Ideally there should not be any one Super User or Super Admin of the ledger (One node being "more equal" than the others) for the idea to work. Otherwise, it is just a data store that other participants on the supply chain "choose" to trust. If the blockchain is completely in NW's control, I don't see how it makes sense for its suppliers and partners as a shared ledger that no one organization has complete control on.

Thoughts?

Enhancement suggestion

One of the major hurdles in Workbench development was the need to rename the smart contract name and file after every enhancement or code change. With 1.6.0, there is versioning introduced.
Thus HoL will do a lot of good by showcasing the way to update a contract and upload it as a new version.
https://github.com/Azure-Samples/blockchain/blob/master/blockchain-workbench/CHANGELOG.md

In the readme, at the end of the file the Service Bus is misspelled.

Slide 9 & 23 - Customer objections. First objection starts "There's tons of buzz about..." I'm not sure that phrase will translate to different regions/languages. Please re-phrase.
Slide 11 - Image but no presenter/trainer notes. Is that correct?
Slide 12 - Same as above
Slide 19 - No presenter/trainer notes?

MCW-Azure-Blockchain/Whiteboard design session/WDS student guide - Azure Blockchain.md

Abstract and Learning Objectives section - The text in this section needs an overhaul - Same comment as the Readme.MD comment that I raised. Those observations are applicable here too as the text is the same.

Slides 11 & 22 - images are not flat, no alt-text.
Slide 18 - no alt-text
Slide 12 - alt-text says "Blockchain" not a detailed enough description of the image
Slide 19 - alt-text not detailed enough for image
Accessibility checker comes back with "Missing alt-text, slide 33" There is no slide 33 visible. Please check for hidden slides and remove.

After the lab, task 2, step 4 (Line 518) - image not showing

Line 528 - image not showing

Line 532 - image not showing

#14 updates are ready for QC

From version 1.5.0, there is a Powershell script available to handle majority of the Azure AD creation step.
It is available in the Deployment documentation at the link
https://github.com/Azure-Samples/blockchain/blob/master/blockchain-workbench/CHANGELOG.md#more-flexibility-and-easier-aad-integration
https://docs.microsoft.com/en-us/azure/blockchain/workbench/deploy#azure-ad-configuration-script

Please include these instructions to make the Workbench deployment pre-requisites simpler.

There is a link to Bletchley whitepaper in the infographics section. Is it required? Are the students required to understand and implement any ESC/Cryptlet/Non-Deterministic transactions in their smart contracts? If not, this may be an information overkill and can really confuse the attendees.

Need to rename files WDS, HOL & Before the HOL. Documents mention an ARM template so I don't want to rename if there are links in the labs that refer to file names as is.

Refer to the sentence - "In this Whiteboard design session, you will work with a group to learn how to build and configure an Internet of Things (IoT) Audit Solution using Azure Blockchain"

Observations

1. There is no such thing as Azure Blockchain. Let us rename this to "using Azure blockchain services". This comment applies to the entire document. (wherever the phrase "Azure Blockchain" is used in isolation.)

This refers to the text in the Workshop section of Readme.MD

https://github.com/Microsoft/MCW-Azure-Blockchain#workshop

Observations:

Rename "Ethereum Blockchain" to "Ethereum ledger" or "Ethereum blockchain ledger". The capital B in the Blockchain denotes that it is the name of the product. That is not correct. This applies to everywhere in readme.md where this phrase appears.

Post 1.5.0 release, the Graph API permissions involve providing access to only the user's BASIC profiles, and not the user's FULL profiles.

https://docs.microsoft.com/en-us/azure/blockchain/workbench/deploy#add-graph-api-required-permissions

Depending on the status of the previous observations I raised, the text on the presentation also needs to change.

Also, in the last slide there is a quote from the customer who is very much satisfied with our solution. :) Can we include quotes from one of the suppliers and also the auditor? Blockchain makes everyone happy, doesn't it? 👍

It will be a great added-value to include DevOps as an integral pillar of Blockchain development specifically with the recent announcement DevOps for Blockchain Apps
and DevOps for Blockchain Smart Contracts

This package is scheduled for an update. Please review the contents of this repo and comment on this issue with suggested updates. All suggestions must be added to this issue by EOD Friday, November 9, 2018.

Comment: "You will do this using Ethereum Blockchain with the use of Smart Contracts to collect device telemetry information in addition to enforce contract specifics related to conditions during transport of goods. Specifically, the IoT devices will report temperature and humidity data that will be validated through the Smart Contracts against agreed upon acceptable ranges."

Observation

This whole paragraph in Hands-on Lab section is a verbatim reproduction of the paragraph just before it in Whiteboard design session section. Please reword.

Same observation about the next paragraph as well in Hands on Lab section.

Hi,

Just went through the lab, very intresting, thanks for the effort creating this!

Task 2 --> Section 2 --> Missing ]
Task 2 --> Section 6 --> "AllowedRoles": ["Admin"] --> Due to this role the users can't create a new contract or take action. Propose to add the role "user" to the AllowedRoles

if you create the users with only user permission, you do not have the create contract button. at least one of the users should have admin access.

Design a Blockchain solution section. You have 1 - 4 listed. Design the IoT device and sensor integration there is 1 - 3 listed.

In the preferred solution section, those numbers are not addressed. The answers are text only and it's hard to tell if each of the issues in the student guide are answered in the trainer guide. Can the answers match the formatting of the questions asked?

Enhancement suggestion

The critical component of Telemetry smart contract demos would be showcase the "Out of Compliance" scenario. We have a sample that provides a step-by-step tutorial to integrate IoT Central to the Workbench contract through the Blockchain Development Kit samples.
https://github.com/Azure-Samples/blockchain/tree/master/blockchain-development-kit/connect/iot/iot-central/blockchain-workbench

Please extend the HoL to integrate with IoT or reference this example in the HoL.

Line 425 - image not showing on document

Task 2 "Create App Registration" in Exercise 1 is missing.
Can you please post it?

Thank you.

I did everything as per the hand on lab. But while running deployment everytime I am getting error that App builder and Azure blckcin failed.

Code Error - Deployment Failed.

{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details.","details":[{"code":"BadRequest","message":"{\r\n "error": {\r\n "message": "Subscription is disabled. CorrelationId: c858c147-8e47-4505-b282-7d1f799a9437",\r\n "code": "BadRequest"\r\n }\r\n}"}]}

{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details.","details":[{"code":"Conflict","message":"{\r\n "status": "Failed",\r\n "error": {\r\n "code": "ResourceDeploymentFailure",\r\n "message": "The resource operation completed with terminal provisioning state 'Failed'.",\r\n "details": [\r\n {\r\n "code": "DeploymentFailed",\r\n "message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details.",\r\n "details": [\r\n {\r\n "code": "Conflict",\r\n "message": "{\r\n \"status\": \"Failed\",\r\n \"error\": {\r\n \"code\": \"ResourceDeploymentFailure\",\r\n \"message\": \"The resource operation completed with terminal provisioning state 'Failed'.\",\r\n \"details\": [\r\n {\r\n \"code\": \"DeploymentFailed\",\r\n \"message\": \"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details.\",\r\n \"details\": [\r\n {\r\n \"code\": \"BadRequest\",\r\n \"message\": \"{\\r\\n \\\"error\\\": {\\r\\n \\\"message\\\": \\\"Subscription is disabled. CorrelationId: c858c147-8e47-4505-b282-7d1f799a9437\\\",\\r\\n \\\"code\\\": \\\"BadRequest\\\"\\r\\n }\\r\\n}\"\r\n }\r\n ]\r\n }\r\n ]\r\n }\r\n}"\r\n }\r\n ]\r\n }\r\n ]\r\n }\r\n}"}]}

{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details.","details":[{"code":"Conflict","message":"{\r\n "status": "Failed",\r\n "error": {\r\n "code": "ResourceDeploymentFailure",\r\n "message": "The resource operation completed with terminal provisioning state 'Failed'.",\r\n "details": [\r\n {\r\n "code": "DeploymentFailed",\r\n "message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details.",\r\n "details": [\r\n {\r\n "code": "BadRequest",\r\n "message": "{\r\n \"error\": {\r\n \"message\": \"Subscription is disabled. CorrelationId: c858c147-8e47-4505-b282-7d1f799a9437\",\r\n \"code\": \"BadRequest\"\r\n }\r\n}"\r\n }\r\n ]\r\n }\r\n ]\r\n }\r\n}"}]}

1st customer objection says "There's tons of buzz about Blockchain," Not sure that will translate to other regions/languages. Please re-phrase.

@crpietschmann
Hi Chris,
Please create an Archive folder inside the Hands-on lab folder and place the HOL unguided lab there. Close this issue when complete.
Thanks!
Dawnmarie

Solution architecture - Image not showing.