Skip to content

Allow to configure FF using config.toml

Kamil Trzciński requested to merge allow-to-configure-ff-using-config-toml into master

Why was this MR needed?

Desc taken from !1320 (diffs).

This MR adds tests on top of !1320 (closed).

Moves the default Feature Flag variables definitions lower in the priority list.

Currently, it's impossible to configure some feature flags from config.toml file, because the default variables are added to the list after the variables form config.toml were added. This MR simply reorders the calls, so if the variable is defined in config.toml, it will overwrite the default value.

Are there points in the code the reviewer needs to double check?

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Added tests for this feature/bug
  • In case of conflicts with master - branch was rebased

What are the relevant issue numbers?

Merge request reports