search

go-bongo / bongo

Go ODM for MongoDB
MIT License
487 stars 40 forks source link

Separated `TimeTracker` #17

Closed ilijamt closed 7 years ago

ilijamt commented 7 years ago

Separated TimeTracker into TimeCreatedTracker and TimeModifiedTracker so if we want we don't need to add Created and Deleted properties in the document.

Now we can implement each field separately and the it's still backward compatible

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.02%) to 84.728% when pulling 49c38a0ee53d4c98bf62a79ffd82fea76105bd2b on ilijamt:separate-time-tracker into c2a77704cea226262fbd909a9595e97e30dad1db on maxwellhealth:master.

securingsincity commented 7 years ago

@ilijamt do you mind rebasing with master so the build can run again?

ilijamt commented 7 years ago

@securingsincity Done.

ilijamt commented 7 years ago

@securingsincity Any reason why they shouldn't be separated?

securingsincity commented 7 years ago

@ilijamt Thanks! I just wanted to take another look at it. It's on master now.