-
Notifications
You must be signed in to change notification settings - Fork 936
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
Add fixtures to catalog #93
Comments
@calvh Thanks you. Fixtures certainly look like a useful feature (which I did not know about). But I'm not sure where they'd be useful in the context of this tutorial. Did you have any particular case in mind? |
I think they'd be useful somewhere around Section 4 where you add data via the admin site. Some people (like me) find it tedious to manually enter data and may want an alternative. It might be worth having two sets of fixtures:
Am I allowed to work on this or is this project restricted to members only? |
You are absolutely allowed to work on this and that would be much appreciated. I think probably it should be added only in one place, and that should probably be either at the end of, or "below", https://developer.mozilla.org/en-US/docs/Learn/Server-side/Django/Models (this might require modification of the admin section too). First, the updates should be made and tested as the demo project would be at the end of the models chapter. My concern here is that the models are not "complete" here (there are "challenges") so we need to be clear that whatever we advise works at that point. Then further we should have an update as a PR to https://github.com/mdn/django-locallibrary-tutorial and to the docs. Does that make sense? |
Yes, thank you. I will draft some changes and put them here before I make a PR. |
No, thank you - I'm looking forward to learning something new :-) |
Closing as no action taken |
I suggest adding some mock data into a fixtures folder
https://docs.djangoproject.com/en/3.2/howto/initial-data/
The text was updated successfully, but these errors were encountered: