@truedat/dq 4.36.9 → 4.37.3

package/CHANGELOG.md

@@ -1,5 +1,31 @@
 # Changelog
 
+## [4.37.3] 2022-02-03
+
+### Added
+
+- [TD-2929] Added support for `UserSearchFilters` on `Rules` and `RuleImplementations`
+
+## [4.37.2] 2022-02-02
+
+### Added
+
+- [TD-4294] Rule audit tab
+
+## [4.37.1] 2022-02-02
+
+### Changed
+
+- [TD-4152] Include aditional information on implementations download
+
+## [4.37.0] 2022-01-25
+
+### Changed
+
+- [TD-2564] `DatasetForm` now has a configurable `defaultFilters` prop
+- Simplified `DatasetForm` and `FiltersField` components by removing obscure
+  code for installation-specific custom filters
+
 ## [4.36.4] 2022-01-19
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@truedat/dq",
-  "version": "4.36.9",
+  "version": "4.37.3",
   "description": "Truedat Web Data Quality Module",
   "sideEffects": false,
   "jsnext:main": "src/index.js",
@@ -82,8 +82,8 @@
   },
   "dependencies": {
     "@apollo/client": "^3.4.10",
-    "@truedat/core": "4.36.9",
-    "@truedat/df": "4.36.9",
+    "@truedat/core": "4.37.3",
+    "@truedat/df": "4.37.3",
     "axios": "^0.19.2",
     "graphql": "^15.5.3",
     "path-to-regexp": "^1.7.0",
@@ -103,5 +103,5 @@
     "react-dom": ">= 16.8.6 < 17",
     "semantic-ui-react": ">= 0.88.2 < 2.1"
   },
-  "gitHead": "4b56a9ae46e2454719a2ae31022bc0d700a65dd3"
+  "gitHead": "1c418784d34e171a6d62dd7fab3dfd05400998ba"
 }

package/src/components/ImplementationsRoutes.js

@@ -8,6 +8,9 @@ import RuleImplementationsLoader from "./RuleImplementationsLoader";
 const TemplatesLoader = React.lazy(() =>
   import("@truedat/df/templates/components/TemplatesLoader")
 );
+const UserSearchFiltersLoader = React.lazy(() =>
+  import("@truedat/dd/components/UserSearchFiltersLoader")
+);
 
 const ImplementationsRoutes = () => (
   <Route
@@ -17,6 +20,7 @@ const ImplementationsRoutes = () => (
         <RuleImplementationsLoader />
         <ImplementationFiltersLoader />
         <TemplatesLoader scope="qe" />
+        <UserSearchFiltersLoader scope="rule_implementation" />
         <RuleImplementations exact />
       </>
     )}

package/src/components/RuleEventRow.js

@@ -0,0 +1,47 @@
+import _ from "lodash/fp";
+import React from "react";
+import PropTypes from "prop-types";
+import { Feed } from "semantic-ui-react";
+import { useIntl } from "react-intl";
+import Moment from "react-moment";
+
+export const RuleEventRow = ({ user, user_name, ts, payload }) => {
+  const { formatMessage, locale } = useIntl();
+  const userName = _.propOr(user_name, "user_name")(user);
+
+  return _.isEmpty(payload) ? null : (
+    <Feed.Event>
+      <Feed.Content>
+        <Feed.Summary>
+          <Feed.User>{userName}</Feed.User>{" "}
+          <Feed.Date>
+            <Moment locale={locale} date={ts} format="YYYY-MM-DD HH:mm" />
+          </Feed.Date>
+        </Feed.Summary>
+        {payload.map(({ field, action, value }, i) => (
+          <Feed.Extra
+            key={i}
+            text
+            content={formatMessage(
+              {
+                id: `rules.events.action_${action}${
+                  typeof value === "undefined" || _.isObject(value) ? "" : "_to"
+                }`,
+              },
+              [field, JSON.stringify(value)]
+            )}
+          />
+        ))}
+      </Feed.Content>
+    </Feed.Event>
+  );
+};
+
+RuleEventRow.propTypes = {
+  user: PropTypes.object,
+  user_name: PropTypes.string,
+  ts: PropTypes.string,
+  payload: PropTypes.array,
+};
+
+export default RuleEventRow;

package/src/components/RuleEvents.js

@@ -0,0 +1,31 @@
+import React from "react";
+import PropTypes from "prop-types";
+import { Feed, Segment } from "semantic-ui-react";
+import { connect } from "react-redux";
+import { getParsedEvents } from "../selectors";
+import { RuleEventRow } from "./RuleEventRow";
+
+export const RuleEvents = ({ events, eventsLoading }) => {
+  return eventsLoading ? null : (
+    <Segment attached="bottom">
+      <Feed size="small">
+        {events.map((e, i) => (
+          <RuleEventRow key={i} {...e} />
+        ))}
+      </Feed>
+    </Segment>
+  );
+};
+RuleEvents.propTypes = {
+  events: PropTypes.array.isRequired,
+  eventsLoading: PropTypes.bool,
+};
+
+const mapStateToProps = (state) => {
+  return {
+    events: getParsedEvents(state),
+    eventsLoading: state.eventsLoading,
+  };
+};
+
+export default connect(mapStateToProps)(RuleEvents);

package/src/components/RuleImplementationSelectedFilters.js

@@ -4,6 +4,11 @@ import { connect } from "react-redux";
 import { injectIntl } from "react-intl";
 import { makeOption } from "@truedat/core/services/i18n";
 import { SelectedFilters } from "@truedat/core/components";
+import {
+  applyUserSearchFilter,
+  deleteUserSearchFilter,
+  saveUserSearchFilters,
+} from "@truedat/dd/routines";
 import {
   closeImplementationFilter,
   openImplementationFilter,
@@ -30,23 +35,36 @@ export const mapStateToProps = (state, ownProps) => {
     _.map(makeOption(translations(formatMessage), selectedFilter))
   )(state);
 
+  const {
+    selectedUserSearchFilter: selectedUserFilter,
+    ruleImplementationActiveFilters: activeFilters,
+    userSearchFilters: userFilters,
+  } = state;
+
   return {
     selectedFilter,
     selectedFilters: getRuleImplementationSelectedFilters(state),
     selectedFilterActiveValues:
       getRuleImplementationSelectedFilterActiveValues(state),
     selectedFilterValues: i18nValues,
+    selectedUserFilter,
+    activeFilters,
+    userFilters,
+    userFilterScope: "rule_implementation",
   };
 };
 
 const mapDispatchToProps = (dispatch) =>
   bindActionCreators(
     {
+      applyUserFilter: applyUserSearchFilter,
       closeFilter: closeImplementationFilter,
+      deleteUserFilter: deleteUserSearchFilter,
       openFilter: openImplementationFilter,
       removeFilter: removeImplementationFilter,
       resetFilters: resetImplementationFilters,
       toggleFilterValue: toggleImplementationFilterValue,
+      saveFilters: saveUserSearchFilters,
     },
     dispatch
   );

package/src/components/RuleImplementationsDownload.js

