timothepearce
Added configurable date column
Change the date field for the time-series, use another field in your Model instead of created_at
Hi All,
Great package and very useful ! I added the capability to choose a different field instead of the standard created_at field. While it works well (I just updated the Tests Models and they all pass) I switched the pull request to draft because I'd like to have the field modifiable at the projection level. The way it is now each Model can only have one date field while I want to achieve flexibility to be able to store statistics on the same model based on different date_fields on the same Model. For Example :
- in Model Reservation for Events, I have 3 important date I want statistics on :
- 1 The reservation date
- 2 the payment date
- 3 the actual event's date
Any ideas you can share on how to achieve this ?
Done, The functionality is now working and by default works as before. All existing tests are passing, and will need to add test for the new functionality. Your Comments are mostly welcome.
The Pull request is now fully operational including Test and Readme file ! I could not run the package with queue job enabled, I found out that the issue was with the SerializesModel Trait and fixed it in this pull request. Thanks !
@salhdev Hi Salhdev, sorry for not answering faster, I'll review your PR when I have time next week :)
