Chris Ng

Results 9 comments of Chris Ng

Data point here is we renamed the link: `https://guides.emberjs.com/release/testing/testing-application/` vs `https://guides.emberjs.com/v2.14.0/testing/acceptance/`

Dug into this a bit. Looks like the `redirect` is being cached when doing browser back, I'm not super familiar with the `guidemaker` but its not in the payload (http://localhost:4200/content/v2.14.0/testing/acceptance.json)...

Doing a dump of links: FrontendMasters: https://frontendmasters.com/courses/ember-octane/ Weekly Ember ecosystem updates: https://emberweekly.com/ Build Ambitious Apps with Confidence: https://balinterdi.com/rock-and-roll-with-emberjs/ Videos on EmberMap: https://embermap.com/

@ljharb thanks for reviewing! We've seen that, when branching code due to AB tests, this can happen quite a lot due to copy paste and not noticing an `if` clause...

@ljharb Sorry will open an issue next time! Basically something like this (this is example code with errors): ```jsx Username: {emailIsEnabled ? ( Email: ) : null} {passwordIsEnabled ? (...

@NullVoxPopuli We may just want to then update the rule `no-arguments-for-html-elements` which would incorrectly assume a lowercase angle bracket component is not an Ember component but an HTML element. See:...

I believe the project ships a babel-ified version of the code so it should work in IE11 The class syntax is just for development

Hey @jimmyangel ! Unfortunately I no longer work at Priceline, left almost 2 years ago :) I think this PR is good to go tho ... 2 years ago haha

Yeah for sure we need a battery of unit tests. Especially for IE and mobile compatibility.