Skip to content

feat: try to detect mailing lists as event attendees - #8063

Draft
ChristophWurst wants to merge 1 commit into
mainfrom
feat/attendee-list-warning
Draft

ChristophWurst wants to merge 1 commit into
mainfrom
feat/attendee-list-warning

Conversation

@ChristophWurst

Copy link
Copy Markdown
Member

The backend doesn't differenciate between individual email and a list. It will just send the invite. For a list, the invite is bogus and can't really be answered. We should warn the user.

image

@ChristophWurst
ChristophWurst requested a review from kra-mo March 12, 2026 14:47
@ChristophWurst ChristophWurst self-assigned this Mar 12, 2026
@ChristophWurst ChristophWurst added 3. to review Waiting for reviews enhancement New feature request labels Mar 12, 2026
@tcitworld

tcitworld commented Mar 12, 2026 •

Copy link
Copy Markdown
Member

Maybe also check for the KIND:group vCard parameter as possibly given by the ContactsManager ?

If the mail app knows that an e-mail address is a mailing-list through e-mail headers, it could also save it when adding the contact to the recently contacted contacts.

@SebastianKrupinski SebastianKrupinski added this to the v6.3.0 milestone Mar 17, 2026
@kra-mo

kra-mo commented Mar 19, 2026

Copy link
Copy Markdown
Member

Good, but from a design side, the clipping text is obviously not ideal. Could the warning wrap and have sufficient space on the bottom?

@ChristophWurst

Copy link
Copy Markdown
Member Author

I did not find an option for that in https://vue-select.org/guide/options.html#options-prop, unfortunately.

@kra-mo

kra-mo commented Mar 20, 2026 •

Copy link
Copy Markdown
Member

Can you add a hard line break, then? If not, the string should be shortened for now and this fixed upstream later.

For a shorter string, something like "List subscribers cannot accept invites."

Obviously not great for localization and a11y, but better than it being broken in default English.

The backend doesn't differenciate between individual email and a list.
It will just send the invite. For a list, the invite is bogus and can't
really be answered. We should warn the user.

Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
@odzhychko
odzhychko force-pushed the feat/attendee-list-warning branch from b4fec1f to 8a56239 Compare September 22, 2026 11:26
@codecov

codecov Bot commented Sep 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@odzhychko
odzhychko marked this pull request as draft September 22, 2026 12:46
@odzhychko

Copy link
Copy Markdown
Contributor

Good, but from a design side, the clipping text is obviously not ideal. Could the warning wrap and have sufficient space on the bottom?

This is possible. I will update it the PR and mark it ready for review when done.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement New feature request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants