Add bidirectional call/sms send

haskellcamargo / gemidao-do-zap

Uma aplicação de linha de comando que liga para alguém tocando o gemidão do Zap

MIT License

1.99k stars 188 forks source link

Add bidirectional call/sms send #18

Open walteraa opened 7 years ago

walteraa commented 7 years ago

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

haskellcamargo commented 7 years ago

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 commented 7 years ago

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

walteraa commented 7 years ago

@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.

cuchi commented 7 years ago

Não gostou da minha ideia? :cry:

walteraa commented 7 years ago

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

AndreyMenezes commented 7 years ago

Puts @walteraa !