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 sql query for create user #497

Closed
wants to merge 1 commit into from
Closed

fix sql query for create user #497

wants to merge 1 commit into from

Conversation

ardani
Copy link

@ardani ardani commented Jan 23, 2018

i try on my computer it doesn't work,
i found the query was generate by code is wrong so i push my fix to make sql user generate work well

i try on my computer it doesn't work, 
i found the query was generate by code is wrong so i push my fix to make sql user generate work well
@lauriejim
Copy link
Contributor

lauriejim commented Jan 23, 2018

The generated request does't work with postgres now.
``` is not recognized by postgres. In fact you just have to make a condition like this line https://github.com/strapi/strapi/pull/497/files#diff-0c0ec60e7cc771f905ecac28620fe70fR286 to apply " if necessary.

@Aurelsicoko
Copy link
Member

The PR (#507) also introduces new changes concerning these lines.

@Aurelsicoko
Copy link
Member

Thank you for this PR, we fixed it in another PR and it has been merged on master.

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