Pedrodias

[pedroHdias]

• Alteracoes na tabela Course (criacao de migracao)
• Criacao e de Controller/Model/Views para Person
• Criacao e de Controller/Model/Views para PersonType
• Criacao e de Controller/Model/Views para Subject
• Criação das rotas para os novos modelos
• Adição de associações entre modelos existentes (Person/PersonType/Subject/Course)

[renatopanda]

Olá @pedroHdias. Não querendo fazer o trabalho do @nelsonmpg, era só para avisar que devem ter alguma atenção aos avisos quando fazem pull requests para garantir menos trabalho a quem vai fazer o merge. Neste caso, o teu branch tem 2 conflitos que me parecem fáceis de resolver (no routes.rb e no schema.rb). Acontece porque o teu branch (o teu código) está 2 commits atrasado em relação ao master e esses commits entram em conflito com os teus commits novos (daí os 2 conflitos). O ideal é, antes de fazer pull request, ter o nosso branch sincronizado com o master (merge do master para o nosso).

[nelsonmpg]

Pedro Dias em relação a este Pull Request podes verificar:

• Nos modelos está Person e na migração está People para a criaçção da tabela.
• Onde está "lenght" deve de estar "length", porque está a dar erro a validar 'LenghtValidator'.

Obrigado.

[pedroHdias]

Boa tarde prof @renatopanda, Eu por acaso fiz "update from master" antes de começar a trabalhar, não sei porque deram esses erros. E em relação ao conflito que o @nelsonmpg comentou, sobre Persons - People, eu acho que o ruby, quando criei o model Person, criou a tabela People (people é o plural de person), e como estava bastante atento a isso criei o controller Persons, e penso que venha daí o problema. Já fiz as alteração de controller Persons para controller People. E obrigado pelos comentários.

[renatopanda]

@pedroHdias de nada. As revisões servem mesmo para isto, corrigir erros e evitar que vão para o master. 😄

O que dizes está correcto sobre os nomes. Person é o singular de People e rails sabe disso e funciona assim. Portanto, se seguires as convenções:

• modelo: Person (person.rb)
• tabela: people
• controlador: people_controller.rb
• vistas em app/views/people/*

Seguindo o que o @nelsonmpg assinalou, tens que fazer uma migração para renomear o campo length também.

[pedroHdias]

Já está feito prof @renatopanda. (e também editar routes.db) E também testado com seeds. Só estou a espera agora que o @nelsonmpg cancele este pull request para fazer um novo com tudo corrigido.

[renatopanda]

Já agora sobre como poderá ter acontecido isso, é fácil, imagina:

• na 2a feira fizeste um novo branch a partir do master
• na 5a o @PedroMatos1972 acabou as alterações dele e fez pull request
• na 5a o @nelsonmpg aceitou o pull resquest do @PedroMatos1972, estes commits alteraram o richeiro schema.rb (p. ex).
• na 6a acabaste as tuas alterações fizeste pull request.

Conclusão, o teu commit indica como alterar um ficheiro (schema.rb versão 2a) para outro (schema.rb tua versão 6a), mas o ficheiro que está no master já não é esse, é o schema.rb de 5a. Logo ele avisa e pede para resolver o possível conflito. O ideal é fazer antes do pull request um merge do master para ti.

[renatopanda]

@pedroHdias Não precisas de cancelar, ele pode aceitar este agora que já corrigiste. É assim que funciona ;)

[pedroHdias]

Entendido @renatopanda. Ah, eu pensei de que, como fiz alterações depois de ter feito pull request, essas alterações tinham de ser enviadas com outro pull request. Então até o @nelsonmpg aceitar o pull request, o que eu alterar segue também nesse pull request?

(Esqueça, já vi aqui que os commits passam para o pull request 😄)

[pedroHdias]

@renatopanda confesso que não me preocupei muito com as views porque iria mexer mais nessa parte quando tivesse ordens do @nelsonmpg para poder já integrar o novo design do site nestas páginas. Mas peço desculpa pelos erros. (e isto de person/people deu me volta à cabeça)