Skip to content

Fix High Risk Matches in Emails#489

Merged
Martinski4GitHub merged 5 commits intodevfrom
Fix-Email-Matches
Jun 18, 2025
Merged

Fix High Risk Matches in Emails#489
Martinski4GitHub merged 5 commits intodevfrom
Fix-Email-Matches

Conversation

@ExtremeFiretop
Copy link
Owner

@ExtremeFiretop ExtremeFiretop commented Jun 17, 2025

Fix Matches in Emails for High Risk Phrases.

Currently the logic only works to highlight the high risk phrases in the email assuming they are all on the same line.
We recently fixed this by flattening the content in PR: #476

But only for the verification process, not for the email contents.
This is the issue found below with "Must be manually"
image

This is the fixed HTML version:
image

And the fixed Plain text version:
image

Fix Matches in Emails
@ExtremeFiretop ExtremeFiretop marked this pull request as ready for review June 17, 2025 09:07
@ExtremeFiretop ExtremeFiretop changed the title Fix Matches in Emails Fix High Risk Matches in Emails Jun 17, 2025
@Martinski4GitHub
Copy link
Collaborator

Fix Matches in Emails for High Risk Phrases.

Currently the logic only works to highlight the high risk phrases in the email assuming they are all on the same line. We recently fixed this by flattening the content in PR: #476

But only for the verification process, not for the email contents. This is the issue found below with "Must be manually" image

This is the fixed HTML version: image

And the fixed Plain text version: image

Very nice catch!!

Copy link
Collaborator

@Martinski4GitHub Martinski4GitHub left a comment

Choose a reason for hiding this comment

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

Approved!!

@Martinski4GitHub Martinski4GitHub merged commit 763724e into dev Jun 18, 2025
1 check passed
@ExtremeFiretop
Copy link
Owner Author

Approved!!

Thank you good Sir!

@ExtremeFiretop ExtremeFiretop deleted the Fix-Email-Matches branch June 18, 2025 14:10
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.

2 participants