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) #36

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

edussilva
Copy link

De acordo com a issue #23

Refatoração do código do @marcelometal de acordo com a nova versão do projeto

@coveralls
Copy link

coveralls commented Aug 16, 2018

Coverage Status

Coverage decreased (-0.2%) to 99.138% when pulling f4f0356 on edussilva:preapprovals into d3d7cc1 on allisson:master.

@marcelometal
Copy link

@edussilva refatorar sem citar meu nome, sem meus commits não é muito de boa, porque você fica como autor de tudo que eu escrevi. Você poderia ter feito novos commits preservando os meus, você pode por favor mante-los?

@edussilva
Copy link
Author

@marcelometal posso sim, mas eu não tenho muita experiência com o github. Eu pensei em fazer a partir do seu, mas achei que daria mais trabalho. Você tem alguma sugestão de como podemos fazer? Se achar melhor você pode copiar a minha branch e enviar o PR, pra mim não tem problemas... só gostaria que a lib avançasse o quanto antes.

@marcelometal
Copy link

@edussilva

  1. Você pode adicionar o meu repositório como remote:
git remote add marcelometal git@github.com:marcelometal/django-pagseguro2.git
  1. mudar para minha branch:
git checkout marcelometal/pre-approvals -b pre-approvals
  1. Assim você salva suas modificações nessa branch, pode fazer um cherry-pick, rebase, merge, etc.

  2. Apagar sua branch antiga preapprovals

git branch -D preapprovals
  1. Renomear a branch pre-approvals para preapprovals
git branch -M preapprovals
  1. Enviar para o github suas modificações, é necessário fazer com -f para sobreescrever sem precisar fechar o PR que vc já enviou
git push origin preapprovals -f

Qualquer dúvida só perguntar...

@edussilva
Copy link
Author

@marcelometal Obrigado pela ajuda! Eu tentei seguir os passos e tive dificuldades no passo 3.. a versão da sua branch é anterior à master do projeto e não consegui manter o código atualizado no merge. Se for mais simples você atualizar o seu PR podemos fazer assim, eu cancelo o meu sem problemas

@marcelometal
Copy link

marcelometal commented Aug 17, 2018

@edussilva para atualizar o meu branch com a master, basta fazer um rebase, vou adicionar alguns passos aos existentes... Vai ter conflitos, basta resolve-los.

3.1) adicione o remote

git remote add allisson git@github.com:allisson/django-pagseguro2.git

3.2) faça o rebase

git rebase allisson/master

3.3) Faça/acrescente as suas modificações....

@marcelometal
Copy link

@edussilva Acabei fazendo o rebase para você, meu branch agora está com o topo da master, você pode seguir dele sem problemas.

Qualquer dúvida só perguntar...

@edussilva
Copy link
Author

@marcelometal acho que agora deu tudo certo! Se ficou faltando alguma coisa é só me avisar
Muito obrigado pela orientação e desculpe o incômodo

@marcelometal
Copy link

@edussilva tranx! mas ainda não tá de boa, tem um commit 4ae9f9f que não deveria ser seu... vou tentar fazer pra você essa junção dos códigos.

@marcelometal
Copy link

marcelometal commented Aug 18, 2018

@edussilva peguei todas as suas modificações e apliquei no meu branch, da uma olhada #30. Valeu por ter corrigido o que quebrou com as ultimas atualizações da master.

Qualquer coisa é só falar

@edussilva
Copy link
Author

@marcelometal perfeito! mais uma vez obrigado

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

3 participants