@truedat/core 4.41.2 → 4.41.3

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [4.41.3] 2022-04-01
+
+### Changed
+
+- [TD-4670] `RouteListener` now dismisses alerts when any change of URL path
+  occurs
+
 ## [4.41.2] 2022-03-30
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@truedat/core",
-  "version": "4.41.2",
+  "version": "4.41.3",
   "description": "Truedat Web Core",
   "sideEffects": false,
   "jsnext:main": "src/index.js",
@@ -106,5 +106,5 @@
     "react-dom": ">= 16.8.6 < 17",
     "semantic-ui-react": ">= 0.88.2 < 2.1"
   },
-  "gitHead": "de1e8cd9c744c0dae85be4585fccc1c50fdf6a07"
+  "gitHead": "946829496288ea4ff05b3029b37d8c8d16393127"
 }

package/src/components/RouteListener.js

@@ -11,10 +11,7 @@ export class RouteListener extends React.Component {
   componentDidUpdate(prevProps) {
     const prevPath = prevProps.location.pathname;
     const currentPath = this.props.location.pathname;
-    if (
-      !_.isEmpty(this.props.message) &&
-      prevPath.split("/")[1] !== currentPath.split("/")[1]
-    ) {
+    if (!_.isEmpty(this.props.message) && prevPath !== currentPath) {
       this.props.dismissAlert();
     }
   }
@@ -27,10 +24,10 @@ export class RouteListener extends React.Component {
 RouteListener.propTypes = {
   location: PropTypes.object,
   message: PropTypes.object,
-  dismissAlert: PropTypes.func
+  dismissAlert: PropTypes.func,
 };
 
-const mapStateToProps = state => ({ message: getMessage(state) });
+const mapStateToProps = (state) => ({ message: getMessage(state) });
 
 const mapDispatchToProps = { dismissAlert };
 

package/src/components/__tests__/RouteListener.spec.js

@@ -5,7 +5,7 @@ import { RouteListener } from "../RouteListener";
 describe("<RouteListener />", () => {
   const props = {
     message: {},
-    location: { pathname: "/test" }
+    location: { pathname: "/test" },
   };
 
   it("matches the latest snapshot", () => {
@@ -20,7 +20,7 @@ describe("<RouteListener />", () => {
     );
     wrapper.setProps({
       message: { not: "empty" },
-      location: { pathname: "/different" }
+      location: { pathname: "/different" },
     });
     expect(dismissAlert.mock.calls.length).toBe(1);
   });
@@ -31,19 +31,7 @@ describe("<RouteListener />", () => {
       <RouteListener dismissAlert={dismissAlert} {...props} />
     );
     wrapper.setProps({
-      location: { pathname: "/different" }
-    });
-    expect(dismissAlert.mock.calls.length).toBe(0);
-  });
-
-  it("when changing subpath location with message will not call dismissAlert", () => {
-    const dismissAlert = jest.fn();
-    const wrapper = shallow(
-      <RouteListener dismissAlert={dismissAlert} {...props} />
-    );
-    wrapper.setProps({
-      message: { not: "empty" },
-      location: { pathname: "/test/different" }
+      location: { pathname: "/different" },
     });
     expect(dismissAlert.mock.calls.length).toBe(0);
   });