I was reading the WebAPI 2 tutorial and found a mistake in this sample. The GetBookDetail action gets a parameter called id, and in the query, this id is checked against the AuthorId, which I think is wrong, it should be checked against the BookId property.
Hello, @Rick-Anderson,
Localization team has reported source content issue that causes localized version to have broken format compared to en-us version.
From the context of this source string, we can assume that writer did not intend this example URL to be rendered as hyperlink, and that text "App Namespace" within brackets {} is expected to be translated.
We found out that OPS is using a Markdig flavor called "Markdig (DocFX)" which does render URL surrounded by space to be rendered as hyperlink.
To avoid this, suggested fix to source string is add an escape char (\) to part of the URL.
Please review and merge the proposed file change to fix to target versions. If you make related fix in another PR then share your PR number so we can confirm and close this PR.
Many thanks in advance.
Hello, @wpickett,
Localization team has reported source content issue that causes localized version to have broken format compared to en-us version.
Please review and merge the proposed file change to fix to target versions. If you make related fix in another PR then share your PR number so we can confirm and close this PR.
Many thanks in advance.
When creating a new PR, please do the following and delete this template text:
* Reference the issue number if there is one:
Fixes #Issue_Number
> The "Fixes #nnn" syntax in the PR description causes
> GitHub to automatically close the issue when this PR is merged.
Hello, @scottaddie,
Localization team has reported source content issue that causes localized version to have broken format compared to en-us version.
Please review and merge the proposed file change to fix to target versions. If you make related fix in another PR then share your PR number so we can confirm and close this PR.
Many thanks in advance.
When creating a new PR, please do the following and delete this template text:
* Reference the issue number if there is one:
Fixes #Issue_Number
> The "Fixes #nnn" syntax in the PR description causes
> GitHub to automatically close the issue when this PR is merged.
Hello, @jrjlee,
Localization team has reported source content issue that causes localized version to have broken format compared to en-us version.
Please review and merge the proposed file change to fix to target versions. If you make related fix in another PR then share your PR number so we can confirm and close this PR.
Many thanks in advance.
* Updated examining-the-details-and-delete-methods.md
examining-the-details-and-delete-methods.md has an extra '.' and the formatting for back tics '`' was incorrect
* Adjust backticks
* Update create-the-project.md
Four months ago, in January, someone asked where the Models folder is at and the answer was to change the authentication. I assume the instructions should have that information and apparently it has been missing since the beginning. For VS 2017 I think the "Create Project" button is now an "OK" button but I don't know if my update about that is relevant.
* Verbiage tweaks
* Update azure-apps-preview-notice.md (#5857)
* Link to .NET Core CLI publish command doc (#5864)
* Link to .NET Core CLI publish command doc
* Verbiage tweak
* fix typo (#5866)
* Update intro.md (#5869)
`classnameID` (where `classname` is the name of the class, such as `Student`) as the primary key. In `classnameID`, `classname` is the name of the class, such as `Student` in
* Update view-components.md (#5872)
altering range of code displayed to remove hanging closing } and closing table tag
* C13511: Spaces should be replaced to avoid localization problems (#5871)
Hello, @rick-anderson,
Localization team has reported source content issue that causes localized version to have broken format compared to en-us version.
Please, help to check my proposed file change into the article and help to merge if you agree with fix. If not, please, let me know either if you would like me to fix it in another way within this PR, if you prefer to fix it in another PR or if I should close this PR as by-design. In case of using another PR, please, let me know of your PR number, so we can confirm and close this PR.
Many thanks in advance.
* CI Update (#5878)
* Recommended language change (#5883)
* Updated with NET Core SDK 2.0 commands (#5886)
* Updated with NET Core SDK 2.0 commands
* Quick UE pass
* Prevent javascript injection. (#5885)
* Troubleshoot SDK warnings (#5880)
* WIP Troubleshoot SDK warnings
* Work
* work
* Address Gauntlet warning in ms.date value
* Adjust H2 title casing and spacing
* Quick UE pass
* Update Configuration doc to use double underscores instead of colons for hierarchical config keys (4968) (#5876)
* changes to configuration doc
* fixes for scott
* change for Dan
* Update index.md
* Code fence double underscore and add missing period
* Eliminate unnecessary sentence
* Missing npm task runner extension (#5893)
* Revert "Missing npm task runner extension (#5893)" (#5894)
This reverts commit a90faebff2.
* Note about task runner explorer option (#5895)
* Note about task runner explorer option
The gulp file has to be in the root directory or in the directory with package.json not in any subdirectory.
* verbiage tweaks
* added anchored link (#5896)
Hello, @rick-anderson,
Localization team has reported source content issue that causes localized version to have broken format compared to en-us version.
Please, help to check my proposed file change into the article and help to merge if you agree with fix. If not, please, let me know either if you would like me to fix it in another way within this PR, if you prefer to fix it in another PR or if I should close this PR as by-design. In case of using another PR, please, let me know of your PR number, so we can confirm and close this PR.
Many thanks in advance.
Stephen Griffiths
ChrisSfanos
petyka
Caro Caserio
thearunster
Jeremie LOSCOS
Luke Latham
thmslearning
Antoine Griffard
Swanand Pangam
Jaydeep Karena
Alfred Myers
Dustan Vodvarka
Rick Anderson
SimpleSamples
任跃兵
Drakeor
markospowellos
hmmwhatsthisdo
Rachel Appel
jasonruz
Zhiliang Xu
yishengjin1413
Scott Addie
Tieson Trowbridge
joe-bolton
Amadeusz Sadowski
Tom Dykstra
Caio Melo
Timmy Kokke
Philip Stratford