Skip to content
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

[FEATURE ember-router-willtransition] #10274

Closed
wants to merge 23 commits into from
Closed

Conversation

asakusuma
Copy link
Contributor

Invoke willTransition function and event on Router, just like Route.

Addresses #10235

Invoke willTransition function and event on Router, just like Route.

Addresses emberjs#10235
@chadhietala
Copy link
Contributor

👍 Not sure if this should be feature flagged or not.

@rwjblue
Copy link
Member

rwjblue commented Jan 23, 2015

This adds new API so it will need a feature flag.

@asakusuma
Copy link
Contributor Author

I'll add a feature flag.

@rwjblue while I'm at it, can I address this issue: #10180

I would just need to pass in the handlers that the underlying tildeio/router hook already passes in. Or should I put that in a separate PR?

@asakusuma asakusuma changed the title Add support for router willTransition hook [FEATURE ember-router-willtransition] Jan 23, 2015
@asakusuma
Copy link
Contributor Author

@wagenet why was the "Needs Submitter Response" tag added here? Is there something else I need to provide?

@rwjblue
Copy link
Member

rwjblue commented Jan 23, 2015

He likely didn't see that you updated the PR as requested.

mmun and others added 15 commits January 26, 2015 10:59
Ember.Test.unregisterWaiter is supposed to unregister only the waiter
that matches the arguments you pass it. Due to a bug it actually
unregisters all waiters with the same context.
In prior versions of Ember you could render attributes via:

```
<div {{someHelper}}></div>
```

This brings that back.
* Return value of helper function is not used when using a block.
* Calling `options.fn()` in a compat helper automatically appends
  content.
* tildeio/route-recognizer#41 adds es6 build to
  npm package, released as 0.1.5
* emberjs/emberjs-build#63 updates emberjs-build
  to use npm vendored RR, released as 0.0.22
Also removes a bunch of argument slices in the macros.
A bug in Esperanto makes this not function properly. The bug has been
fixed, but I believe that this still results in a more readable
implementation.
Seems like `eachComputedProperty` and `metaForProperty` are actually static members of Ember.CoreObject. I was confused when `console.log()` of `instanceOfEmObject.eachComputedProperty` returned `undefined`. However I could use it with `Foo.eachComputedProperty()` where `Foo = Em.Object.extend();`. [Demo here](http://emberjs.jsbin.com/powelibeve/4/edit). I hope it will save time and frustration of other developers.
[BUGFIX release] view keyword should not chain directly off parent's view keyword
@asakusuma
Copy link
Contributor Author

Can anyone shed light on why it's failing now? Was passing before I merged bower.json.

Here is the job that is failing: https://travis-ci.org/emberjs/ember.js/jobs/48380622
Here is the same job passing in master: https://travis-ci.org/emberjs/ember.js/jobs/48389900

I don't see any errors besides the Error: Element .does-not-exist not found., which is also present in the passing build.

@asakusuma
Copy link
Contributor Author

@machty mind taking a look?

@machty
Copy link
Contributor

machty commented Jan 26, 2015

@asakusuma regardless of these issues you'll need to rebase your changes on top of master and squash your (presently) 3 commits into one; once you do that, you might find that the issues you're running into magically disappear, so give that a shot first.

rwjblue and others added 3 commits January 26, 2015 17:05
…elper-fix

[FEATURE] ember-testing-checkbox-helpers
Add support for willTransition in Ember.Router

Addresses emberjs#10235
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants