-
-
Notifications
You must be signed in to change notification settings - Fork 251
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
feat(message-system): add ab testing message #15281
base: develop
Are you sure you want to change the base?
Conversation
🚀 Expo preview is ready!
|
c16f0f3
to
e412009
Compare
c5db5a1
to
cd14491
Compare
Hello @adderpositive , please let's wait for @tomasklim and let's discuss this in more broader context. Personally I do not like the idea to mix it with message system, so lets discuss. |
Yes, of course. We have been already discussing the extension of the system. I will go ahead and briefly describe it here.
I hope it is understandable a little bit. :) |
cd14491
to
4cb3aa7
Compare
Description
Example of experiments
experiments
property is placed in the top level of config.v[version].jsonconditions
are the same property as for the action - https://github.com/trezor/trezor-suite/blob/develop/docs/features/message-system.mdexperiment
has required parametersid
andgroups
id
must be unique, but can also be used for nice naming (e.g. "button-testing")groups
as an array of testing groups - minimum length of items is twopercentage
is a number between 0 and 100, a sum of all percentages has to be 100variant
can be any string - name of the groupRelated Issue
Resolve #15279
Resolve #15067