4791-refactor-sendsms #1160
No reviewers
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#1160
Loading…
Reference in New Issue
No description provided.
Delete Branch "4791-refactor-sendsms"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -0,0 +2,4 @@
import './style.scss';
export default class sendSmsDialog {
constructor($element, $scope, $http, $translate, vnApp) {
Porqué no extiende a dialog?
@ -0,0 +10,4 @@
this.vnApp = vnApp;
}
open(route) {
En vez de pasar la ruta, emitir un evento
send
al cual te conectas desde donde se use el componente y envias el sms.@ -0,0 +34,4 @@
this.vnApp.showMessage(this.$t.instant('SMS sent!'));
});
} catch (e) {
this.vnApp.showError(this.$t.instant(e.message));
Solo se debería traducir los
UserError
@ -0,0 +41,4 @@
}
}
sendSmsDialog.$inject = ['$element', '$scope', '$http', '$translate', 'vnApp'];
Los servicios
'$http', '$translate', 'vnApp'
no es necesario inyectarlos, si se extiende la claseDialog
, que hereda deComponent
, ya vienen "de serie".@ -118,2 +116,2 @@
sms="$ctrl.newSMS">
</vn-client-sms>
<vn-sms-dialog
vn-id="sms"
Falta tabulación.
@ -0,0 +32,4 @@
if (this.charactersRemaining() < 0)
throw new Error(`The message it's too long`);
this.$http.post(this.route, this.sms).then(res => {
En lugar de llamar a una ruta, emitir un evento
send
, si tienes dudas me dices y lo hablamos.@ -0,0 +43,4 @@
}
}
ngModule.vnComponent('vnSmsDialog', {
Definir este componente en salix en lugar de el core.
@ -0,0 +11,4 @@
this: this
}
});
document.addEventListener('openSmsDialog', e => {
Açò haviem quedat que ja no feia falta, no?