search

catalyst / moodle-tool_abconfig

A way to A/B test config, or slowly turn on config for certain audiences or % of traffic
https://moodle.org/plugins/tool_abconfig
6 stars 4 forks source link

The order of the condition sets in the edit form is not the same as shown in the experiments #15

Closed brendanheywood closed 4 years ago

brendanheywood commented 4 years ago

I am guessing the edit form is natural db order (order inserted) vs the other page which is ordered by shortname?

brendanheywood commented 4 years ago

Ideally it would be good if here:

https://github.com/catalyst/moodle-tool_abconfig/blob/master/classes/form/edit_experiment.php#L104

did not touch the db at all and just called the experience_manager so all the db stuff is in one place

Peterburnett commented 4 years ago

Fixed