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

minor clean up #4431

Merged
merged 1 commit into from
Sep 27, 2017
Merged

minor clean up #4431

merged 1 commit into from
Sep 27, 2017

Conversation

Rick-Anderson
Copy link
Contributor

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.

  • Suggest reviewers if you know who should review the PR, using the GitHub Reviewers UI.

@Rick-Anderson Rick-Anderson merged commit 9274682 into master Sep 27, 2017
@Rick-Anderson Rick-Anderson deleted the 4/ra/test2 branch September 27, 2017 23:57
Rick-Anderson added a commit that referenced this pull request Sep 29, 2017
* Add PO localization doc

* More edits

* Updates TaskCache to Task

Since [TaskCache](dotnet/extensions#238) was removed this updates the samples to use `Task.CompletedTask` instead

* More edits

* More edits

* Change code snippet type from C# to CSHTML

* Add missing snippet types

* Add Required note to model validation topic

Fixes #1928

* MVC views overview update

Updates

Updates

Updates

Updates

Update

* React to feedback

* React to feedback

* Add support disclaimer

* UE pass on Publishing to IIS topic

* React to feedback

Minor update

* Move VS Tools for Docker doc to ASP.NET Core repo

* React to feedback

* React to feedback

* Drop literal_block comments and fix a few code blocks

* Fix demo code for AddAuthentication.AddCookie

In .Net Core 2.0 using the code as supplied results in an exception, the fix is to pass the scheme name when calling AddCookie.

Fails:
services.AddAuthentication("MyCookieAuthenticationScheme")
        .AddCookie

Works:
Fix demo code for services.AddAuthentication.AddCookie
services.AddAuthentication("MyCookieAuthenticationScheme")
        .AddCookie("MyCookieAuthenticationScheme",

* React to feedback

* Fix one

* React to feedback

* Convert title to sentence case

* Add missing param to AddCookie call

* React to feedback

* React to feedback

* Add RP tutorial step 'Uploading files' to TOC

* fix breadcrumb link to core

* Update model-binding.md

Fixing type `CancelationToken` -> `CancellationToken`

* minor clean up (#4431)

* Updates

* Fix a link

* Explain post delete method

Update

* Fix inadvertent uid change (typo)

* React to feedback

* React to feedback

* React to feedback

* Alter app cookie configuration code samples

* Update first-web-api-mac.md (#4441)

* Update web-api-vsc.md

* Remove dead project file (#4427)

* Updates to the intro: layouts, partials, VCs

* Update web-publishing-vs.md (#4447)

* Update web-publishing-vs.md

* Update web-publishing-vs.md

* Update identity-enable-qrcodes.md

* Update identity-enable-qrcodes.md

* Update identity-enable-qrcodes.md

* Remove extra space

* Minor edits

* Add TOC link to snapshot debugging doc

* Update index.md (#4443)

* Update index.md

* Update index.md
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