Jay Phelps jayphelps

Developer Program Member

Organizations

@mythologylabs @Netflix @Resflow @ember-cli @graffitijs
jayphelps commented on issue emberjs/ember.js#4751
@jayphelps

That seems like a fair thing to write..? Just like you might write a redirect to add www. On Apr 24, 2015, at 3:29 PM, Garth Poitras notification…

jayphelps commented on issue emberjs/ember.js#4751
jayphelps commented on issue emberjs/ember.js#4751
@jayphelps

@gdub22 it has been too long for me to remember if it's fixable but you're in unsupported waters if you do bypass that assertion because the locati…

jayphelps commented on issue emberjs/ember.js#4751
@jayphelps

@gdub22 it has been too long for me to remember if it's fixable but you're in unsupported waters if you do bypass that assertion because the locati…

@jayphelps
jayphelps commented on pull request blesh/RxEmber#7
@jayphelps

@blesh get in here

jayphelps commented on pull request ember-cli/ember-cli#3909
@jayphelps

@abuiles I totally agree there are diff benefits, this argument can be applied to many things we don't do. Off the top of my head, parameters: func…

@jayphelps
  • @jayphelps 0aa2cdc
    scheduleEmberAction() needs an Ember.run when auto-run is off
jayphelps closed pull request blesh/RxEmber#7
@jayphelps
scheduleEmberAction() needs an Ember.run when auto-run is off
jayphelps commented on pull request blesh/RxEmber#7
@jayphelps

@machty yep, you're right. Dammit @blesh!! :imp:

jayphelps opened pull request blesh/RxEmber#7
@jayphelps
scheduleEmberAction() needs an Ember.run when auto-run is off
1 commit with 7 additions and 5 deletions
@jayphelps
  • @jayphelps 1880cf4
    scheduleEmberAction() needs an Ember.run when auto-run is off
jayphelps commented on pull request ember-cli/ember-cli#3909
@jayphelps

@stefanpenner we should have a limit, i dont want 10 import long statements. either 2 or 3 should likely be the limit. I agree. I personally hav…

@jayphelps
@jayphelps

@blesh technically, #44 is a duplicate of this. :yum:

@jayphelps
injection vs. inject?
@jayphelps
Remove MutationObserver dependency to support < IE 11
@jayphelps
phatomjs -> phantomjs spelling correction
1 commit with 1 addition and 1 deletion
@jayphelps
@jayphelps

@blesh ah okay. If this isn't impacting nf-graph, feel free to reclose

@jayphelps

Reopening until upstream bug is fixed.

@jayphelps
Changing selected doesn't change CSS selected class