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

[FAB-17128] CHANNEL_NAME variable missing in two places in channel update tutorial #301

Merged
merged 1 commit into from
Nov 22, 2019

Conversation

maniankara
Copy link

CHANNEL_NAME variable missing in 2 places in channel update tutorial

The channel 'mychannel' is hardcoded in the middle of the channel update tutorial.
This looks like a silly typo, but when working with different channel names this can be a pain.

Signed-off-by: Anoop Vijayan Maniankara anoop@tuxera.com

Type of change

  • Documentation update

Description

The channel 'mychannel' is hardcoded in the middle of the channel update tutorial.
This looks like a silly typo, but when working with different channel names this can be a pain.

…g in 2 places in channel update tutorial

The channel 'mychannel' is hardcoded in the middle of the channel update tutorial.
This looks like a silly typo, but when working with different channel names this can be a pain.

Signed-off-by: anoop <anoop@tuxera.com>
Copy link
Contributor

@yacovm yacovm left a comment

Choose a reason for hiding this comment

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

LGTM

@lindluni
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yacovm
Copy link
Contributor

yacovm commented Nov 22, 2019

/azp run IntegrationTests

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@yacovm
Copy link
Contributor

yacovm commented Nov 22, 2019

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yacovm yacovm merged commit 87f6240 into hyperledger:master Nov 22, 2019
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.

None yet

4 participants