Remove logs, update release notes, move Router outside of ternary ope…

…rator

Signed-off-by: Vladimir <[email protected]>

RELEASE.md

@@ -10,8 +10,7 @@ Please follow the established format:
 
 - Fix incorrect rendering of datasets in modular pipelines. (#1439)
 - Fix broken SVG/PNG exports in light theme. (#1463)
-- Fix error when setting globalToolbar prop value as false. (#1351)
-- Fix error error in dropdown component. (#1351)
+- Fix dataset and global tool bar error with standalone React component (#1351)
 
 # Release 6.3.4
 

src/components/wrapper/wrapper.js

@@ -47,9 +47,9 @@ export const Wrapper = ({ displayGlobalToolbar, theme }) => {
       })}
     >
       <h1 className="pipeline-title">Kedro-Viz</h1>
-      {displayGlobalToolbar ? (
-        <GraphQLProvider>
-          <Router>
+      <Router>
+        {displayGlobalToolbar ? (
+          <GraphQLProvider>
             <GlobalToolbar isOutdated={isOutdated} />
             <SettingsModal
               isOutdated={isOutdated}
@@ -69,11 +69,11 @@ export const Wrapper = ({ displayGlobalToolbar, theme }) => {
                 <ExperimentWrapper />
               </Route>
             </Switch>
-          </Router>
-        </GraphQLProvider>
-      ) : (
-        <FlowChartWrapper />
-      )}
+          </GraphQLProvider>
+        ) : (
+          <FlowChartWrapper />
+        )}
+      </Router>
     </div>
   );
 };

src/store/load-data.js

@@ -11,7 +11,6 @@ import { getUrl } from '../utils';
  */
 const loadJsonData = (path = getUrl('main'), fallback = {}) =>
   json(path).catch(() => {
-    console.log(path);
     const fullPath = `/public${path.substr(1)}`;
 
     // For main route throw a user error