Replies: 3 comments 6 replies
-
I'm open for adding postgres support for this package. Would #209 still be ok for merging in? Or are other changes needed? |
Beta Was this translation helpful? Give feedback.
-
Fixed by #534 No code changes but test coverage added. |
Beta Was this translation helpful? Give feedback.
-
I wrote that original PR way back when, at the time ~6 years ago there were not a ton of changes really necessary. But if I remember correctly, there was not an interest in maintaining Postgres compatibility at that time because Spatie was not really using it, so it was closed. If that has changed, please do not use my PR, it is minimal and very outdated. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
It's been asked here before but seeing the pricing and flexibility advantages of serverless Postgres on Laravel cloud, and the fork and PR which seem to do most of the needed work to support Postgres, I'd like to bring this up again.
For anyone who doesn't know the docs say you need a database that supports JSON columns (and gives MySQL as an example) but the raw queries in this package aren't compatible with Postgres' JSON implementation.
There's a closed PR from 2019 and a much more promising fork out there.
There's also this blog post breaking down some of these issues and linking to the fork.
Beta Was this translation helpful? Give feedback.
All reactions