Skip to content

Commit 267567d

Browse files
authored
[VEL-3593/VEL-3588]  Remove unused ember-tether (#397)
* remove unused ember-tether * fix Run loop and computed dot access deprecation * remove comments * feature-flagged: fix property access ambiguous path
1 parent 6fd05fb commit 267567d

File tree

6 files changed

+7
-82
lines changed

6 files changed

+7
-82
lines changed

addon/activity-notifier/service.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { alias, notEmpty } from '@ember/object/computed';
22
import Service, { inject as service } from '@ember/service';
3-
import { run } from '@ember/runloop';
3+
import { cancel, later } from '@ember/runloop';
44
import { observer } from '@ember/object';
55
import { getOwner } from '@ember/application';
66
import Configuration from '@upfluence/ember-upf-utils/configuration';
@@ -61,7 +61,7 @@ export default Service.extend({
6161
this._isRunning = false;
6262

6363
if (this._timer) {
64-
run.cancel(this._timer);
64+
cancel(this._timer);
6565
}
6666
},
6767

@@ -80,7 +80,7 @@ export default Service.extend({
8080
.then((p) => {
8181
this._from = p.next;
8282
this.displayNotifications(p.notifications);
83-
this._timer = run.later(this, this.fetchNotifications, this.waitTime());
83+
this._timer = later(this, this.fetchNotifications, this.waitTime());
8484
})
8585
.finally(() => {
8686
this._inFetch = false;

addon/components/feature-flagged/template.hbs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
{{#if allowedFeature}}
1+
{{#if this.allowedFeature}}
22
{{yield}}
33
{{else}}
44
{{yield to="inverse"}}

addon/mixins/activity-runner.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
import { inject as service } from '@ember/service';
21
import Mixin from '@ember/object/mixin';
3-
import { run } from '@ember/runloop';
2+
import { later } from '@ember/runloop';
3+
import { inject as service } from '@ember/service';
44

55
export default Mixin.create({
66
activityNotifier: service(),
77

88
init() {
99
this._super();
10-
run.later(this, () => this.activityNotifier.start(), 3000);
10+
later(this, () => this.activityNotifier.start(), 3000);
1111
}
1212
});

package.json

-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@
4545
"ember-cli-less": "^2.0.6",
4646
"ember-cli-typescript": "^4.2.1",
4747
"ember-named-blocks-polyfill": "^0.2.4",
48-
"ember-tether": "^1.0.0-beta.2",
4948
"ember-uploader": "^2.x",
5049
"moment": "^2.29.4",
5150
"tinycolor2": "^1.4.1"

pnpm-lock.yaml

-59
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

tests/dummy/config/targets.js

-15
Original file line numberDiff line numberDiff line change
@@ -2,21 +2,6 @@
22

33
const browsers = ['last 1 Chrome versions', 'last 1 Firefox versions', 'last 1 Safari versions'];
44

5-
// Ember's browser support policy is changing, and IE11 support will end in
6-
// v4.0 onwards.
7-
//
8-
// See https://deprecations.emberjs.com/v3.x#toc_3-0-browser-support-policy
9-
//
10-
// If you need IE11 support on a version of Ember that still offers support
11-
// for it, uncomment the code block below.
12-
//
13-
// const isCI = Boolean(process.env.CI);
14-
// const isProduction = process.env.EMBER_ENV === 'production';
15-
//
16-
// if (isCI || isProduction) {
17-
// browsers.push('ie 11');
18-
// }
19-
205
module.exports = {
216
browsers
227
};

0 commit comments

Comments
 (0)