search

jamesmills / laravel-timezone

Enable user Timezones in your application.
MIT License
677 stars 91 forks source link

Support Money as well ? (Enhancement) #38

Closed dgillier closed 4 years ago

dgillier commented 4 years ago

Hello,

Any chance that you will support Money as well (as geoIP is also supporting it) ? Thanks, Denis

jamesmills commented 4 years ago

Hi Denis,

I think as the package is focused on Timezones then we shouldn't look at adding other things into it.

You could hook into the underlying geoIP package to save this info on the user if you wanted to.

I find money is a rather complex thing so we could store the users local currency on the user model but I wouldn't want to get into trying to then show the user money in their local currency. I would suggest looking at something like https://github.com/cknow/laravel-money for this.

Thanks,

James