Skip to content
This repository has been archived by the owner on Oct 24, 2024. It is now read-only.

Date type instead of string for datetime fields #21

Open
ppaskaris opened this issue May 12, 2017 · 2 comments
Open

Date type instead of string for datetime fields #21

ppaskaris opened this issue May 12, 2017 · 2 comments

Comments

@ppaskaris
Copy link

Greetings,

I really like the simplicity of this package, but I was wondering: is there a particular reason that this uses strings for the deleted and visible fields?

Using a date field would allow for adding a TTL Index to the deleted field, enabling the database to automatically clean up old messages.

@TomKaltz
Copy link

+1

1 similar comment
@ianitsky
Copy link

ianitsky commented Oct 6, 2020

+1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants