Database url hint should include database name

gissilabs / charts

Apache License 2.0

38 stars 22 forks source link

Database url hint should include database name #38

Closed AikedeJongste closed 1 year ago

AikedeJongste commented 1 year ago

Database url hint should include database name. Because otherwise it will silently fall back to sqlite. I saw someone created a PR to fix the unintended fallback to sqlite. But this is just the config hint. I hope it will save someone some time figuring out why their database url is not working.

sgissi commented 1 year ago

Hi @AikedeJongste, thank you for the PR and sorry for the delay to review it. I'll merge this and also update the README to reflect the hint. Thank you again!