can.route references delegate but does not include it #373

Merged
merged 2 commits into from May 6, 2013

Conversation

Projects
None yet
3 participants
@amcdnl
Contributor

amcdnl commented Apr 30, 2013

Fix for can.route references delegate but does not include it

#353

@justinbmeyer

This comment has been minimized.

Show comment
Hide comment
@justinbmeyer

justinbmeyer Apr 30, 2013

Contributor

I don't think the function type check is necessary.

Sent from my iPhone

On Apr 30, 2013, at 11:47 AM, Austin notifications@github.com wrote:

Fix for can.route references delegate but does not include it

#353

You can merge this Pull Request by running

git pull https://github.com/bitovi/canjs issue-353-route-delegate
Or view, comment on, or merge it at:

#373

Commit Summary

can.route references delegate but does not include it - #353
File Changes

M route/route.js (7)
Patch Links:

https://github.com/bitovi/canjs/pull/373.patch
https://github.com/bitovi/canjs/pull/373.diff

Contributor

justinbmeyer commented Apr 30, 2013

I don't think the function type check is necessary.

Sent from my iPhone

On Apr 30, 2013, at 11:47 AM, Austin notifications@github.com wrote:

Fix for can.route references delegate but does not include it

#353

You can merge this Pull Request by running

git pull https://github.com/bitovi/canjs issue-353-route-delegate
Or view, comment on, or merge it at:

#373

Commit Summary

can.route references delegate but does not include it - #353
File Changes

M route/route.js (7)
Patch Links:

https://github.com/bitovi/canjs/pull/373.patch
https://github.com/bitovi/canjs/pull/373.diff

daffl added a commit that referenced this pull request May 6, 2013

Merge pull request #373 from bitovi/issue-353-route-delegate
can.route references delegate but does not include it

@daffl daffl merged commit 9b4107a into master May 6, 2013

1 check passed

default The Travis build passed
Details

@daffl daffl deleted the issue-353-route-delegate branch May 6, 2013

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