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

Ordem dos parâmetros na documentação incorreta #31

Open
vjoao opened this issue Jun 13, 2018 · 2 comments
Open

Ordem dos parâmetros na documentação incorreta #31

vjoao opened this issue Jun 13, 2018 · 2 comments
Assignees

Comments

@vjoao
Copy link

vjoao commented Jun 13, 2018

Para assinaturas, a documentação manda fazer:
https://dev.moip.com.br/v1.5/reference#criar-assinatura

moip.subscription.create('assinatura01',{ 
    new_customer: false,
    amount: 9990,
    payment_method: "CREDIT_CARD",
    plan : {
        code : 'plan01'
    },
    customer : {
        code : 'cliente01'
  }  
})

Só que esta chamada está incorreta. A chamada está sendo feita usando o primeiro parâmetro como payload, o que é errado. Também não há nenhuma forma de passar o parâmetro new_customer do jeito que está implementado, já que, segundo a documentação, este parametro deve ser passado diretamente na URL, nos forçando a usar o config da chamada.

Só consegui chamar a API assim:

moip.subscription.create({ 
    code: 'assinatura1',
    payment_method: "CREDIT_CARD",
    plan : {
        code : 'plan01'
    },
    customer
  }
}, { customUrl: 'https://sandbox.moip.com.br/assinaturas/v1/subscriptions?new_customer=true'  })

o que é extremamente inconveniente. Sei que vocês estão em processo migratório na API de assinaturas, mas não dá pra introduzir breaking changes assim e deixar a documentação a desejar.

@Igor-Lopes Igor-Lopes self-assigned this Jun 14, 2018
@Igor-Lopes
Copy link
Contributor

Igor-Lopes commented Oct 26, 2018

@vjoao Consultando uma branch antiga minha, não houve alteração deste parâmetro na API v1.5, ele sempre foi na url, isso na verdade é um erro desta funcionalidade na SDK,que passou após alterações, mas deveria permitir o envio deste parâmetro sem alteração desta configuração na chamada. De qualquer modo, a forma como você exemplificou já é um workaround enquanto isso não é corrigido.

@somentelucas
Copy link
Contributor

Além disso, retirei o parâmetro new_customer: false da documentação, realmente a informação desse parâmetro no body da request não é correta. Valeu pelo toque, @vjoao!

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

No branches or pull requests

3 participants