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

Missing migrations for detect_spines app? #6

Closed
mr-nati opened this issue Jun 19, 2022 · 2 comments
Closed

Missing migrations for detect_spines app? #6

mr-nati opened this issue Jun 19, 2022 · 2 comments

Comments

@mr-nati
Copy link

mr-nati commented Jun 19, 2022

Hi, I apologize in advance if I'm doing something silly but I can't seem to run (or find) the migrations for this app.

Running makemigrate and migrate only installs migrations for the admin, auth, contenttypes, sessions packages but none related to the detect_spines app. All of the api calls fail because the database tables don't exist.

Internal Server Error: /api/create-bookshelf/
Traceback (most recent call last):
File "/Users/natitessema/workspace/Bookshelf-Reader-API-master/venv/lib/python3.10/site-packages/django/db/backends/utils.py", line 89, in _execute
return self.cursor.execute(sql, params)
File "/Users/natitessema/workspace/Bookshelf-Reader-API-master/venv/lib/python3.10/site-packages/django/db/backends/sqlite3/base.py", line 477, in execute
return Database.Cursor.execute(self, query, params)
sqlite3.OperationalError: no such table: detect_spines_bookshelf

Is there a missing migrations/ directory for the detect_spines app or am I doing something wrong? Thanks in advance and thanks for making this project @LakshyaKhatri 🙂

@LakshyaKhatri
Copy link
Owner

Thanks mate! 😄
I think you are right. It's been a long time since I worked on this project, just checked the code and I couldn't find any migrations either. Did you find a fix though? From what I see, we need to write migrations manually 🤦🏽‍♂️
I would love a solution to be proposed ❤️

@LakshyaKhatri
Copy link
Owner

Ah! I see. The models defines the fields in django:
https://github.com/LakshyaKhatri/Bookshelf-Reader-API/blob/master/detect_spines/models.py#L94-L105

In that case, I don't think we need any migrations here

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

No branches or pull requests

2 participants