diff --git a/CHANGELOG.rst b/CHANGELOG.rst index 89a17f6..1a3072b 100644 --- a/CHANGELOG.rst +++ b/CHANGELOG.rst @@ -14,6 +14,10 @@ Change Log Unreleased ~~~~~~~~~~ +[0.9.2] - 2021-09-07 +~~~~~~~~~~~~~~~~~~~~ +* Update IDV signal handler field names to be more explicit about the received names. + [0.9.1] - 2021-09-07 ~~~~~~~~~~~~~~~~~~~~ * Add extra validation for the VerifiedName serializer, throwing a 400 error if diff --git a/edx_name_affirmation/__init__.py b/edx_name_affirmation/__init__.py index b9b310a..acc22dd 100644 --- a/edx_name_affirmation/__init__.py +++ b/edx_name_affirmation/__init__.py @@ -2,6 +2,6 @@ Django app housing name affirmation logic. """ -__version__ = '0.9.1' +__version__ = '0.9.2' default_app_config = 'edx_name_affirmation.apps.EdxNameAffirmationConfig' # pylint: disable=invalid-name diff --git a/edx_name_affirmation/handlers.py b/edx_name_affirmation/handlers.py index d05b9de..250f6f4 100644 --- a/edx_name_affirmation/handlers.py +++ b/edx_name_affirmation/handlers.py @@ -32,7 +32,7 @@ def verified_name_approved(sender, instance, **kwargs): # pylint: disable=unuse ) -def idv_attempt_handler(attempt_id, user_id, status, full_name, profile_name, **kwargs): +def idv_attempt_handler(attempt_id, user_id, status, photo_id_name, full_name, **kwargs): """ Receiver for IDV attempt updates @@ -40,14 +40,14 @@ def idv_attempt_handler(attempt_id, user_id, status, full_name, profile_name, ** attempt_id(int): ID associated with the IDV attempt user_id(int): ID associated with the IDV attempt's user status(str): status in IDV language for the IDV attempt - full_name(str): name to be used as verified name - profile_name(str): user's current profile name + photo_id_name(str): name to be used as verified name + full_name(str): user's pending name change or current profile name """ if not is_verified_name_enabled(): return trigger_status = VerifiedNameStatus.trigger_state_change_from_idv(status) - verified_names = VerifiedName.objects.filter(user__id=user_id, verified_name=full_name).order_by('-created') + verified_names = VerifiedName.objects.filter(user__id=user_id, verified_name=photo_id_name).order_by('-created') if verified_names: # if there are VerifiedName objects, we want to update existing entries # for each attempt with no attempt id (either proctoring or idv), update attempt id @@ -89,8 +89,8 @@ def idv_attempt_handler(attempt_id, user_id, status, full_name, profile_name, ** user = User.objects.get(id=user_id) verified_name = VerifiedName.objects.create( user=user, - verified_name=full_name, - profile_name=profile_name, + verified_name=photo_id_name, + profile_name=full_name, verification_attempt_id=attempt_id, status=(trigger_status if trigger_status else VerifiedNameStatus.PENDING), )