user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
From: "Eric Wong (Contractor, The Linux Foundation)" <e@80x24.org>
To: meta@public-inbox.org
Subject: [PATCH 00/11] duplicate support in UI + tests
Date: Tue, 27 Mar 2018 11:11:21 +0000	[thread overview]
Message-ID: <20180327111132.20681-1-e@80x24.org> (raw)

Threading messages sharing the same Message-ID requires
using our "preferred" Message-ID.

For the Atom feed and new.html, this requires some effort to
fake SearchMsg results for v1 where Xapian was optional, but
should allow us to handle corner cases where legit messages
share a Message-ID.

I'm not sure if messages with the same Message-ID should
be rendered as relatives, or if they should only be grouped
together in the same rootset (as if their "Subject:" matches)
but not rendered as ancestors.

Eric Wong (Contractor, The Linux Foundation) (11):
  import: consolidate mid prepend logic, here
  www: $MESSAGE_ID/raw endpoint supports "duplicates"
  search: reopen DB if each_smsg_by_mid fails
  t/psgi_v2: minimal test for Atom feed and t.mbox.gz
  feed: fix new.html for v2
  view: permalink (per-message) view shows multiple messages
  searchidx: warn about vivifying multiple ghosts
  v2writable: warn on unseen deleted files
  www: get rid of unnecessary 'inbox' name reference
  searchview: remove unnecessary imports from MID module
  view: depend on SearchMsg for Message-ID

 MANIFEST                         |   1 +
 lib/PublicInbox/Feed.pm          |  98 +++++++++------------
 lib/PublicInbox/Import.pm        |  31 +++++--
 lib/PublicInbox/Inbox.pm         |   9 ++
 lib/PublicInbox/Mbox.pm          |  65 ++++++++++++--
 lib/PublicInbox/Search.pm        |   6 ++
 lib/PublicInbox/SearchIdx.pm     |   4 +
 lib/PublicInbox/SearchMsg.pm     |  13 ++-
 lib/PublicInbox/SearchView.pm    |  16 ++--
 lib/PublicInbox/V2Writable.pm    |  15 +++-
 lib/PublicInbox/View.pm          | 180 +++++++++++++++++++++++++++------------
 lib/PublicInbox/WWW.pm           |  10 +--
 lib/PublicInbox/WwwAtomStream.pm |   9 +-
 t/psgi_v2.t                      | 172 +++++++++++++++++++++++++++++++++++++
 14 files changed, 474 insertions(+), 155 deletions(-)
 create mode 100644 t/psgi_v2.t

-- 
EW

             reply	other threads:[~2018-03-27 11:11 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-27 11:11 Eric Wong (Contractor, The Linux Foundation) [this message]
2018-03-27 11:11 ` [PATCH 01/11] import: consolidate mid prepend logic, here Eric Wong (Contractor, The Linux Foundation)
2018-03-27 11:11 ` [PATCH 02/11] www: $MESSAGE_ID/raw endpoint supports "duplicates" Eric Wong (Contractor, The Linux Foundation)
2018-03-27 11:11 ` [PATCH 03/11] search: reopen DB if each_smsg_by_mid fails Eric Wong (Contractor, The Linux Foundation)
2018-03-27 11:11 ` [PATCH 04/11] t/psgi_v2: minimal test for Atom feed and t.mbox.gz Eric Wong (Contractor, The Linux Foundation)
2018-03-27 11:11 ` [PATCH 05/11] feed: fix new.html for v2 Eric Wong (Contractor, The Linux Foundation)
2018-03-27 11:11 ` [PATCH 06/11] view: permalink (per-message) view shows multiple messages Eric Wong (Contractor, The Linux Foundation)
2018-03-27 11:11 ` [PATCH 07/11] searchidx: warn about vivifying multiple ghosts Eric Wong (Contractor, The Linux Foundation)
2018-03-27 11:11 ` [PATCH 08/11] v2writable: warn on unseen deleted files Eric Wong (Contractor, The Linux Foundation)
2018-03-27 11:11 ` [PATCH 09/11] www: get rid of unnecessary 'inbox' name reference Eric Wong (Contractor, The Linux Foundation)
2018-03-27 11:11 ` [PATCH 10/11] searchview: remove unnecessary imports from MID module Eric Wong (Contractor, The Linux Foundation)
2018-03-27 11:11 ` [PATCH 11/11] view: depend on SearchMsg for Message-ID Eric Wong (Contractor, The Linux Foundation)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://public-inbox.org/README

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180327111132.20681-1-e@80x24.org \
    --to=e@80x24.org \
    --cc=meta@public-inbox.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/public-inbox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).