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

#13 Melhoria das instruções de instalação #14

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

Conversation

douglas-queiroz
Copy link

No description provided.

@vitorbaptista
Copy link
Owner

@douglas-queiroz Obrigado pelo pull request, Douglas 👍

Ele melhora as instruções, mas falta ainda uma parte importante: carregar os seeds do hackathon.sql em https://github.com/vitorbaptista/escola-que-queremos/tree/master/db/seeds. Idealmente, o https://github.com/vitorbaptista/escola-que-queremos/blob/master/db/seeds.rb seria alterado pra bastar executar rake db:seed para carregar esse SQL.

Faz tempo que não programo em Rails, então não lembro qual a melhor forma de fazer isso. Se você tiver um tempo, poderia fazer essa alteração?

Valeu!

@douglas-queiroz
Copy link
Author

Ops! Desculpa, não tinha visto o sql.
Posso sim! Hoje a noite crio o seed.rb, altero a readme e comito blz?
=)

ps: Só não faço agora pq to no trampo e aqui é windows. =/

@vitorbaptista
Copy link
Owner

Tranquilo! 👍

@douglas-queiroz
Copy link
Author

@vitorbaptista estava dando uma analisada no hackathon.sql, e vi que ele tem muitas coisas além do seeds, teremos que migrar as definições de tabela pra as migrations antes de colocar os inserts no arquivo seeds. Estou fazendo aqui aos poucos e te envio para você analisar. blz?
Boa noite!

@vitorbaptista
Copy link
Owner

Pois é, pelo que me lembro o hackathon.sql é um dump do banco. Se ficar complicado extrair pra migrations, acho de boa simplesmente carregar o dump 👍

@douglas-queiroz
Copy link
Author

@vitorbaptista to fazendo um script pra gerar o seed pelos inserts, acho q essa semana eu te mostro algo. Blz?

@vitorbaptista
Copy link
Owner

@douglas-queiroz Considerando que o SQL tem 29 MB, acho que usar um dump é a melhor opção. Chuto que passar isso pro Rails vai tornar o carregamento dos seeds muito lento.

A melhor saída me parece:

  1. Carrega o hackathon.sql no BD (considerando que a criação das tabelas está no .sql);
  2. Roda as migrations.

Então só precisaria alterar as instruções nesse pull request.

P.S.: Encontrei uma gem que faz o que você tá querendo: https://github.com/rroblak/seed_dump

@douglas-queiroz
Copy link
Author

Seria isso @vitorbaptista
P.S.:Obrigado pela dica! =)

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.

2 participants