cypht icon indicating copy to clipboard operation
cypht copied to clipboard

Make cypht search shows latest results when searching in a mail box

Open mmarcwabo opened this issue 1 year ago • 6 comments

Pullrequest

Make cypht search show latest results

Make default sort mode on search by internal date from the newest to the latest

Issues

  • [X] None

Checklist

  • [X] None

How2Test

  • [X] Make a search from any page and notice that found messages at the top are the recent ones

Todo

  • [ ] Make sure sort by date choosen by the user works as well

mmarcwabo avatar Sep 13 '23 17:09 mmarcwabo

Shouldn't we sort by the default field user is seeing messages sorted? Might be internal date, might be date of arrival.

kroky avatar Sep 15 '23 12:09 kroky

Shouldn't we sort by the default field user is seeing messages sorted? Might be internal date, might be date of arrival

I have found that internal_date and date are the same. The goal here were to make sure by default newers messages are on top. I am still working on this PR to make sure that sort follows users choices / preferences (Ascending or Descending).

mmarcwabo avatar Sep 16 '23 00:09 mmarcwabo

There are certainly 2 different dates that user can choose for default sort order in imap module - Date of the msg (date sent) and date of arrival (which is probably the internal date). I think we should be in sync here with the setting.

kroky avatar Sep 19 '23 06:09 kroky

@mmarcwabo Please take this over the finish line :-)

marclaporte avatar Dec 15 '23 01:12 marclaporte

Just adding my 2¢, Cypht might have an opportunity to do truly smart searches... by doing a "relevance" search and sorting with the most relevant results at the top.

I haven't yet used Cypht in the real world, so this is just a thought off the top of my head.

TheDigitalOrchard avatar Apr 16 '24 03:04 TheDigitalOrchard

Just adding my 2¢, Cypht might have an opportunity to do truly smart searches... by doing a "relevance" search and sorting with the most relevant results at the top.

ok, let's track here: https://github.com/cypht-org/cypht/issues/959

marclaporte avatar Apr 16 '24 11:04 marclaporte

Closing this in favor of https://github.com/cypht-org/cypht/pull/1120 - turns out we needed both front and backend fixes to make sorting work in imap folders.

kroky avatar Jul 15 '24 16:07 kroky