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

Cobrança Automática (assinatura) #30

Closed
wants to merge 5 commits into from

Conversation

marcelometal
Copy link

Relação com #23

@marcelometal
Copy link
Author

@allisson vc chegou a dar uma olhada nesse PR? Você poderia revisar e apontar os pontos que não estão bons?

@edussilva
Copy link

@marcelometal pelo que eu vi o travis está acusando o seguinte erro: ImportError: No module named 'corsheaders'
Não precisaria incluir esta dependência no projeto?

@marcelometal
Copy link
Author

@edussilva precisa sim; além disso tá com conflito, como ninguém respondeu até hoje sobre como tá o MR, não vou investir mais tempo nele... mas se tiver algum tempo sobrando tento resolver...

@edussilva
Copy link

@marcelometal pois é; eu dei um fork no projeto atual e incluí as suas alterações. Vou criar um PR pra conferir se agora está tudo certo e te referenciar como autor

@marcelometal
Copy link
Author

@edussilva Era so pegar o meu branch e continuar a partir dele, o remote e branch é marcelometal:pre-approvals

@marcelometal marcelometal force-pushed the pre-approvals branch 2 times, most recently from 466ac4c to 183f5dd Compare August 17, 2018 02:29
@marcelometal marcelometal force-pushed the pre-approvals branch 2 times, most recently from fc059d1 to ce225ba Compare August 17, 2018 02:53
Added mock as dev dependency
@marcelometal marcelometal force-pushed the pre-approvals branch 2 times, most recently from 2e53724 to 55994ce Compare August 18, 2018 01:15
@coveralls
Copy link

coveralls commented Aug 18, 2018

Coverage Status

Coverage decreased (-0.2%) to 99.12% when pulling 5b24133 on marcelometal:pre-approvals into d3d7cc1 on allisson:master.

@rafa-acioly
Copy link

Falta alguma coisa (alem dos conflitos) pra seguirmos com esse feature?

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.

None yet

4 participants