search

tpetry / laravel-postgresql-enhanced

Support for many missing PostgreSQL specific features
MIT License
772 stars 31 forks source link

Monetary type #2

Closed vurpa closed 3 years ago

vurpa commented 3 years ago

This pull requests adds the monetary data type.

https://www.postgresql.org/docs/current/datatype-money.html

tpetry commented 3 years ago

Thank your for work and input but i will not merge it. Over the years, the PostgreSQL community has come to the conclusion that the Money data type was not a good idea. You can find the reasons for this in the PostgreSQL wiki: PostgreSQL Wiki: Don't use money

I understand if you don't agree with the decision. But I have already experienced the problems of the money type and therefore support the opinion of the PostgreSQL community.

tpetry commented 3 years ago

I added a note to the readme that in the future opening an issue first to discuss the contribution is very beneficial. Sorry @vurpa for the inconvenience.