-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
CharField to TextField #923
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vkosuri Thank you, these changes look good. The only thing that needs to be updated is that Django is going to need migrations for this change.
@gunthercox I try to make migrations on django_chatterbot it was saying no changes found. I tried like this. Any tips how do migration on django_chatterbot? (env)vkosuri@vm-vkosuri:~/ChatterBot$ python examples/django_app/manage.py migrate django_chatterbot
Operations to perform:
Apply all migrations: django_chatterbot
Running migrations:
No migrations to apply.
(env)vkosuri@vm-vkosuri:~/ChatterBot$ python examples/django_app/manage.py makemigrations django_chatterbot
No changes detected in app 'django_chatterbot' |
I'll look into this and see if I can recreate the issue. I'll let you know what I have to do to get them to generate. |
This PR will solve this problem gunthercox/chatterbot-corpus#36 |
@gunthercox Thanks fix this issue, could you please let me know how to generate django_chatterbot migrations? |
These are the steps. They work but I wish I had easier ones.
|
Closes #921
Reference: https://docs.djangoproject.com/en/1.10/ref/models/fields/#django.db.models.CharField