Skip to content
This repository was archived by the owner on Nov 22, 2018. It is now read-only.

Api review changes #42

Closed
wants to merge 2 commits into from
Closed

Api review changes #42

wants to merge 2 commits into from

Conversation

kichalla
Copy link
Member

@Eilon

I will post an announcement once this and reaction changes go in.

@@ -4,7 +4,7 @@
using System;
using System.Threading.Tasks;
using Microsoft.AspNet.Builder;
using Microsoft.AspNet.Cors.Core;
using Microsoft.AspNet.Cors.Infrastructure;
using Microsoft.AspNet.Http;
using Microsoft.Extensions.Primitives;

Copy link
Contributor

Choose a reason for hiding this comment

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

I believe this type is supposed to move to .Infrastructure

Copy link
Member Author

Choose a reason for hiding this comment

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

good catch...you are right...also the CorsPolicyBuilder I think...

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah everything except the two attributes I think. It's in #41.

@Eilon
Copy link
Contributor

Eilon commented Oct 13, 2015

Other than that, :shipit:

@kichalla
Copy link
Member Author

4757712

@kichalla kichalla closed this Oct 15, 2015
@kichalla
Copy link
Member Author

#41

@kichalla kichalla deleted the kiran/cors-api-review branch August 4, 2016 18:53
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.

3 participants