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: prisma2 introspect for mysql database #683

Merged
merged 1 commit into from
Oct 18, 2019

Conversation

wardpeet
Copy link
Contributor

@wardpeet wardpeet commented Oct 7, 2019

When using prisma2 init I can create a connection with my mysql connection and get a prisma.schema generated. However when I run prisma2 introspect it gives me Please provide a database name error. Schema is not necessary for mysql but it is for Postgres so I updated the statements to support MySQL as well.

@CLAassistant
Copy link

CLAassistant commented Oct 7, 2019

CLA assistant check
All committers have signed the CLA.

@janpio
Copy link
Member

janpio commented Oct 7, 2019

Ha, database vs. schema already caused bugs in Prisma 1 ;)

@pantharshit00
Copy link
Contributor

Thanks for the PR!

This bug was not reported but I can confirm it.
image

@janpio janpio requested review from divyenduz and removed request for timsuchanek October 11, 2019 15:53
@janpio janpio added this to the Preview 15 milestone Oct 11, 2019
@janpio
Copy link
Member

janpio commented Oct 11, 2019

@divyenduz will look into both the bug and the possible fix via this PR to check if this solves the problem.

@wardpeet
Copy link
Contributor Author

Do you want me to post a before/after gif?

@divyenduz
Copy link
Contributor

@wardpeet That would be great thanks 🙌

@janpio janpio added bug/2-confirmed Bug has been reproduced and confirmed. kind/bug A reported bug. labels Oct 16, 2019
Copy link
Contributor

@divyenduz divyenduz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@timsuchanek timsuchanek merged commit 388d7bb into prisma:master Oct 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/2-confirmed Bug has been reproduced and confirmed. kind/bug A reported bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants