-
Notifications
You must be signed in to change notification settings - Fork 46
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
Implementing Sub Accounts Management (Follow #4) #29
Conversation
…h authorization token
Acredito que precise ainda da refatoração que o @paezao comentou #4 (comment) onde sugere que deveria mudar os métodos verify e configuration para métodos de instância. O que acham? Parece mais elegante que a solução de deixar tudo como métodos de classe. O problema é que esses métodos não retornam o mesmo JSON de retorno do fetch na API. |
Outra situação é os testes dos métodos transfer e withdraw pois para testá-los é necessário uma sub conta com saldo. Seria possível implementarem esses specs? |
Revi aqui e acredito que essa integração precisa ir além. Pois precisa permitir que as outras chamadas como Iugu::Invoices sejam utilizadas com o token da Iugu::Account. Somente assim seria de valia. Estou desenvolvendo essa integração para o sistema que estou construindo nesse branch. Espero que futuramente seja integrado. Mas está WIP por enquanto |
Esse PR ficou estaganado? |
Esse pull request continua o PR #4 dado que ele ficou estagnado.