Enable Android ppro messaging part 1 #118
Conversation
|
Your PR is hosted at https://staticcdn.duckduckgo.com/remotemessaging/config/staging/android-config.json |
karlenDimla
force-pushed
the
feature/karl/android-ppro-messaging-p1
branch
from
315ab8e
to
029602a
Compare
karlenDimla
force-pushed
the
feature/karl/android-ppro-messaging-p1
branch
from
029602a
to
3ce6446
Compare
| @@ -59,6 +59,9 @@ | |||
| }, | |||
| "matchingRules": [ | |||
| 12 | |||
| ], | |||
| "exclusionRules": [ | |||
There was a problem hiding this comment.
Excluding message if already interacted with previous or current ones.
| @@ -405,7 +417,29 @@ | |||
| { | |||
| "id": 12, | |||
| "targetPercentile": { | |||
| "before": 0.05 | |||
| "before": 0.0625 | |||
There was a problem hiding this comment.
4 messages should have now 6.25%
There was a problem hiding this comment.
Updating the first one to be 5% as we are updating the taarget to 10% (https://app.asana.com/0/0/1208299529724942/1208890135916138/f)
| { | ||
| "id": 13, | ||
| "targetPercentile": { | ||
| "before": 0.0666 |
There was a problem hiding this comment.
Because of how we process messages (in order and first match wins), to keep the same probability in all 4 variants, we should calculate percentile based on the message position. For example: percentile of message B to be 6.25% should consider probability of message A not showing.
|
@karlenDimla do we need to put any minVersion supported here? |
…d with previous or current ones.
cmonfortep
force-pushed
the
feature/karl/android-ppro-messaging-p1
branch
from
bc6a54e
to
303dad9
Compare
karlenDimla
force-pushed
the
feature/karl/android-ppro-messaging-p1
branch
from
303dad9
to
649a9cd
Compare
karlenDimla
force-pushed
the
feature/karl/android-ppro-messaging-p1
branch
from
c6e01fa
to
469c370
Compare
karlenDimla
force-pushed
the
feature/karl/android-ppro-messaging-p1
branch
from
469c370
to
382e8ea
Compare
Specs: https://app.asana.com/0/1207619243206445/1208299529724942/f
Target enable date: Dec 2, 2024