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

Updated models.py to support MYSQL JSON Field #31

Merged
merged 3 commits into from
Jun 12, 2020

Conversation

patrickscottbest
Copy link
Contributor

Need eventlog to work with MySQL.

@KatherineMichel KatherineMichel merged commit 5a779ce into pinax:master Jun 12, 2020
@KatherineMichel
Copy link
Member

@patrickscottbest Awesome, thank you!

@blag
Copy link

blag commented Jul 23, 2020

This PR changed models.py, but didn't update the database migrations: #32.

@KatherineMichel
Copy link
Member

@patrickscottbest Is this something you might be able to submit a PR to fix?

@patrickscottbest
Copy link
Contributor Author

patrickscottbest commented Jul 27, 2020 via email

@patrickscottbest
Copy link
Contributor Author

patrickscottbest commented Jul 29, 2020

I've commented on this in the bug, we seem to have a code divergence problem since migration 0003_auto_20160111_0208.py.

See #32

In other words, pinax-eventlog may be locked into postgresql with the inclusion of this file in the overall package. Take it out, and now you've bonked the installed userbase with bad migrations

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

Successfully merging this pull request may close these issues.

None yet

3 participants