force to compare the bool value from localStorage, it's make sense and avoid a bug if setItem with false explicit

Author: AllenFang

src-react/reducers/app.js

@@ -5,7 +5,7 @@ function app(state, action, config) {
   switch(action.type) {
     case types.RECEIVE_UNGIT_CONFIG: {
       const { ungitConfig } = config;
-      const bugtrackingNagscreenDismissed = localStorage.getItem('bugtrackingNagscreenDismissed');
+      const bugtrackingNagscreenDismissed = localStorage.getItem('bugtrackingNagscreenDismissed') === 'true';
       const showBugtrackingNagscreen = !ungitConfig.config.bugtracking && !bugtrackingNagscreenDismissed;
 
       return { ...state, showBugtrackingNagscreen };
@@ -14,9 +14,9 @@ function app(state, action, config) {
     case types.RECEIVE_GIT_VERSION: {
       const { ungitConfig, versions: { gitVersion } } = config;
       const gitVersionCheckOverride = ungitConfig.config && ungitConfig.config.gitVersionCheckOverride;
-      const gitVersionErrorDismissed = localStorage.getItem('gitVersionErrorDismissed');
+      const gitVersionErrorDismissed = localStorage.getItem('gitVersionErrorDismissed') === 'true';
       const gitVersionError = gitVersion && !gitVersion.satisfied && gitVersion.error;
-      const gitVersionErrorVisible = !gitVersionCheckOverride && gitVersionError && gitVersionErrorDismissed;
+      const gitVersionErrorVisible = !gitVersionCheckOverride && gitVersionError && !gitVersionErrorDismissed;
 
       return { ...state, gitVersionErrorVisible };
     }
@@ -32,7 +32,7 @@ function app(state, action, config) {
 
     case types.RECEIVE_USER_CONFIG: {
       const { userConfig } = config;
-      const bugtrackingNagscreenDismissed = localStorage.getItem('bugtrackingNagscreenDismissed');
+      const bugtrackingNagscreenDismissed = localStorage.getItem('bugtrackingNagscreenDismissed') === 'true';
       const showBugtrackingNagscreen = !userConfig.bugtracking && !bugtrackingNagscreenDismissed;
 
       return { ...state, showBugtrackingNagscreen };