allisson / django-pagseguro2

Integração da API v2 do PagSeguro com o Django.
MIT License
95 stars 44 forks source link

Cobrança Automática (assinatura) #30

Closed marcelometal closed 4 years ago

marcelometal commented 7 years ago

Relação com #23

marcelometal commented 6 years ago

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

edussilva commented 6 years ago

@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 commented 6 years ago

@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 commented 6 years ago

@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 commented 6 years ago

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

coveralls commented 6 years ago

Coverage Status

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

rafa-acioly commented 4 years ago

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