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

[2.2] Converte migrations de insert em seeders #559

Conversation

Projects
None yet
3 participants
@DaniloBenevides
Copy link

commented Apr 20, 2019

Descrição

Converte migrations de insert em seeders conforme #538

Contexto e motivação

Tipos de alterações

  • New feature (Não quebra outras funcionalidades e adiciona funcionalidades)

Checklist:

  • Eu li o documento CONTRIBUTING. [REQUIRED]
  • Meu código segue a PSR2. [REQUIRED]
  • Todos os testes novos e existentes estão passando. [REQUIRED]

@edersoares edersoares changed the title converte migrations de insert em seeders Converte migrations de insert em seeders Apr 22, 2019

@edersoares

This comment has been minimized.

Copy link
Member

commented Apr 22, 2019

A seed database/seeds/DefaultOrgaoEmissorTableSeeder.php está com o nome da classe errado.

Renomear arquivos e classes:

Atual Correto
DefaultDeficienciaTableSeeder DefaultCadastroDeficienciaTableSeeder
DefaultEscolaridadeTableSeeder DefaultCadastroEscolaridadeTableSeeder
DefaultEstadoCivilTableSeeder DefaultCadastroEstadoCivilTableSeeder
DefaultCadastroEstadoCivilTableSeeder DefaultCadastroOrgaoEmissorRgTableSeeder
DefaultRacaTableSeeder DefaultCadastroRacaTableSeeder
DefaultPessoaTableSeeder DefaultCadastroPessoaTableSeeder
DefaultPaisTableSeeder DefaultPublicPaisTableSeeder
DefaultUfTableSeeder DefaultPublicUfTableSeeder
DefaultMunicipioTableSeeder DefaultPublicMunicipioTableSeeder
DefaultDistritoTableSeeder DefaultPublicDistritoTableSeeder
DefaultUrbanoTipoLogradouroTableSeeder DefaultUrbanoTipoLogradouroTableSeeder
DefaultModulesModulesTabelaArredondamentoTableSeeder DefaultModulesTabelaArredondamentoTableSeeder
DefaulModulesTipoVeiculoTableSeeder DefaultModulesTipoVeiculoTableSeeder
@edersoares
Copy link
Member

left a comment

Mudanças solicitadas no comentário anterior.

@vicentimartins

This comment has been minimized.

Copy link

commented May 4, 2019

Mudanças solicitadas no comentário anterior.

@DaniloBenevides você implementou os testes das seeds?!

@DaniloBenevides

This comment has been minimized.

Copy link
Author

commented May 5, 2019

Mudanças solicitadas no comentário anterior.

@DaniloBenevides você implementou os testes das seeds?!

Olá @vicentimartins não implementei não , fiz somente as alterações solicitadas na issue.
Devo estar subindo amanhã as alterações solicitadas

@DaniloBenevides DaniloBenevides force-pushed the DaniloBenevides:converte-migrations-de-inserts-em-seeders branch from e5bee12 to 446d140 May 6, 2019

@DaniloBenevides

This comment has been minimized.

Copy link
Author

commented May 6, 2019

A seed database/seeds/DefaultOrgaoEmissorTableSeeder.php está com o nome da classe errado.

Renomear arquivos e classes:

Atual Correto
DefaultDeficienciaTableSeeder DefaultCadastroDeficienciaTableSeeder
DefaultEscolaridadeTableSeeder DefaultCadastroEscolaridadeTableSeeder
DefaultEstadoCivilTableSeeder DefaultCadastroEstadoCivilTableSeeder
DefaultCadastroEstadoCivilTableSeeder DefaultCadastroOrgaoEmissorRgTableSeeder
DefaultRacaTableSeeder DefaultCadastroRacaTableSeeder
DefaultPessoaTableSeeder DefaultCadastroPessoaTableSeeder
DefaultPaisTableSeeder DefaultPublicPaisTableSeeder
DefaultUfTableSeeder DefaultPublicUfTableSeeder
DefaultMunicipioTableSeeder DefaultPublicMunicipioTableSeeder
DefaultDistritoTableSeeder DefaultPublicDistritoTableSeeder
DefaultUrbanoTipoLogradouroTableSeeder DefaultUrbanoTipoLogradouroTableSeeder
DefaultModulesModulesTabelaArredondamentoTableSeeder DefaultModulesTabelaArredondamentoTableSeeder
DefaulModulesTipoVeiculoTableSeeder DefaultModulesTipoVeiculoTableSeeder

@edersoares acho que terminei, qualquer coisa só avisar

@vicentimartins

This comment has been minimized.

Copy link

commented May 6, 2019

Mudanças solicitadas no comentário anterior.

@DaniloBenevides você implementou os testes das seeds?!

Olá @vicentimartins não implementei não , fiz somente as alterações solicitadas na issue.
Devo estar subindo amanhã as alterações solicitadas

Posso te ajudar com essa implementação. @edersoares ainda dá tempo de implementar esses testes?!

@edersoares

This comment has been minimized.

Copy link
Member

commented May 6, 2019

@vicentimartins claro! Pode utilizar esta branch como base.

Assim que possível verifico se está tudo OK @DaniloBenevides.

Obrigado aos dois <3

@DaniloBenevides

This comment has been minimized.

Copy link
Author

commented May 7, 2019

Mudanças solicitadas no comentário anterior.

@DaniloBenevides você implementou os testes das seeds?!

Olá @vicentimartins não implementei não , fiz somente as alterações solicitadas na issue.
Devo estar subindo amanhã as alterações solicitadas

Posso te ajudar com essa implementação. @edersoares ainda dá tempo de implementar esses testes?!

Que legal ! Vou acompanhar @vicentimartins
Obrigado

@edersoares edersoares merged commit 5295ab3 into portabilis:2.2 May 8, 2019

1 check passed

Travis CI - Pull Request Build Passed
Details

@edersoares edersoares changed the title Converte migrations de insert em seeders [2.2] Converte migrations de insert em seeders May 8, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.