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

Redeliver holding pen emails if there's a single guess #7790

Merged
merged 4 commits into from
Sep 21, 2023

Conversation

alexander-griffen
Copy link
Contributor

@alexander-griffen alexander-griffen commented Jun 23, 2023


Relevant issue(s)

Fixes #4990

What does this do?

This changes the behaviour of the holding pen such that if an email is similar enough to the expected email for an info request that it only has one Guess attributed, it'll get attached to that IR instead of being sent to the holding pen.

Why was this needed?

Reduce holding pen noise

Implementation notes

  • Originally, the 'subject' part of the Guess implementation was also used for this feature, but this has been removed, as it will be assigned as a perfect score regardless of how close it is.

Screenshots

Notes to reviewer

@alexander-griffen alexander-griffen self-assigned this Jun 23, 2023
@alexander-griffen alexander-griffen marked this pull request as ready for review July 6, 2023 17:24
@gbp gbp changed the base branch from develop to 5988-info-requests-in-headers July 14, 2023 14:11
@gbp gbp changed the base branch from 5988-info-requests-in-headers to develop July 21, 2023 15:59
@alexander-griffen alexander-griffen force-pushed the 5990-single-guess-redeliver branch 2 times, most recently from bf0c1fb to deddd0b Compare July 28, 2023 14:43
@gbp gbp force-pushed the 5990-single-guess-redeliver branch 2 times, most recently from 1577b4c to e582f03 Compare August 8, 2023 09:19
Allow guesses to be passed different components. These components will
make up key/values used to calculate how good the guess is.
@gbp gbp force-pushed the 5990-single-guess-redeliver branch from e582f03 to 9058c68 Compare September 21, 2023 08:40
gbp and others added 2 commits September 21, 2023 15:07
Test how close the id/idhash components are to those on the info
request. These scores can then be used to deliver incoming messages to a
request or not.
Now, the 'Guessing' mechanism will be used when a mail is
received. If there is only one guess produced by this mechanism,
then the mail will go straight to that guess's IR
@gbp gbp force-pushed the 5990-single-guess-redeliver branch from 9058c68 to 6c455ef Compare September 21, 2023 14:07
@gbp gbp merged commit b6c48e4 into develop Sep 21, 2023
4 checks passed
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.

Automatically redeliver messages sent to holding pen when there's a single guess
2 participants