search

dwyl / hapi-postgres-connection

:zap: Creates a PostgreSQL Connection (Pool) available anywhere in your Hapi App
GNU General Public License v2.0
40 stars 13 forks source link

PG.connect is deprecated #27

Closed Kuchiriel closed 7 years ago

Kuchiriel commented 7 years ago

it won't normally happen, only if I use babel targeting node 3

When I try to start my server, I see this.

(node:3834) DeprecationWarning: PG.connect is deprecated - please see the upgrade guide at https://node-postgres.com/guides/upgrading

nelsonic commented 7 years ago

@Kuchiriel thanks for reporting this! @naazy do you have time to investigate/update?

Kuchiriel commented 7 years ago

Guys, I'm sorry, it won't normally happen, only if I use babel targeting node 3. I forgot to inform this.

nelsonic commented 7 years ago

@Kuchiriel thanks for the additional info. (those are useful details, and v. specific!) 👍 We still need to upgrade to the latest version in any case because: image https://david-dm.org/dwyl/hapi-postgres-connection But it's unlikely that we are going to "target" node v.3 as it's no longer supported: https://github.com/nodejs/LTS#lts-schedule1

naazy commented 7 years ago

@Kuchiriel this has now been fixed in version 6.2.0. Please upgrade to hapi-postgres-connection 6.2.0 and close the issue if satisfied that this has been resolved :smile:

Kuchiriel commented 7 years ago

@nelsonic @naazy, Thank you very much guys!