substance / notes

Real-time collaborative notes editing.
Other
20 stars 5 forks source link

Improve db seeding #2

Closed michael closed 8 years ago

michael commented 8 years ago

For development it would be nice if we could do a complete reseed (kill the db and run the seed script) when an option --seed is provided.

Is it possible to do npm start --seed ?

Integral commented 8 years ago

It's already possible in storage branch. Just run npm run seed

On Sat, Feb 13, 2016, 18:35 Michael Aufreiter notifications@github.com wrote:

For development it would be nice if we could do a complete reseed (kill the db and run the seed script) when an option --seed is provided.

Is it possible to do npm start --seed ?

— Reply to this email directly or view it on GitHub https://github.com/substance/notepad/issues/2.

Integral commented 8 years ago

Wait, maybe you should also run prepare... Will do it. Maybe we should provide more data?

On Sat, Feb 13, 2016, 18:59 beilinson@gmail.com beilinson@gmail.com wrote:

It's already possible in storage branch. Just run npm run seed

On Sat, Feb 13, 2016, 18:35 Michael Aufreiter notifications@github.com wrote:

For development it would be nice if we could do a complete reseed (kill the db and run the seed script) when an option --seed is provided.

Is it possible to do npm start --seed ?

— Reply to this email directly or view it on GitHub https://github.com/substance/notepad/issues/2.

michael commented 8 years ago

I know it's possible like that. Just to have it embedded in the app start process would make it more convenient for development. But as a first step if npm run seed also wipes the database that would be good already.

Integral commented 8 years ago

Tell me what you want. Like each time when you start dev mode you want fresh db?

On Sat, Feb 13, 2016, 19:55 Michael Aufreiter notifications@github.com wrote:

I know it's possible like that. Just to have it embedded in the app start process would make it more convenient for development. But as a first step if npm run seed also wipes the database that would be good already.

— Reply to this email directly or view it on GitHub https://github.com/substance/notepad/issues/2#issuecomment-183693888.

michael commented 8 years ago

Each time when i add --seed as a command line option to node server.js / npm start

michael commented 8 years ago

Done.