gemidao-do-zap icon indicating copy to clipboard operation
gemidao-do-zap copied to clipboard

Add bidirectional call/sms send

Open walteraa opened this issue 8 years ago • 6 comments

Adiciona a flag --bidirecional para fazer chamada para ambos os números. gemidao-do-zap --de=x --para=y --token=t --bidirecional

Resolve a issue #16

walteraa avatar Aug 08 '17 19:08 walteraa

Parece uma feature legal! Só precisamos adaptar o coding-style:

  • A função bidirecional não tem corpo e não tá sendo usada
  • Rode um npm run lint para ver o que é necessário para adaptar ao coding-style

Obrigado! :heart:

haskellcamargo avatar Aug 08 '17 19:08 haskellcamargo

Vou testar a branch na prática a branch ao chegar em casa para poder mesclar! :heart: Obrigado!

haskellcamargo avatar Aug 08 '17 20:08 haskellcamargo

@haskellcamargo quando eu chegar em casa vou dar uma melhorada, passar a usar promises. Mas se quiser ir aceitando o PR já pode, posso abrir outro em breve.

walteraa avatar Aug 08 '17 21:08 walteraa

Não gostou da minha ideia? :cry:

cuchi avatar Aug 09 '17 18:08 cuchi

Sim @cuchi, mas desacoplar os dados e usar promises "seems better"!

walteraa avatar Aug 09 '17 18:08 walteraa

Puts @walteraa !

AndreyMenezes avatar Aug 09 '17 20:08 AndreyMenezes