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

Show non-MvcRoute route types #783

Merged
merged 1 commit into from Apr 25, 2014

Conversation

Projects
None yet
3 participants
@bsimanov
Contributor

bsimanov commented Apr 25, 2014

Display route type instead of Glimpse.AspNet.Model.RouteModel when showing non-MvcRoutes

glimpsefix

Show non-MvcRoute route types
Display route type instead of Glimpse.AspNet.Model.RouteModel when showing non-MvcRoutes
@avanderhoorn

This comment has been minimized.

Show comment
Hide comment
@avanderhoorn

avanderhoorn Apr 25, 2014

Member

@PaulAtkins You cool with this change based on the work you have done?

Member

avanderhoorn commented Apr 25, 2014

@PaulAtkins You cool with this change based on the work you have done?

@avanderhoorn

This comment has been minimized.

Show comment
Hide comment
@avanderhoorn

avanderhoorn Apr 25, 2014

Member

@bsimanov p.s. meant to mention, looks great and a good addition!

Member

avanderhoorn commented Apr 25, 2014

@bsimanov p.s. meant to mention, looks great and a good addition!

@PaulAtkins

This comment has been minimized.

Show comment
Hide comment
@PaulAtkins

PaulAtkins Apr 25, 2014

Contributor

@avanderhoorn @bsimanov Yep looks good to me!!

Contributor

PaulAtkins commented Apr 25, 2014

@avanderhoorn @bsimanov Yep looks good to me!!

avanderhoorn added a commit that referenced this pull request Apr 25, 2014

Merge pull request #783 from bsimanov/master
Show non-MvcRoute route types

@avanderhoorn avanderhoorn merged commit c6f7b1b into Glimpse:master Apr 25, 2014

@avanderhoorn avanderhoorn added this to the vNext milestone Apr 25, 2014

@avanderhoorn avanderhoorn self-assigned this Apr 25, 2014

@avanderhoorn

This comment has been minimized.

Show comment
Hide comment
@avanderhoorn

avanderhoorn Apr 25, 2014

Member

Thanks @bsimanov for the update/fix!

Member

avanderhoorn commented Apr 25, 2014

Thanks @bsimanov for the update/fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment