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

Fix database engine #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

suquant
Copy link

@suquant suquant commented Dec 5, 2017

In case of use different database than MySQL

@fpytloun
Copy link
Member

fpytloun commented Dec 5, 2017

I am not sure this is backward compatible, you should probably use something like:
{% if server.database.engine == 'mysql' %}mysql+pymysql{% else %}{{ server.database.engine }}{% endif %}

@fpytloun fpytloun requested a review from pupapaik December 5, 2017 11:08
@suquant suquant force-pushed the fix-database-engine branch from 628c19e to 2da9423 Compare December 9, 2017 11:23
@suquant suquant force-pushed the fix-database-engine branch from 2da9423 to 965fa11 Compare December 9, 2017 11:31
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.

2 participants