Skip to content

Commit 32cfd31

Browse files
Remove action helper
1 parent 9f7217e commit 32cfd31

3 files changed

Lines changed: 14 additions & 2 deletions

File tree

tests/dummy/app/controllers/index.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import { action } from '@ember/object';
44

55
import { dropTask, timeout } from 'ember-concurrency';
66
import RSVP from 'rsvp';
7+
import { tracked } from '@glimmer/tracking';
78

89
export default class IndexController extends Controller {
910
simple1 = A([
@@ -134,7 +135,7 @@ export default class IndexController extends Controller {
134135

135136
sourceOnly2 = A([{ name: 'Quux' }]);
136137

137-
networkFailure = false;
138+
@tracked networkFailure = false;
138139

139140
@action
140141
dragEnd({ sourceList, sourceIndex, targetList, targetIndex }) {
@@ -200,6 +201,10 @@ export default class IndexController extends Controller {
200201
);
201202
}
202203

204+
updateNetworkFailure = (event) => {
205+
this.networkFailure = event.target.checked;
206+
};
207+
203208
dragEndTask = dropTask(
204209
async ({ sourceList, sourceIndex, targetList, targetIndex }) => {
205210
if (sourceList === targetList && sourceIndex === targetIndex)

tests/dummy/app/templates/index.hbs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@
8989
<input
9090
type="checkbox"
9191
checked={{this.networkFailure}}
92-
onclick={{action (mut this.networkFailure) value="target.checked"}}
92+
{{on "change" this.updateNetworkFailure}}
9393
/>
9494
Simulate network failure.
9595
</label>

tests/dummy/config/ember-try.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ module.exports = async function () {
1111
name: 'ember-lts-3.28',
1212
npm: {
1313
devDependencies: {
14+
'@ember/test-helpers': '^2.9.3',
1415
'ember-cli': '~4.12.0',
1516
'ember-qunit': '^6.0.0',
1617
'ember-resolver': '^10.0.0',
@@ -22,7 +23,10 @@ module.exports = async function () {
2223
name: 'ember-lts-4.4',
2324
npm: {
2425
devDependencies: {
26+
'@ember/test-helpers': '^2.9.3',
2527
'ember-cli': '~4.12.0',
28+
'ember-qunit': '^6.0.0',
29+
'ember-resolver': '^10.0.0',
2630
'ember-source': '~4.4.0',
2731
},
2832
},
@@ -31,7 +35,10 @@ module.exports = async function () {
3135
name: 'ember-lts-4.8',
3236
npm: {
3337
devDependencies: {
38+
'@ember/test-helpers': '^2.9.3',
3439
'ember-cli': '~4.12.0',
40+
'ember-qunit': '^6.0.0',
41+
'ember-resolver': '^10.0.0',
3542
'ember-source': '~4.8.0',
3643
},
3744
},

0 commit comments

Comments
 (0)