From 7e135cef47f90855507a75a0eb6e2118a96c8fe2 Mon Sep 17 00:00:00 2001
From: Chi Vinh Le <vinh@wikiwi.io>
Date: Thu, 1 Feb 2018 17:46:06 +0100
Subject: [PATCH] Add a little comment

---
 .../src/tabs/stream/containers/DraftArea.js   | 21 ++++++++++++++-----
 1 file changed, 16 insertions(+), 5 deletions(-)

diff --git a/client/coral-embed-stream/src/tabs/stream/containers/DraftArea.js b/client/coral-embed-stream/src/tabs/stream/containers/DraftArea.js
index 63ebbebea..b9b7fd58a 100644
--- a/client/coral-embed-stream/src/tabs/stream/containers/DraftArea.js
+++ b/client/coral-embed-stream/src/tabs/stream/containers/DraftArea.js
@@ -25,10 +25,19 @@ export default class DraftAreaContainer extends React.Component {
   };
 
   onChange = e => {
-    this.context.pymSessionStorage.setItem(this.getPath(), e.target.value);
     this.props.onChange && this.props.onChange(e.target.value);
   };
 
+  componentWillReceiveProps(nextProps) {
+    if (this.props.value !== nextProps.value) {
+      if (nextProps.value) {
+        this.context.pymSessionStorage.setItem(this.getPath(), nextProps.value);
+      } else {
+        this.context.pymSessionStorage.removeItem(this.getPath());
+      }
+    }
+  }
+
   render() {
     return (
       <DraftArea
@@ -47,17 +56,19 @@ export default class DraftAreaContainer extends React.Component {
 }
 
 DraftAreaContainer.contextTypes = {
+  // We use pymSessionStorage instead to persist the data directly on the parent page,
+  // in order to mitigate strict cross domain security settings.
   pymSessionStorage: PropTypes.object,
 };
 
 DraftAreaContainer.propTypes = {
   charCountEnable: PropTypes.bool,
   maxCharCount: PropTypes.number,
-  id: PropTypes.string,
-  value: PropTypes.string,
+  id: PropTypes.string.isRequired,
+  value: PropTypes.string.isRequired,
   placeholder: PropTypes.string,
-  onChange: PropTypes.func,
+  onChange: PropTypes.func.isRequired,
   disabled: PropTypes.bool,
   rows: PropTypes.number,
-  label: PropTypes.string,
+  label: PropTypes.string.isRequired,
 };