7c203a9220
* Create PopoverSelect component and styling * Create CatalogToolbar component and Styling * ui: Adds `selectable-key-values` helper (#7472) Preferably we want all copy/text to live in the template. Whilst you can achieve what we've done here with a combination of different helpers, as we will be using this approach in various places it's probably best to make a helper. We also hit an ember bug related to using the `let` helper and trying to access `thingThatWasLet.firstObject` (which can also be worked around using `object-at`). Moving everything to a helper 'sorted' everything. Probably worthwhile noting that if the sort option themselves become dynamic, I'm not sure if the helper here would actually react as you would expect (I'm aware that ember helpers on react on the root arguments, not necesarily sub properties of those arguments). If we get to that point this helper could take the same approach as what I believe ember-composable-helpers does to get around this, or move them to the view controller. If we do ever moved this to the view controller, we can still use the exported function from the new helper here to keep using the same functionality and tests we have here. * Create tests for sorting services with CatalogToolbar * Add rule to print 'ember/no-global-jquery' as a warning Co-authored-by: John Cowen <johncowen@users.noreply.github.com>
54 lines
1.3 KiB
JavaScript
54 lines
1.3 KiB
JavaScript
module.exports = {
|
|
root: true,
|
|
parser: 'babel-eslint',
|
|
parserOptions: {
|
|
ecmaVersion: 2018,
|
|
sourceType: 'module',
|
|
ecmaFeatures: {
|
|
legacyDecorators: true
|
|
}
|
|
},
|
|
plugins: ['ember'],
|
|
extends: ['eslint:recommended', 'plugin:ember/recommended'],
|
|
env: {
|
|
browser: true,
|
|
},
|
|
rules: {
|
|
'no-unused-vars': ['error', { args: 'none' }],
|
|
'ember/no-new-mixins': ['warn'],
|
|
'ember/no-jquery': 'warn',
|
|
'ember/no-global-jquery': 'warn'
|
|
},
|
|
overrides: [
|
|
// node files
|
|
{
|
|
files: [
|
|
'.eslintrc.js',
|
|
'.dev.eslintrc.js',
|
|
'.template-lintrc.js',
|
|
'ember-cli-build.js',
|
|
'testem.js',
|
|
'blueprints/*/index.js',
|
|
'config/**/*.js',
|
|
'lib/*/index.js',
|
|
'server/**/*.js'
|
|
],
|
|
parserOptions: {
|
|
sourceType: 'script'
|
|
},
|
|
env: {
|
|
browser: false,
|
|
node: true
|
|
},
|
|
plugins: ['node'],
|
|
rules: Object.assign({}, require('eslint-plugin-node').configs.recommended.rules, {
|
|
// add your custom rules and overrides for node files here
|
|
|
|
// this can be removed once the following is fixed
|
|
// https://github.com/mysticatea/eslint-plugin-node/issues/77
|
|
'node/no-unpublished-require': 'off'
|
|
})
|
|
}
|
|
]
|
|
};
|