search

regiov / voluntarios

Website voluntarios.com.br
GNU General Public License v3.0
14 stars 12 forks source link

Campos de cidade e estado alterados para ModelChoiceField, alterações no html e script para atualizar BD. #48

Closed Diogojlq closed 1 year ago

Diogojlq commented 1 year ago

Olá @regiov , acredito que esteja pronto, qualquer coisa me retorna e eu conserto.

Diogojlq commented 1 year ago

Vou criar alguns comentários e limpar algumas coisas no código.

Diogojlq commented 1 year ago

Boa noite @regiov. Fiz esse commit na mesma branch por engano, estou entendendo como corrigir e logo envio um novo pull request com o código para a funcionalidade de favoritar entidades.

regiov commented 1 year ago

Legal, @Diogojlq, eu ia comentar isso mesmo. Penso que o ideal seja, a cada nova funcionalidade que você for implementar, criar um novo branch no seu repositório, trabalhar nele, e quando terminar fazer o pull request a partir dele. Depois que eles forem incorporados aqui pelo repositório principal, você volta para o o seu master/main, atualiza ele, e assim por diante. Por sinal, pretendo sem falta incorporar suas alterações das cidades/estados nesse fim de semana. É que tá bem corrido por aqui ultimamente.

Diogojlq commented 1 year ago

Boa noite @regiov , finalmente consegui entender a coisa toda, fiz um git reset na new_branch, mas antes movi as alterações responsáveis pela funcionalidade de favoritar entidades para uma nova branch, então fiz o um git push -f, apagando o commit que deveria ter feito na outra branch. Cheguei em um ponto onde acredito ser uma questão de preferência, pois não parece haver um padrão de como fazer isso : Devo abrir o PR com as alterações para a funcionalidade de favoritar entidades a partir de uma branch que já contém o código da "new_branch", ou "criar" essas alterações em uma nova branch que foi criada a partir da master, que não contém as alterações da "new_branch" ?

regiov commented 1 year ago

@Diogojlq, quando a segunda issue tem muita intersecção com a issue anterior (ou seja, alterações nas mesmas áreas de código), acho que eu criaria um branch a partir do branch da issue anterior. Do contrário, criaria um branch a partir do master mesmo. Repare que muito disso nem precisaríamos estar nos preocupando se eu tivesse conseguido revisar o primeiro pull request com mais celeridade, mas talvez seja interessante acontecer por aprendizado mesmo. Já fiz pull requests em outros projetos que levaram vários meses para serem analisados. Enfim, coincidentemente acho que hoje de manhã consigo rever novamente o primeiro pull request.