Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

Reacting to Routing changes #3861

Closed
wants to merge 2 commits into from
Closed

Reacting to Routing changes #3861

wants to merge 2 commits into from

Conversation

pranavkm
Copy link
Contributor

@pranavkm pranavkm commented Jan 4, 2016

No description provided.

@pranavkm
Copy link
Contributor Author

pranavkm commented Jan 4, 2016

Follow up for aspnet/Routing#273

cc @rynowak

rynowak
rynowak reviewed Jan 5, 2016
View reviewed changes
src/Microsoft.AspNet.Mvc.Core/Routing/UrlHelper.cs
}
else
{
return !pathData.VirtualPath.StartsWith("/", StringComparison.Ordinal)
Copy link
Member

Choose a reason for hiding this comment

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

else if YO

@rynowak
Copy link
Member

rynowak commented Jan 5, 2016

⌚ good cleanup

@pranavkm
Copy link
Contributor Author

pranavkm commented Jan 5, 2016

🆙 📅

@rynowak
Copy link
Member

rynowak commented Jan 5, 2016

:shipit:

@pranavkm pranavkm closed this Jan 5, 2016
@pranavkm pranavkm deleted the prkrishn/virtual-path branch January 5, 2016 20:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pranavkm @rynowak @dnfclas