@tsteur opened this Pull Request on September 9th 2020 Member

Not needed in developer changelog as we automatically migrate existing setting and if that doesn't work then we still support previous settings for BC (important as it is a security related feature, not always do config changes work eg when they forget to distribute it across multiple servers etc). Also it's not a developer setting but just a regular user setting.

Also renames login_whitelist_apply_to_reporting_api_requests to login_allowlist_apply_to_reporting_api_requests

once merged will need to update the FAQ (I'll do this)

@tsteur commented on September 30th 2020 Member

@sgiehl applied the review comments. replaced the word also in few other places but not in places like the validator or so when it requires changes across plugins as it's not as easy to do and needs a separate PR anyway

@sgiehl commented on October 1st 2020 Member

Seems you added the screenshot directly, and not to LFS...

@tsteur commented on October 1st 2020 Member

fixed

This Pull Request was closed on October 1st 2020
Powered by GitHub Issue Mirror