Skip to content

Agent/listing related mails - #414

Closed
emnl51 wants to merge 24 commits into
orangecoding:masterfrom
emnl51:agent/listing-related-mails
Closed

Agent/listing related mails#414
emnl51 wants to merge 24 commits into
orangecoding:masterfrom
emnl51:agent/listing-related-mails

Conversation

@emnl51

@emnl51 emnl51 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary

This feature displays all emails associated with a property directly on the listing details page.

Users can see:

  • Email subject and sender
  • Received date
  • Matching method and confidence
  • Full email content in an expandable section

Email privacy is preserved. Related emails are only visible to the mailbox owner, even when a listing is shared with another user.

Validation

  • 16 focused tests passed
  • Lint passed
  • Frontend production build passed
  • English, German, and Turkish translations included

@emnl51 emnl51 closed this Aug 12, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7b6552eace

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +119 to +121
WHEN mail_accounts.host <> excluded.host
OR mail_accounts.username <> excluded.username
OR mail_accounts.mailbox <> excluded.mailbox

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Reset the IMAP cursor when the endpoint changes

When a user changes only port or secure, these conditions preserve the previous uid_validity and last_uid, even though both fields alter the endpoint used by performMailAccountSync. If the new endpoint reports the same UIDVALIDITY—a value that is not globally unique—the next sync resumes from the old server's UID and can permanently skip messages whose UIDs are below that cursor. Include port and secure in both cursor-reset conditions.

Useful? React with 👍 / 👎.

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

Successfully merging this pull request may close these issues.

1 participant