mirror of
https://github.com/status-im/consul.git
synced 2025-01-10 05:45:46 +00:00
03a1a86dfe
* v3.20.2...v3.24.0 * Fix handle undefined outlet in route component * Don't use template helper for optional modal.open Using the optional-helper here will trigger a computation in the same runloop error. This is because we are setting the `modal`-property when the `<Ref>` component gets rendered which will update the `this.modal`-property which will then recompute the `optional`-helper leading to this error. Instead we will create an action that will call the `open`-method on the modal when it is defined. This gets rid of the double computation error as we will not access the modal property twice in the same runloop when `modal` is getting set. * Fix - fn needs to be passed function tab-nav We create functions in the component file instead so that fn-helper stops complaining about the need to pass a function. * Update ember-exam to 6.1 version "Makes it compatible" with ember-qunit v5 * scheduleOnce setMaxHeight paged-collection We need to schedule to get around double-computation error. * Fix - model.data is removed from ember-data This has been private API all along - we need to work around the removal. Reference: https://github.com/emberjs/data/pull/7338/files#diff-9a8746fc5c86fd57e6122f00fef3155f76f0f3003a24b53fb7c4621d95dcd9bfL1310 * Fix `propContains` instead of `deepEqual` policy Recent model.data works differently than iterating attributes. We use `propContains` instead of `deepEqual`. We are only interested in the properties we assert against and match the previous behavior with this change. * Fix `propContains` instead of `deepEqual` token * Better handling single-records repo test-helper `model.data` has been removed we need to handle proxies and model instances differently. * Fix remaining repository tests with propContains We don't want to match entire objects - we don't care about properties we haven't defined in the assertion. * Don't use template helper for optional modal.open Using a template helper will give us a recomputation error - we work around it by creating an explicit action on the component instead. * Await `I $verb the $pageObject object` step * Fix no more customization ember-can No need to customize, the helper handles destruction fine on its own. * Fix - don't pass `optional` functions to fn We will declare the functions on the component instead. This gives us the same behavior but no error from `fn`, which expects a function to be passed. * Fix - handle `undefined` state on validate modifier StateChart can yield out an undefined `state` we need to handle that in the validate modifier * Fix linting errors tests directory * Warn / turn off new ember linting issues We will tackle them one by one and don't want to autofix issues that could be dangerous to auto-fix. * Auto-fix linting issues * More linting configuration * Fix remaining linting issues * Fix linting issues new files after rebase * ui: Remove ember-cli-uglify config now we are using terser (#14574) Co-authored-by: John Cowen <johncowen@users.noreply.github.com>
159 lines
4.7 KiB
JavaScript
159 lines
4.7 KiB
JavaScript
import Mixin from '@ember/object/mixin';
|
|
import { inject as service } from '@ember/service';
|
|
import { set, get } from '@ember/object';
|
|
import { singularize } from 'ember-inflector';
|
|
|
|
/** With Blocking Actions
|
|
* This mixin contains common write actions (Create Update Delete) for routes.
|
|
* It could also be an Route to extend but decoration seems to be more sense right now.
|
|
*
|
|
* Each 'blocking action' (blocking in terms of showing some sort of blocking loader) is
|
|
* wrapped in the functionality to signal that the page should be blocked
|
|
* (currently via the 'feedback' service) as well as some reasonable default hooks for where the page
|
|
* should go when the action has finished.
|
|
*
|
|
* Hooks can and are being overwritten for custom redirects/error handling on a route by route basis.
|
|
*
|
|
* Notifications are part of the injectable 'feedback' service, meaning different ones
|
|
* could be easily swapped in an out as need be in the future.
|
|
*
|
|
*/
|
|
export default Mixin.create({
|
|
_feedback: service('feedback'),
|
|
settings: service('settings'),
|
|
init: function () {
|
|
this._super(...arguments);
|
|
const feedback = this._feedback;
|
|
const route = this;
|
|
set(this, 'feedback', {
|
|
execute: function (cb, type, error) {
|
|
const temp = route.routeName.split('.');
|
|
temp.pop();
|
|
const routeName = singularize(temp.pop());
|
|
|
|
return feedback.execute(cb, type, error, routeName);
|
|
},
|
|
});
|
|
},
|
|
afterCreate: function (item) {
|
|
// do the same as update
|
|
return this.afterUpdate(...arguments);
|
|
},
|
|
afterUpdate: function (item) {
|
|
// e.g. dc.intentions.index
|
|
const parts = this.routeName.split('.');
|
|
// e.g. index or edit
|
|
parts.pop();
|
|
// e.g. dc.intentions, essentially go to the listings page
|
|
return this.transitionTo(parts.join('.'));
|
|
},
|
|
afterDelete: function (item) {
|
|
// e.g. dc.intentions.index
|
|
const parts = this.routeName.split('.');
|
|
// e.g. index or edit
|
|
const page = parts.pop();
|
|
switch (page) {
|
|
case 'index':
|
|
// essentially if I'm on the index page, stay there
|
|
return this.refresh();
|
|
default:
|
|
// e.g. dc.intentions essentially do to the listings page
|
|
return this.transitionTo(parts.join('.'));
|
|
}
|
|
},
|
|
errorCreate: function (type, e) {
|
|
return type;
|
|
},
|
|
errorUpdate: function (type, e) {
|
|
return type;
|
|
},
|
|
errorDelete: function (type, e) {
|
|
return type;
|
|
},
|
|
actions: {
|
|
cancel: function () {
|
|
// do the same as an update, or override
|
|
return this.afterUpdate(...arguments);
|
|
},
|
|
create: function (item) {
|
|
return this.feedback.execute(
|
|
() => {
|
|
return this.repo.persist(item).then((item) => {
|
|
return this.afterCreate(...arguments);
|
|
});
|
|
},
|
|
'create',
|
|
(type, e) => {
|
|
return this.errorCreate(type, e);
|
|
}
|
|
);
|
|
},
|
|
update: function (item) {
|
|
return this.feedback.execute(
|
|
() => {
|
|
return this.repo.persist(item).then(() => {
|
|
return this.afterUpdate(...arguments);
|
|
});
|
|
},
|
|
'update',
|
|
(type, e) => {
|
|
return this.errorUpdate(type, e);
|
|
}
|
|
);
|
|
},
|
|
delete: function (item) {
|
|
return this.feedback.execute(
|
|
() => {
|
|
return this.repo.remove(item).then(() => {
|
|
return this.afterDelete(...arguments);
|
|
});
|
|
},
|
|
'delete',
|
|
(type, e) => {
|
|
return this.errorDelete(type, e);
|
|
}
|
|
);
|
|
},
|
|
use: function (item) {
|
|
return this.repo
|
|
.findBySlug({
|
|
dc: get(item, 'Datacenter'),
|
|
ns: get(item, 'Namespace'),
|
|
partition: get(item, 'Partition'),
|
|
id: get(item, 'AccessorID'),
|
|
})
|
|
.then((item) => {
|
|
return this.settings.persist({
|
|
token: {
|
|
AccessorID: get(item, 'AccessorID'),
|
|
SecretID: get(item, 'SecretID'),
|
|
Namespace: get(item, 'Namespace'),
|
|
Partition: get(item, 'Partition'),
|
|
},
|
|
});
|
|
});
|
|
},
|
|
logout: function (item) {
|
|
return this.settings.delete('token');
|
|
},
|
|
clone: function (item) {
|
|
let cloned;
|
|
return this.feedback.execute(() => {
|
|
return this.repo
|
|
.clone(item)
|
|
.then((item) => {
|
|
cloned = item;
|
|
// cloning is similar to delete in that
|
|
// if you clone from the listing page, stay on the listing page
|
|
// whereas if you clone from another token, take me back to the listing page
|
|
// so I can see it
|
|
return this.afterDelete(...arguments);
|
|
})
|
|
.then(function () {
|
|
return cloned;
|
|
});
|
|
}, 'clone');
|
|
},
|
|
},
|
|
});
|