From b48c4707e34a7d95257501f4bca007c2aecd8a12 Mon Sep 17 00:00:00 2001 From: Harold Wanyama Date: Tue, 6 Feb 2024 21:35:49 +0300 Subject: [PATCH] [#3897]Feature/ICLA and ECLA use case - Changed flow for icla and ccla checks for an invalidated contributor use case Signed-off-by: Harold Wanyama --- cla-backend/cla/utils.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/cla-backend/cla/utils.py b/cla-backend/cla/utils.py index 88940243b..1d222fce1 100644 --- a/cla-backend/cla/utils.py +++ b/cla-backend/cla/utils.py @@ -698,10 +698,10 @@ def user_signed_project_signature(user: User, project: Project) -> bool: # Verify if user has been approved: https://github.com/communitybridge/easycla/issues/332 cla.log.debug(f'{fn} - CCLA signature check - ' 'checking to see if the user is in one of the approval lists...') - if project.get_project_ccla_requires_icla_signature() is True: - cla.log.debug(f'{fn} - CCLA signature check - ' - 'project requires ICLA signature as well as CCLA signature ') - elif user.is_approved(signature) : + # if project.get_project_ccla_requires_icla_signature() is True: + # cla.log.debug(f'{fn} - CCLA signature check - ' + # 'project requires ICLA signature as well as CCLA signature ') + if user.is_approved(signature) : ccla_pass = True else: # Set user signatures approved = false due to user failing whitelist checks