Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suporte ao DirectCall #25

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

iagocaldeira
Copy link

@iagocaldeira iagocaldeira commented Aug 9, 2017

De acordo com o PR #11 criada pelo @luigifreitas, referente à issue #6
Refatorei o código e estou fazendo esse PR para alguem continuar o trabalho a partir deste ponto onde já é possivel testar e fazer requisições para a API da DirectCall.
A maior dificuldade agora está justamente no fluxo da API deles e na não aceitação do meu Token dado como não informado.
Além disso essa chamada de API link somente cadastra o audio. Para fazer o envio da chamada é usada este método link com os parâmetros Origem, Destino e audio_id

Copy link
Collaborator

@williamokano williamokano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vai na fé

@@ -10,7 +10,8 @@ const emitError = message => console.log(red(` ✗ Erro: ${message}`));

function cli(args) {
gemidao(args)
.then(() => {
.then(res => {
console.log(res);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

De repente tirar esse console.log? rsrs

Morgan ou winston seria uma boa, não?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Como falei, esse código não estava pronto, o PR foi mais de fix do anterior. Deixei o log mais para deixar claro onde pegar o resultado da requisição. Além disso para um possivel PR ter mensagens de erro de acordo com a resposta da API

@almeidamarcell
Copy link

Então ainda não temos uma alternativa ao total voice?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants