router.js icon indicating copy to clipboard operation
router.js copied to clipboard

Consider Transition object being a promise sublcass

Open stefanpenner opened this issue 11 years ago • 5 comments

The Transition object already mimics a thenable via then but lacks catch and finally etc, why not just make it a subclass

https://github.com/tildeio/router.js/blob/master/lib/router/transition.js

stefanpenner avatar Apr 18 '14 22:04 stefanpenner

Resolved in https://github.com/tildeio/router.js/commit/83f33bad29472ec03a973117e23c52b241fa1440

jamesplease avatar Aug 10 '15 00:08 jamesplease

@jmeas no, that doesn't resolve it. That merely creates a promise like entity that decorates a rail promise, it does not create a subclass.

stefanpenner avatar Aug 10 '15 00:08 stefanpenner

Ah, gotcha :+1:

jamesplease avatar Aug 10 '15 00:08 jamesplease

I'm doing some issues-gardening 🍀 🌷 and stumbled upon this. Is this still an issue? if not, perhaps it can be closed?

sandstrom avatar Nov 05 '16 11:11 sandstrom

Yes, it is still an issue.

rwjblue avatar Nov 05 '16 12:11 rwjblue