Skip to content

feat: add ai hint for smart replies#12646

Open
GretaD wants to merge 4 commits intomainfrom
feat/add-ai-hint-for-smarreply
Open

feat: add ai hint for smart replies#12646
GretaD wants to merge 4 commits intomainfrom
feat/add-ai-hint-for-smarreply

Conversation

@GretaD
Copy link
Copy Markdown
Contributor

@GretaD GretaD commented Mar 23, 2026

fixes #12390

rich text
Screenshot from 2026-03-23 17-12-20


plain text
Screenshot from 2026-03-23 17-14-03

I wrote Part of this reply because people can also add more manually as a reply. Happy to change it if someone has a better idea.

buttons
open
Screenshot from 2026-03-27 17-42-16

closed
Screenshot from 2026-03-27 17-42-00

@GretaD GretaD self-assigned this Mar 23, 2026
@GretaD GretaD changed the title feat: add ai hint for smart replys feat: add ai hint for smart replies Mar 23, 2026
Signed-off-by: greta <gretadoci@gmail.com>
@GretaD GretaD force-pushed the feat/add-ai-hint-for-smarreply branch from fa8078f to 5139ef7 Compare March 23, 2026 16:55
@GretaD GretaD requested a review from ChristophWurst March 23, 2026 16:55
@ChristophWurst
Copy link
Copy Markdown
Member

The first requirement of the ticket is left out. The buttons are not changed. What's the plan?

@GretaD
Copy link
Copy Markdown
Contributor Author

GretaD commented Mar 24, 2026

The first requirement of the ticket is left out. The buttons are not changed. What's the plan?

I was planning to ask on the ticket, but forgot :)

I already fixed that here: #12546 (comment)

But then I thought that maybe it's not necessary to have both, the AI design and also a hint. Is it mandatory to have it also written when it is visually very present that that's an AI-generated button? If so, i can add the closed commit on this pr.

@ChristophWurst
Copy link
Copy Markdown
Member

Yes it has to be clear

Signed-off-by: greta <gretadoci@gmail.com>
Signed-off-by: greta <gretadoci@gmail.com>
@GretaD GretaD requested a review from kesselb March 30, 2026 15:33
if (reply) {
const original = await this.fetchMessage(reply.data.databaseId)
if (reply?.smartReply) {
const aiDisclaimerText = t('mail', '(Parts of this reply were generated with AI)')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe?

Suggested change
const aiDisclaimerText = t('mail', '(Parts of this reply were generated with AI)')
const aiDisclaimerText = t('mail', '(All or part of this reply was generated by AI)')

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, this sounds good as well

Signed-off-by: greta <gretadoci@gmail.com>
@GretaD GretaD requested a review from tcitworld March 30, 2026 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve clarity of generated content in suggested replies

4 participants