Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Cross-list conversations are missing on some mailing lists #21

Open
LinqLover opened this issue Jan 13, 2024 · 0 comments
Open

Cross-list conversations are missing on some mailing lists #21

LinqLover opened this issue Jan 13, 2024 · 0 comments

Comments

@LinqLover
Copy link
Contributor

Example is this conversation which has been sent to both squeak-dev and vm-dev:

<Mailbird-86ed6f34-65f4-4851-b203-003507ad2baa@hpi.uni-potsdam.de>

https://lists.squeakfoundation.org/archives/list/squeak-dev@lists.squeakfoundation.org/thread/YV6AXZVKPZ3W6DJINBIEREX6C3PQ3MLL/ and https://lists.squeakfoundation.org/archives/list/vm-dev@lists.squeakfoundation.org/thread/YV6AXZVKPZ3W6DJINBIEREX6C3PQ3MLL/

SqhMailmanAggregator class>>#conversationsOn: lists this conversation only one of the lists (squeak-dev in my image). Maybe SqhMailWrapper>>#listName should be plural instead? Or should there be separate wrappers for messages on each list? But how to distinguish them as they are only indexed through their message id? See also the cross-list warning in SqhMailmanAggregator class>>#resetCacheForMailInReplyTo. Not sure about the possible consequences listNames in plural ...

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

No branches or pull requests

1 participant