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

DecodeURI on dbname from connection string #1

Closed
wants to merge 1 commit into from

Conversation

alexbakerdev
Copy link
Owner

@alexbakerdev alexbakerdev commented Sep 21, 2017

Addresses issue described here sequelize#545

@aledalgrande
Copy link

Did you want to make a PR to the upstream repo instead?

@alexbakerdev
Copy link
Owner Author

I have what is probably a better solution that I will do a proper PR for tomorrow :D after I've written tests ect. 842a428

This branch https://github.com/Xanderite/cli/commits/build/3.0.0 includes the built files if you need to use the package sooner than later, which is what I'm temporarily doing before this gets merged upstream.

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