Redirect changed to Redirect push to to keep history. (#869)

* Redirect to changed to Redirect push to to keep history.

* Rewiev - remove false changes in comments

Author: MilanPospisil

developer_guidelines.md

@@ -82,8 +82,8 @@ Example:
 
 ```
 // Static route
-<Redirect to={Paths.search}/>
+<Redirect push to={Paths.search}/>
 
 // Dynamic route
-<Redirect to={formatPath(Paths.editNamespace, {namespace: NSname});}/>
+<Redirect push to={formatPath(Paths.editNamespace, {namespace: NSname});}/>
 ```

src/containers/edit-namespace/edit-namespace.tsx

@@ -92,7 +92,7 @@ class EditNamespace extends React.Component<RouteComponentProps, IState> {
     }
 
     if (redirect) {
-      return <Redirect to={redirect} />;
+      return <Redirect push to={redirect} />;
     }
     return (
       <React.Fragment>

src/containers/execution-environment-detail/base.tsx

@@ -80,7 +80,7 @@ export function withContainerRepo(WrappedComponent) {
           />
         );
       } else if (this.state.redirect === 'notFound') {
-        return <Redirect to={Paths.notFound} />;
+        return <Redirect push to={Paths.notFound} />;
       }
 
       if (this.state.loading) {

src/containers/group-management/group-list.tsx

@@ -115,7 +115,7 @@ class GroupList extends React.Component<RouteComponentProps, IState> {
     const noData = groups.length === 0 && !filterIsSet(params, ['name']);
 
     if (redirect) {
-      return <Redirect to={redirect} />;
+      return <Redirect push to={redirect} />;
     }
 
     return (

src/containers/login/login.tsx

@@ -34,7 +34,7 @@ class LoginPage extends React.Component<RouteComponentProps, IState> {
 
   render() {
     if (this.state.redirect) {
-      return <Redirect to={this.state.redirect}></Redirect>;
+      return <Redirect push to={this.state.redirect}></Redirect>;
     }
 
     const helperText = (

src/containers/namespace-detail/namespace-detail.tsx

@@ -113,7 +113,7 @@ export class NamespaceDetail extends React.Component<IProps, IState> {
     } = this.state;
 
     if (redirect) {
-      return <Redirect to={redirect} />;
+      return <Redirect push to={redirect} />;
     }
 
     if (!namespace) {

src/containers/settings/user-profile.tsx

@@ -123,7 +123,7 @@ class UserProfile extends React.Component<RouteComponentProps, IState> {
           },
           () => this.context.setUser(result.data),
         );
-        //redirect to login page when password is changed
+        //Redirect to login page when password is changed
         if (user.password) {
           this.setState({ redirect: Paths.login });
         }

src/containers/user-management/user-edit.tsx

@@ -86,7 +86,7 @@ class UserEdit extends React.Component<RouteComponentProps, IState> {
     const { user } = this.state;
     UserAPI.update(user.id.toString(), user)
       .then(() => {
-        //redirect to login page when password of logged user is changed
+        //Redirect to login page when password of logged user is changed
         if (this.context.user.id === user.id && user.password) {
           this.setState({ redirect: Paths.login });
         } else {

src/containers/user-management/user-list.tsx

@@ -113,7 +113,7 @@ class UserList extends React.Component<RouteComponentProps, IState> {
     const { user } = this.context;
 
     if (redirect) {
-      return <Redirect to={redirect} />;
+      return <Redirect push to={redirect} />;
     }
 
     return (

src/loaders/insights/Routes.js

@@ -281,7 +281,7 @@ export const Routes = (props) => {
       <Route
         render={() =>
           some(Paths, (p) => p === path) ? null : (
-            <Redirect to={Paths.notFound} />
+            <Redirect push to={Paths.notFound} />
           )
         }
       />

src/loaders/standalone/routes.tsx

@@ -128,7 +128,7 @@ class AuthHandler extends React.Component<
     // only enforce this if feature flags are set. Otherwise the container
     // registry will always return a 404 on the first load.
     if (this.props.isDisabled) {
-      return <Redirect to={Paths.notFound}></Redirect>;
+      return <Redirect push to={Paths.notFound}></Redirect>;
     }
 
     return <Component {...props}></Component>;