@@ -10,7 +10,8 @@ const staticHeaderLabels = [
   "implementation_key",
   "implementation_type",
   "rule",
-  "template",
+  "rule_template",
+  "implementation_template",
   "goal",
   "minimum",
   "business_concept",

package/src/components/RuleRoutes.js

@@ -9,6 +9,7 @@ import { useAuthorized } from "@truedat/core/hooks";
 import {
   RULE,
   RULE_EDIT,
+  RULE_EVENTS,
   RULE_IMPLEMENTATION_RESULT_DETAILS,
   RULE_IMPLEMENTATION_RESULTS_DETAILS,
   RULE_IMPLEMENTATION,
@@ -32,6 +33,7 @@ import NewRule from "./NewRule";
 import NewRuleImplementation from "./NewRuleImplementation";
 import Rule from "./Rule";
 import RuleCrumbs from "./RuleCrumbs";
+import RuleEvents from "./RuleEvents";
 import RuleImplementation from "./RuleImplementation";
 import RuleImplementationEvents from "./RuleImplementationEvents";
 import RuleImplementationLoader from "./RuleImplementationLoader";
@@ -61,6 +63,12 @@ const ImplementationStructuresLoader = React.lazy(() =>
 const QualityTemplatesLoader = () => <TemplatesLoader scope="dq" />;
 const ImplementationTemplatesLoader = () => <TemplatesLoader scope="ri" />;
 
+const RuleEventsLoader = () => {
+  const match = useRouteMatch();
+  const id = _.path("params.id")(match);
+  return <EventsLoader resource_id={id} resource_type="rule" />;
+};
+
 const ImplementationEventsLoader = () => {
   const match = useRouteMatch();
   const id = _.path("params.implementation_id")(match);
@@ -135,6 +143,22 @@ const RuleRoutes = ({
                     </>
                   )}
                 />
+
+                <Route
+                  exact
+                  path={RULE_EVENTS}
+                  render={() => (
+                    <>
+                      <RuleCrumbs />
+                      <Segment>
+                        <RuleEventsLoader />
+                        {ruleLoaded && <Rule />}
+                        {ruleLoaded && <RuleEvents />}
+                      </Segment>
+                    </>
+                  )}
+                />
+
                 <Route
                   exact
                   path={RULE_IMPLEMENTATIONS}

package/src/components/RuleSelectedFilters.js

@@ -4,6 +4,11 @@ import { connect } from "react-redux";
 import { injectIntl } from "react-intl";
 import { makeOption } from "@truedat/core/services/i18n";
 import { SelectedFilters } from "@truedat/core/components";
+import {
+  applyUserSearchFilter,
+  deleteUserSearchFilter,
+  saveUserSearchFilters,
+} from "@truedat/dd/routines";
 import {
   closeRuleFilter,
   openRuleFilter,
@@ -30,22 +35,35 @@ export const mapStateToProps = (state, ownProps) => {
     _.map(makeOption(translations(formatMessage), selectedFilter))
   )(state);
 
+  const {
+    selectedUserSearchFilter: selectedUserFilter,
+    ruleActiveFilters: activeFilters,
+    userSearchFilters: userFilters,
+  } = state;
+
   return {
     selectedFilter,
     selectedFilters: getRuleSelectedFilters(state),
     selectedFilterActiveValues: getRuleSelectedFilterActiveValues(state),
     selectedFilterValues: i18nValues,
+    selectedUserFilter,
+    activeFilters,
+    userFilters,
+    userFilterScope: "rule",
   };
 };
 
 const mapDispatchToProps = (dispatch) =>
   bindActionCreators(
     {
+      applyUserFilter: applyUserSearchFilter,
       closeFilter: closeRuleFilter,
+      deleteUserFilter: deleteUserSearchFilter,
       openFilter: openRuleFilter,
       removeFilter: removeRuleFilter,
       resetFilters: resetRuleFilters,
       toggleFilterValue: toggleRuleFilterValue,
+      saveFilters: saveUserSearchFilters,
     },
     dispatch
   );

package/src/components/RuleTabs.js

@@ -6,7 +6,12 @@ import { Link } from "react-router-dom";
 import { connect } from "react-redux";
 import { FormattedMessage } from "react-intl";
 import { usePath } from "@truedat/core/hooks";
-import { RULE, RULE_IMPLEMENTATIONS, linkTo } from "@truedat/core/routes";
+import {
+  RULE,
+  RULE_IMPLEMENTATIONS,
+  RULE_EVENTS,
+  linkTo,
+} from "@truedat/core/routes";
 
 const RuleTabs = ({ rule }) => {
   const path = usePath();
@@ -22,12 +27,21 @@ const RuleTabs = ({ rule }) => {
       >
         <FormattedMessage id="tabs.dq.ruleImplementations" />
       </Menu.Item>
+      <Menu.Item
+        active={path === RULE_EVENTS}
+        as={Link}
+        to={linkTo.RULE_EVENTS({
+          id: rule.id,
+        })}
+      >
+        <FormattedMessage id="tabs.dq.rule.audit" />
+      </Menu.Item>
     </Menu>
   );
 };
 
 RuleTabs.propTypes = {
-  rule: PropTypes.object
+  rule: PropTypes.object,
 };
 
 const mapStateToProps = ({ rule }) => ({ rule });

package/src/components/RulesRoutes.js

@@ -13,13 +13,25 @@ const DomainsLoader = React.lazy(() =>
 const TemplatesLoader = React.lazy(() =>
   import("@truedat/df/templates/components/TemplatesLoader")
 );
+const UserSearchFiltersLoader = React.lazy(() =>
+  import("@truedat/dd/components/UserSearchFiltersLoader")
+);
 const QualityTemplatesLoader = () => <TemplatesLoader scope="dq" />;
 
 const RulesRoutes = () => (
   <>
     <Route path={RULES} component={RulesLoader} />
     <Route path={RULES} component={RuleFiltersLoader} />
-    <Route path={RULES} component={Rules} exact />
+    <Route
+      exact
+      path={RULES}
+      render={() => (
+        <>
+          <UserSearchFiltersLoader scope="rule" />
+          <Rules />
+        </>
+      )}
+    />
     <Switch>
       <Route
         exact

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

@@ -0,0 +1,33 @@
+import React from "react";
+import { render } from "@truedat/test/render";
+import { RuleEventRow } from "../RuleEventRow";
+
+describe("<RuleEventsRow />", () => {
+  const props = {
+    payload: [
+      {
+        field: "field1",
+        action: "changed",
+        value: "new value",
+      },
+    ],
+    user_name: "some.user@domain.tld",
+    user: {
+      email: "some.user@domain.tld",
+      full_name: "Some User",
+      id: 142,
+      user_name: "some.user@domain.tld",
+    },
+    ts: "2021-09-16T07:36:32.848603Z",
+  };
+  const renderOpts = {
+    messages: {
+      en: { "rules.events.action_changed_to": "Field {0} changed to: {1}" },
+    },
+  };
+
+  it("matches the latest snapshot", () => {
+    const { container } = render(<RuleEventRow {...props} />, renderOpts);
+    expect(container).toMatchSnapshot();
+  });
+});

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

@@ -0,0 +1,64 @@
+import React from "react";
+import { render } from "@truedat/test/render";
+import { RuleEvents } from "../RuleEvents";
+import { getParsedEvents } from "../../selectors/getParsedEvents";
+
+describe("<RuleEvents />", () => {
+  const events = [
+    {
+      id: 100217,
+      service: "td_dd",
+      resource_id: 777,
+      resource_type: "rule",
+      event: "rule_updated",
+      payload: {
+        domain_id: 140,
+      },
+      user_id: 142,
+      user_name: null,
+      user: {
+        email: "some.user@domain.tld",
+        full_name: "Some User",
+        id: 142,
+        user_name: "some.user@domain.tld",
+      },
+      ts: "2021-09-16T07:36:32.848603Z",
+    },
+    {
+      id: 100436,
+      service: "td_dd",
+      resource_id: 777,
+      resource_type: "rule",
+      event: "rule_updated",
+      payload: {
+        domain_id: 168,
+      },
+      user_id: 171,
+      user_name: null,
+      user: {
+        email: "some.user@domain.tld",
+        full_name: "Some User",
+        id: 171,
+        user_name: "some.user",
+      },
+      ts: "2021-09-16T09:31:03.061864Z",
+    },
+  ];
+  const renderOpts = {
+    messages: {
+      en: {
+        "rules.events.action_created": "Rule created",
+        "rules.events.action_changed": "Field {0} changed: {1}",
+        "rules.events.action_changed_to": "Field {0} changed to: {1}",
+        "rules.events.action_updated": "Rule updated",
+      },
+    },
+  };
+
+  const props = { events: getParsedEvents({ events }), eventsLoading: false };
+
+  it("matches the latest snapshot", () => {
+    const { container } = render(<RuleEvents {...props} />, renderOpts);
+    expect(container).toMatchSnapshot();
+  });
+});

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

@@ -24,9 +24,8 @@ describe("<RuleImplementationEvents />", () => {
   const renderOpts = {
     messages: {
       en: {
-        "ruleImplementations.events.field_restored": "implementation restored",
-        "ruleImplementations.events.field_deprecated":
-          "implementation deprecated",
+        "ruleImplementations.events.action_deprecated": "deprecated",
+        "ruleImplementations.events.action_restored": "restored",
       },
     },
   };

package/src/components/__tests__/__snapshots__/RuleEventRow.spec.js.snap

@@ -0,0 +1,38 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`<RuleEventsRow /> matches the latest snapshot 1`] = `
+<div>
+  <div
+    class="event"
+  >
+    <div
+      class="content"
+    >
+      <div
+        class="summary"
+      >
+        <a
+          class="user"
+        >
+          some.user@domain.tld
+        </a>
+         
+        <div
+          class="date"
+        >
+          <time
+            datetime="1631777792848"
+          >
+            2021-09-16 07:36
+          </time>
+        </div>
+      </div>
+      <div
+        class="text extra"
+      >
+        Field field1 changed to: "new value"
+      </div>
+    </div>
+  </div>
+</div>
+`;

package/src/components/__tests__/__snapshots__/RuleEvents.spec.js.snap

@@ -0,0 +1,78 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`<RuleEvents /> matches the latest snapshot 1`] = `
+<div>
+  <div
+    class="ui bottom attached segment"
+  >
+    <div
+      class="ui small feed"
+    >
+      <div
+        class="event"
+      >
+        <div
+          class="content"
+        >
+          <div
+            class="summary"
+          >
+            <a
+              class="user"
+            >
+              some.user@domain.tld
+            </a>
+             
+            <div
+              class="date"
+            >
+              <time
+                datetime="1631777792848"
+              >
+                2021-09-16 07:36
+              </time>
+            </div>
+          </div>
+          <div
+            class="text extra"
+          >
+            Field domain_id changed to: 140
+          </div>
+        </div>
+      </div>
+      <div
+        class="event"
+      >
+        <div
+          class="content"
+        >
+          <div
+            class="summary"
+          >
+            <a
+              class="user"
+            >
+              some.user
+            </a>
+             
+            <div
+              class="date"
+            >
+              <time
+                datetime="1631784663061"
+              >
+                2021-09-16 09:31
+              </time>
+            </div>
+          </div>
+          <div
+            class="text extra"
+          >
+            Field domain_id changed to: 168
+          </div>
+        </div>
+      </div>
+    </div>
+  </div>
+</div>
+`;

package/src/components/__tests__/__snapshots__/RuleImplementationEvents.spec.js.snap

@@ -34,7 +34,7 @@ exports[`<RuleImplementationEvents /> matches the latest snapshot 1`] = `
           <div
             class="text extra"
           >
-            ruleImplementations.events.action_restored
+            restored
           </div>
         </div>
       </div>
@@ -64,7 +64,7 @@ exports[`<RuleImplementationEvents /> matches the latest snapshot 1`] = `
           <div
             class="text extra"
           >
-            ruleImplementations.events.action_deprecated
+            deprecated
           </div>
         </div>
       </div>