search

Flowframe / laravel-trend

Generate trends for your models. Easily generate charts or reports.
MIT License
671 stars 70 forks source link

Raw date #8

Closed ju5t closed 1 year ago

ju5t commented 2 years ago

The package uses {$this->getSqlDate()} as date, in its query builder. We have some models with a date column already so we get a SQL error when using this package.

Can we either make this configurable or use a different default? I don't think it's a breaking change as the public data is in TrendValue but if I'm wrong just let me know :)

haheap commented 2 years ago

I encountered the same issue.

I'd be glad to do a PR to make it configurable - and possibly with a different default to make it even less likely for someone to encounter the issue in the first place.