search

miragejs / discuss

Ask questions, get help, and share what you've built with Mirage
MIT License
2 stars 0 forks source link

Remove Fixtures as a feature #13

Open samselikoff opened 5 years ago

samselikoff commented 5 years ago

Fixtures are just sugar over

server.db.loadData({
  countries: [ ... ]
})

and are confusing when used in conjunction with the default scenario, because of the load order & conditional loading logic based on environment. We should remove them and teach people how to do the same thing using loadData.

samselikoff commented 5 years ago

Alternatively remove automatic loading behavior (which occurs under certain circumstances and is confusing) and just instruct folks to use the existing server.loadFixtures() API themselves, either in the default scenario or in tests.

samselikoff commented 4 years ago

FYI: Transferred this to our Discuss repo, our new home for more open-ended conversations about Mirage!

If things become more concrete + actionable we can create a tracking issue in the main repo.