Arvados

Ready for review at 89f8175850a9279036053b2a6785980a479e08d8 on branch 18001-add-UserNotifierEmailBcc

Tests ran at developer-run-tests: #2631 . The wb functionals had an unrelated failure, it passed when re-run at developer-run-tests-apps-workbench-functionals: #2673 .

arvcfg.declare_config "Users.UserNotifierEmailBcc", Hash, ->(cfg, k, v) { arrayToHash cfg, "Users.UserNotifierEmailBcc", v }

The ->(cfg, k, v) {...} part is incorrect, that's only for keys that need to be migrated from legacy application.yml. It should be just:

arvcfg.declare_config "Users.UserNotifierEmailBcc", Hash

The rest LGTM

Peter Amstutz wrote:

[...]

The ->(cfg, k, v) {...} part is incorrect, that's only for keys that need to be migrated from legacy application.yml. It should be just:

[...]

The rest LGTM

Great thanks, merged with that change.