Skip to content

Fix #166 Messages not sorted by date#313

Open
LC43 wants to merge 1 commit into
mailhog:masterfrom
linsomniac:master
Open

Fix #166 Messages not sorted by date#313
LC43 wants to merge 1 commit into
mailhog:masterfrom
linsomniac:master

Conversation

@LC43

@LC43 LC43 commented Sep 15, 2020

Copy link
Copy Markdown

This will fix #166

Tested the commits from @linsomniac, and it works.

@lombad

lombad commented Mar 23, 2021

Copy link
Copy Markdown

If this fix could be merged. That would help me a lot, because we use mailhog for testing of services which are sending a lot of mails. This is a bit boring, since no sorting is supported yet.

@gedge

gedge commented Sep 29, 2021

Copy link
Copy Markdown
Member

can this be a PR into the vendored repo, please? Mailhog-server should get this change first

@gedge gedge added the waiting Waiting for something label Sep 29, 2021
@AlexHimself

Copy link
Copy Markdown

I think I put the fix in manually and compiled it for those who need date sorting like I did.

This is my first time touching GO, GOX, make, etc. I just downloaded the repo, put in the code fix, Googled how the tools worked, smashed lots of buttons, and eventually managed to get it to spit compiled executables out that seem to resolve the sorting in about an hour. I make no guarantees, but so far so good! If it doesn't work for you, I'm not really going to put more time into it, but hopefully it does.

Without email sort, it was a deal breaker for me.

https://github.com/AlexHimself/MailhogDateSortFixed

@afaianswq

Copy link
Copy Markdown

Dear MailHog authors, can you please accept this MR?

aguedoptc pushed a commit to aguedoptc/MailHog that referenced this pull request Sep 14, 2022
@JohnMDavisSCD

Copy link
Copy Markdown

Using on Windows Server 2016 64-bit

git clone https://github.com/mailhog/MailHog.git
cd MailHog/
go mod init
go mode tidy
go get github.com/mailhog/MailHog
go mod vendor
{{insert changes from [linsomniac:master]}}
go build

works great!

@romikoops

Copy link
Copy Markdown

Hey guys, when the PR will be merged? Now it is total mess without proper sorting in Maildir mode

thaomaniac added a commit to thaomaniac/MailHog that referenced this pull request May 29, 2024
thaomaniac added a commit to thaomaniac/MailHog that referenced this pull request May 29, 2024
thaomaniac added a commit to thaomaniac/MailHog that referenced this pull request May 29, 2024
thaomaniac added a commit to thaomaniac/MailHog that referenced this pull request May 29, 2024
ale-rinaldi added a commit to ale-rinaldi/MailHog-Server that referenced this pull request Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting Waiting for something

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Messages not sorted by date

8 participants