From 8903c490b4255fb59fe0224c97f794bf72a7de26 Mon Sep 17 00:00:00 2001 From: Walter Alves Date: Tue, 8 Aug 2017 16:27:35 -0300 Subject: [PATCH 1/4] Add bidirecional call/sms send --- src/cli.js | 31 ++++++++++++++++++++++++++----- 1 file changed, 26 insertions(+), 5 deletions(-) diff --git a/src/cli.js b/src/cli.js index 0ecb479..8da4042 100644 --- a/src/cli.js +++ b/src/cli.js @@ -9,11 +9,28 @@ const emitSuccess = message => console.log(green(` ✔ Sucesso: ${message}`)); const emitError = message => console.log(red(` ✗ Erro: ${message}`)); function cli(args) { - gemidao(args) - .then(() => { - emitSuccess(args.sms ? 'sms enviado!' : 'chamada efetuada!'); - }) - .catch(pipe(prop('message'), emitError)); + if(args.bidirecional){ + send(args); + var temp = args.de; + args.de = args.para; + args.para = temp; + send(args); + }else{ + send(args); + } + +} + +function bidirecional(args){ + +} + +function send(arg){ + gemidao(arg) + .then(() => { + emitSuccess(arg.sms ? 'sms enviado!' : 'chamada efetuada!'); + }) + .catch(pipe(prop('message'), emitError)); } cli(yargs @@ -34,6 +51,10 @@ cli(yargs describe: 'Se definido, será enviado um SMS ao invés de uma chamada', type: 'boolean' }) + .option('bidirecional', { + describe: 'Defini se a chamada deve ser enviada para ambos os números', + type: 'boolean' + }) .demandOption(['para', 'token']) .locale('pt_BR') .strict() From 6fde12abe23d315e37ded8e65cce0ced9bdc0ec9 Mon Sep 17 00:00:00 2001 From: Walter Alves Date: Tue, 8 Aug 2017 17:05:46 -0300 Subject: [PATCH 2/4] Fixes flag description --- src/cli.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cli.js b/src/cli.js index 8da4042..6ef80f4 100644 --- a/src/cli.js +++ b/src/cli.js @@ -52,7 +52,7 @@ cli(yargs type: 'boolean' }) .option('bidirecional', { - describe: 'Defini se a chamada deve ser enviada para ambos os números', + describe: 'Se definido, realiza uma nova ligação, desta vez com o de/para invertidos', type: 'boolean' }) .demandOption(['para', 'token']) From 3e1856cdf646af2d68d298a1fafc57251ec68144 Mon Sep 17 00:00:00 2001 From: Walter Alves Date: Tue, 8 Aug 2017 17:11:52 -0300 Subject: [PATCH 3/4] Fixes coding style --- src/cli.js | 25 ++++++++++--------------- 1 file changed, 10 insertions(+), 15 deletions(-) diff --git a/src/cli.js b/src/cli.js index 6ef80f4..1619846 100644 --- a/src/cli.js +++ b/src/cli.js @@ -9,24 +9,19 @@ const emitSuccess = message => console.log(green(` ✔ Sucesso: ${message}`)); const emitError = message => console.log(red(` ✗ Erro: ${message}`)); function cli(args) { - if(args.bidirecional){ - send(args); - var temp = args.de; - args.de = args.para; - args.para = temp; - send(args); - }else{ - send(args); + if (args.bidirecional) { + send(args); + const temp = args.de; + args.de = args.para; + args.para = temp; + send(args); + } else { + send(args); } - -} - -function bidirecional(args){ - } -function send(arg){ - gemidao(arg) +function send(arg) { + gemidao(arg) .then(() => { emitSuccess(arg.sms ? 'sms enviado!' : 'chamada efetuada!'); }) From 0d308bacf1dc7148c3ae586beb318039723a40eb Mon Sep 17 00:00:00 2001 From: Walter Alves Date: Wed, 9 Aug 2017 13:52:00 -0300 Subject: [PATCH 4/4] Use promises to send bidirectional calls --- src/cli.js | 16 ++-------------- src/gemidao.js | 12 +++++++++--- 2 files changed, 11 insertions(+), 17 deletions(-) diff --git a/src/cli.js b/src/cli.js index 1619846..53540f4 100644 --- a/src/cli.js +++ b/src/cli.js @@ -9,21 +9,9 @@ const emitSuccess = message => console.log(green(` ✔ Sucesso: ${message}`)); const emitError = message => console.log(red(` ✗ Erro: ${message}`)); function cli(args) { - if (args.bidirecional) { - send(args); - const temp = args.de; - args.de = args.para; - args.para = temp; - send(args); - } else { - send(args); - } -} - -function send(arg) { - gemidao(arg) + gemidao(args) .then(() => { - emitSuccess(arg.sms ? 'sms enviado!' : 'chamada efetuada!'); + emitSuccess(args.sms ? 'sms enviado!' : 'chamada efetuada!'); }) .catch(pipe(prop('message'), emitError)); } diff --git a/src/gemidao.js b/src/gemidao.js index c884548..961a707 100644 --- a/src/gemidao.js +++ b/src/gemidao.js @@ -16,7 +16,10 @@ const sms = (to, token) => request.post(route('/sms')) const call = (from, to, token) => request.post(route('/composto')) .set('Access-Token', token) .set('Accept', 'application/json') - .send({ + .send(msg_data(from, to)); + +function msg_data(from, to) { + return { numero_destino: to, dados: [ { @@ -27,7 +30,8 @@ const call = (from, to, token) => request.post(route('/composto')) } ], bina: from - }); + }; +} export default function gemidao(args) { if (!/^[a-f0-9]{32}$/.test(args.token)) { @@ -40,7 +44,9 @@ export default function gemidao(args) { const action = args.sms ? sms(args.para, args.token) - : call(args.de, args.para, args.token); + : (args.bidirecional + ? call(args.de, args.para, args.token) + : call(args.de, args.para, args.token).send(msg_data(args.para, args.de))); return action .catch(err => {