search

artesaos / moip

Pagamentos Moip
https://moip.com.br/
MIT License
38 stars 19 forks source link

O que aconteceu com este merge? (Sobre accounts) #24

Closed lucassena closed 7 years ago

lucassena commented 7 years ago

Olá, eu tinha feito um PR há um tempo que já tinha sido aprovado.

Mas o código referente a ele sumiu do master: https://github.com/artesaos/moip/commit/95869d22fb2b3e147d0e778b1e41985a17122e50

Eu lembro que o código estava no src/Moip.php, o que aconteceu?

Vi que ele foi retirado no commit: https://github.com/artesaos/moip/commit/ebf16d65ceac828098df5697900d9df31fae79dd

Abraços!

SOSTheBlack commented 7 years ago

Olá @lucassena, boa tarde! Do modo que estava o package não estava rodando no Laravel 5.4. Está enfrentando algum problema ?

lucassena commented 7 years ago

Sim, @SOSTheBlack, estou tendo problemas!

Estou precisando acessar o resource Account (referente à Conta Moip via API) da Moip. Por isso tinha feito aquele Pull Request anteriormente.

Por enquanto resolvi com meu Fork, só queria entender por que foi retirado e, talvez corrigir e dar um PR novamente sobre isso.

Sobre o "Criar Conta Moip" na documentação: https://dev.moip.com.br/v2.0/reference#contas-moip

SOSTheBlack commented 7 years ago

@lucassena pode resolver e dar um PR, sem problemas, apenas se certifique que será compativel com o Laravel 5.4

lucassena commented 7 years ago

Ok, vou fazer isso! Valeuu!!!