Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update mailman.cfg to modify the DB scheme to postgresql:// #49

Closed
wants to merge 1 commit into from

Conversation

e-jim
Copy link

@e-jim e-jim commented Dec 11, 2024

As seen in issue #48 the newer versions of sqlalchemy doesn't use the "postgres://" scheme anymore but expect it to be postgresql:// (see sqlalchemy/sqlalchemy#6083 (comment))

I did not test the fix as a Yunohost upgrade but did change locally my /etc/mailman3/mailman.cfg file.

PR Status

  • Code finished and ready to be reviewed/tested
  • The fix/enhancement were manually tested (if applicable)

As seen in issue YunoHost-Apps#48 the newer versions of sqlalchemy doesn't use the "postgres://" scheme anymore but expect it to be postgresql://
(see sqlalchemy/sqlalchemy#6083 (comment))
This was referenced Dec 31, 2024
Merged
Merged
@yalh76 yalh76 closed this in #51 Jan 2, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant