Skip to content

Commit 4413d3d

Browse files
committed
Auto merge of #2375 - Turbo87:redundante-prop, r=locks
dashboard: Remove redundant `visibleStats` property and enable ESLint rule that prevents it in the future :) r? @locks
2 parents be1447e + 4b837d2 commit 4413d3d

File tree

3 files changed

+1
-6
lines changed

3 files changed

+1
-6
lines changed

.eslintrc.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ module.exports = {
1919
'ember/no-empty-attrs': 'off',
2020
'ember/no-get': 'off',
2121
'ember/no-mixins': 'off',
22-
'ember/require-computed-macros': 'off',
2322
'ember/require-computed-property-dependencies': 'off',
2423
},
2524
overrides: [

app/controllers/dashboard.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,6 @@ export default Controller.extend({
2626
return this.myFollowing.slice(0, TO_SHOW);
2727
}),
2828

29-
visibleStats: computed('myStats', function () {
30-
return this.myStats;
31-
}),
32-
3329
hasMoreCrates: computed('myCrates.[]', function () {
3430
return this.get('myCrates.length') > TO_SHOW;
3531
}),

app/templates/dashboard.hbs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<div local-class="stats">
77
<div local-class='downloads'>
88
<img src="/assets/download.svg">
9-
<span local-class='num'>{{format-num this.visibleStats.total_downloads}}</span>
9+
<span local-class='num'>{{format-num this.myStats.total_downloads}}</span>
1010
<span local-class="stats-label">Total Downloads</span>
1111
</div>
1212
</div>

0 commit comments

Comments
 (0)