Skip to content

corrected the localhost port #389

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

Merged
merged 1 commit into from
Aug 6, 2020
Merged

corrected the localhost port #389

merged 1 commit into from
Aug 6, 2020

Conversation

krish-gh
Copy link
Contributor

@krish-gh krish-gh commented Aug 6, 2020

Purpose

The example port 44316 in the readme is not aligned with the web app port 44321 mentioned in the rest of the doc. Corrected it.

Does this introduce a breaking change?

[ ] Yes
[x ] No

Pull Request Type

What kind of change does this Pull Request introduce?
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x ] Documentation content changes
[ ] Other... Please describe:

How to Test

N/A. Doc update.

What to Check

N/A. Doc update.

Other Information

Copy link
Contributor

@jmprieur jmprieur left a comment

Choose a reason for hiding this comment

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

LGTM.
Thanks @krish-gh

@jmprieur jmprieur merged commit a88cde3 into Azure-Samples:master Aug 6, 2020
@krish-gh krish-gh deleted the patch-1 branch August 6, 2020 16:27
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.

2 participants