Set new user in main app instead of utils

sho-87 · Mar 14, 2020 · 8e60adc · 8e60adc
1 parent fbe223b
commit 8e60adc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/App.js b/src/App.js
@@ -208,6 +208,10 @@ class App extends Component {
     if (!this.props.isEmbed) {
       localStorage.setItem('version', version);
     }
+
+    if (isNewUser()) {
+      localStorage.setItem('isNewUser', false);
+    }
   }
 
   /**
@@ -688,7 +692,7 @@ class App extends Component {
               ? isUpgrade(localStorage.getItem('version'), version)
               : true
           }
-          isNewUser={isNewUser}
+          isNewUser={isNewUser()}
           isInvalidBuild={this.invalidModalFlag}
         />
 

diff --git a/src/utils.js b/src/utils.js
@@ -199,7 +199,6 @@ export function isNewUser() {
     !localStorage.getItem('isNewUser') ||
     JSON.parse(localStorage.getItem('isNewUser') === true)
   ) {
-    localStorage.setItem('isNewUser', false);
     return true;
   } else {
     return false;
@@ -215,7 +214,7 @@ export function isNewUser() {
  */
 export function isUpgrade(oldVer, newVer) {
   // New users are never considered upgrades
-  if (!localStorage.getItem('isNewUser')) {
+  if (isNewUser()) {
     return false;
   }