-
Notifications
You must be signed in to change notification settings - Fork 82
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
After choosing Postgres as the production DB the production type in orm.config.ts is set as postgresql and gives an error #70
Comments
Thanks! I have fixed that... could you try with the bugfix branch here? |
I tried to install the specific branch using:
npm install -g
https://github.com/jhipster/generator-jhipster-nodejs/tree/bug/postegres
which was complaining about a missing package.json. Any ideas ?
…On Sat, Oct 26, 2019 at 11:49 PM Angelo Manganiello < ***@***.***> wrote:
Thanks! I have fixed that... could you try with the bugfix branch here
<https://github.com/jhipster/generator-jhipster-nodejs/tree/bug/postegres>
?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#70?email_source=notifications&email_token=AA3NFK2BZW3JC44YISLOIFTQQQ7XZA5CNFSM4JFMCQIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECKHYAA#issuecomment-546601984>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA3NFKZFT6CRMDD23Q4K24LQQQ7XZANCNFSM4JFMCQIA>
.
|
Hi @robconnell9 ,
These are the steps written in the CONTRIBUTING GUIDE. As it is not the first time that are not clear (or read) these steps in that guide, I will explain that in the README.md. |
I followed the steps and this fixes the problem that I raised |
Good! thanks for the collaboration! You are welcome to contribute with a donation or with the development of new features, and introduce new people in this project/community :) |
Describe the bug
When there is a prompt for the production database and postgresql is chosen this populates orm.config.ts with the prod database type of postgresql at line 28. This value should be postgres as per the value in PostgresConnectioOptions.d.ts
To Reproduce
Steps to reproduce the behavior:
jhipster --blueprint nodejs
choose monolithic
choose postgresql for production database
Expected behavior
value in orm.config.ts should be postgres.
Screenshots
not required
Desktop (please complete the following information):
IOS
Chrome
current as per 26/10/19
Smartphone (please complete the following information):
NA
Additional context
Supplied as required
The text was updated successfully, but these errors were encountered: