From 03c9f08a7a9f3cf923cb74177ef85a03fd78bfce Mon Sep 17 00:00:00 2001 From: Erin Mounts Date: Fri, 11 Mar 2016 10:52:44 -0800 Subject: [PATCH] [BRIDGE-1183] 409 on consent just means... ...they already signed it. --- .../Activities/APCActivitiesViewController.m | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/APCAppCore/APCAppCore/UI/TabBarControllers/Activities/APCActivitiesViewController.m b/APCAppCore/APCAppCore/UI/TabBarControllers/Activities/APCActivitiesViewController.m index 8b6f6bd2..f1d24208 100644 --- a/APCAppCore/APCAppCore/UI/TabBarControllers/Activities/APCActivitiesViewController.m +++ b/APCAppCore/APCAppCore/UI/TabBarControllers/Activities/APCActivitiesViewController.m @@ -771,7 +771,10 @@ - (void)handleSigninResponseWithError:(NSError *)error { } - (void)handleConsentResponseWithError:(NSError *)error { - if (error) { + // 409 Conflict in this context means consent already signed, which happens if a user tries to re-sign-up + // with an existing email account rather than signing in with it. In any case it means they've already signed + // the consent so we can just mark the user as consented and move on. + if (error && error.code != 409) { APCLogError2(error); [self showConsentError: error]; }