From ab626991483db378461fc6876271dc280aa89a4d Mon Sep 17 00:00:00 2001
From: Adam Butterworth <abutterworth@edx.org>
Date: Mon, 13 May 2019 14:37:13 -0600
Subject: [PATCH] fix: send social link value when empty string is supplied
 (#36)

* fix: send social link value when empty string is supplied

* refactor: add clarity
---
 src/account-settings/service.js | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/account-settings/service.js b/src/account-settings/service.js
index 93761fb..8c941ac 100644
--- a/src/account-settings/service.js
+++ b/src/account-settings/service.js
@@ -63,9 +63,10 @@ function packAccountCommitData(commitData) {
   const packedData = commitData;
 
   SOCIAL_PLATFORMS.forEach(({ id, key }) => {
-    if (commitData[key]) {
-      packedData.social_links = [{ platform: id, social_link: commitData[key] }];
-    }
+    // Skip missing values. Empty strings are valid values and should be preserved.
+    if (commitData[key] === undefined) return;
+
+    packedData.social_links = [{ platform: id, social_link: commitData[key] }];
     delete packedData[key];
   });