Skip to content
This repository has been archived by the owner on Sep 30, 2022. It is now read-only.

Change sqlalchemy version #102

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Change sqlalchemy version #102

wants to merge 1 commit into from

Conversation

FayasNoushad
Copy link

For avoiding sqlalchemy.exc.NoSuchModuleError: Can't load plugin: sqlalchemy.dialects:postgres

For avoiding `sqlalchemy.exc.NoSuchModuleError: Can't load plugin: sqlalchemy.dialects:postgres`
@jk6521
Copy link

jk6521 commented Dec 15, 2021

To avoid that you should use postgresql:// instead of postgres://

@FayasNoushad
Copy link
Author

Ok

@FayasNoushad
Copy link
Author

Ok

@FayasNoushad FayasNoushad reopened this Dec 15, 2021
@PaulSonOfLars
Copy link
Owner

The purpose of this PR is very unclear.

Why does pinning the version fix this? What changed in newer sqlalchemy versions? What is the actual issue?

@jk6521
Copy link

jk6521 commented Apr 5, 2022

The purpose of this PR is very unclear.

Why does pinning the version fix this? What changed in newer sqlalchemy versions? What is the actual issue?

In older version of sqlalchemy it accepted both postgres:// and postgresql:// but in the newer version they stopped support for postgres:// in database URI. The error he mentioned is happening mostly on heroku since the DATABASE_URL from heroku-postgres can't be edited and they haven't added support for this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants