Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

message.date property should be internaldate #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

message.date property should be internaldate #3

wants to merge 2 commits into from

Conversation

rjbs
Copy link
Member

@rjbs rjbs commented Jan 30, 2017 •
edited
Loading

The date property on a message should be the server assigned import/create time, not a header value.

Based on earlier work by Matthew Horsfall.

rjbs added 2 commits January 30, 2017 16:06
@rjbs
Test that sorting on date matches date header
ad2b7f9 
Are we sorting by internaldate?  message.date should be that.

Are we sorting by header?  message.date should be that.

Based on earlier work by Matthew Horsfall.
@rjbs
message.date: it should definitely be internaldate
22eca93
@rjbs rjbs changed the title Test that sorting on date matches date header message.date property should be internaldate Jan 30, 2017
@rjbs rjbs mentioned this pull request Jan 30, 2017
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rjbs