From bac4b78e0908c0cb01a3842436950c7bcf793cf9 Mon Sep 17 00:00:00 2001 From: David Svantesson Date: Wed, 4 Dec 2019 02:08:56 +0100 Subject: Branch protection: Possibility to not use whitelist but allow anyone with write access (#9055) * Possibility to not use whitelist but allow anyone with write access * fix existing test * rename migration function * Try to give a better name for migration step * Clear settings if higher level setting is not set * Move official reviews to db instead of counting approvals each time * migration * fix * fix migration * fix migration * Remove NOT NULL from EnableWhitelist as migration isn't possible * Fix migration, reviews are connected to issues. * Fix SQL query issues in GetReviewersByPullID. * Simplify function GetReviewersByIssueID * Handle reviewers that has been deleted * Ensure reviews for test is in a well defined order * Only clear and set official reviews when it is an approve or reject. --- web_src/js/index.js | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'web_src') diff --git a/web_src/js/index.js b/web_src/js/index.js index 1683210107..7f3874b857 100644 --- a/web_src/js/index.js +++ b/web_src/js/index.js @@ -1043,6 +1043,11 @@ function initRepository() { $($(this).data('target')).addClass('disabled'); } }); + $('.disable-whitelist').change(function () { + if (this.checked) { + $($(this).data('target')).addClass('disabled'); + } + }); } } -- cgit v1.2.3