Skip to content
This repository has been archived by the owner on Jan 31, 2023. It is now read-only.

Fixed wrong information in the tutorial #1861

Merged
merged 3 commits into from
Feb 4, 2020

Conversation

KorbinianK
Copy link
Contributor

The tutorial mistakenly writes, that the user should right-click on Smart Contracts > Installed >demoContract@0.0.1. This is not possible, the correct way to upgrade is Smart Contracts > Instantiated >demoContract@0.0.1

Additionally I removed the second 1 that does not belong there.

@Jakeeyturner
Copy link
Contributor

Hi @KorbinianK,

Thanks for creating a PR.
Would you be able to sign-off on your commit?

You can do this by running the following:

git commit --amend --signoff
git push origin patch-1 --force-with-lease

Thanks!

The tutorial mistakenly writes, that the user should right-click on `Smart Contracts` > `Installed` >`demoContract@0.0.1`. This is not possible, the correct way to upgrade is `Smart Contracts` > `Instantiated` >`demoContract@0.0.1`

Additionally I removed the second `1` that does not belong there.

Signed-off-by: KorbinianK <korbiniank@outlook.de>
@KorbinianK
Copy link
Contributor Author

@Jakeeyturner done :)

Copy link
Contributor

@cazfletch cazfletch left a comment

Choose a reason for hiding this comment

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

LGTM

@cazfletch
Copy link
Contributor

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lquintai
Copy link
Contributor

lquintai commented Feb 3, 2020

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@cazfletch
Copy link
Contributor

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sonarcloud
Copy link

sonarcloud bot commented Feb 3, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
14.1% 14.1% Duplication

@Jakeeyturner Jakeeyturner merged commit 811fe30 into IBM-Blockchain:master Feb 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants