@truedat/dd 4.33.7 → 4.34.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [4.33.8] 2021-11-29
+
+### Fixed
+
+- [TD-4353] Fix Bulk update
+
 ## [4.33.5] 2021-11-28
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@truedat/dd",
-  "version": "4.33.7",
+  "version": "4.34.0",
   "description": "Truedat Web Data Dictionary",
   "sideEffects": false,
   "jsnext:main": "src/index.js",
@@ -34,7 +34,7 @@
     "@testing-library/jest-dom": "^5.14.1",
     "@testing-library/react": "^12.0.0",
     "@testing-library/user-event": "^13.2.1",
-    "@truedat/test": "4.33.7",
+    "@truedat/test": "4.34.0",
     "babel-jest": "^27.0.6",
     "babel-plugin-dynamic-import-node": "^2.3.3",
     "babel-plugin-lodash": "^3.3.4",
@@ -84,9 +84,9 @@
     ]
   },
   "dependencies": {
-    "@truedat/auth": "4.33.7",
-    "@truedat/core": "4.33.7",
-    "@truedat/df": "4.33.7",
+    "@truedat/auth": "4.34.0",
+    "@truedat/core": "4.34.0",
+    "@truedat/df": "4.34.0",
     "lodash": "^4.17.21",
     "path-to-regexp": "^1.7.0",
     "prop-types": "^15.7.2",
@@ -107,5 +107,5 @@
     "react-dom": ">= 16.8.6 < 17",
     "semantic-ui-react": ">= 0.88.2 < 2.1"
   },
-  "gitHead": "417ace837309731f004f2d56371df420e82a4380"
+  "gitHead": "b1fe4f405c612bd342510cf34e8e9d89de0280fb"
 }

package/src/components/GraphLoader.js

@@ -9,7 +9,7 @@ import { fetchGraph, clearGraph, cancelPoll } from "../routines";
 export const GraphLoader = ({ clearGraph, fetchGraph, graph, cancelPoll }) => {
   const match = useRouteMatch(GRAPH);
   const id = _.path("params.id")(match);
-  
+
   const history = useHistory();
 
   useEffect(() => {
@@ -26,7 +26,7 @@ export const GraphLoader = ({ clearGraph, fetchGraph, graph, cancelPoll }) => {
         fetchGraph({ id });
       }
     }
-  }, [/*graph, *//*fetchGraph,*/ id]);
+  }, [/*graph, */ /*fetchGraph,*/ id]);
 
   useEffect(() => {
     return () => {
@@ -52,4 +52,3 @@ const mapStateToProps = ({ graph }) => ({ graph });
 export default connect(mapStateToProps, { clearGraph, fetchGraph, cancelPoll })(
   GraphLoader
 );
-

package/src/components/LineageEvents.js

@@ -1,7 +1,7 @@
 import React from "react";
 import { Header, Icon, Segment, Dimmer, Loader } from "semantic-ui-react";
 import { FormattedMessage } from "react-intl";
-import LineageEventsTable from "./LineageEventsTable"
+import LineageEventsTable from "./LineageEventsTable";
 
 export const LineageEvents = () => (
   <Segment>

package/src/components/LineageEventsTable.js

@@ -3,7 +3,7 @@ import React, { useEffect } from "react";
 import PropTypes from "prop-types";
 import { FormattedMessage } from "react-intl";
 import { Header, Icon, Table } from "semantic-ui-react";
-import { useHistory, } from "react-router-dom";
+import { useHistory } from "react-router-dom";
 import { connect } from "react-redux";
 import { useIntl } from "react-intl";
 import { columnDecorator } from "@truedat/core/services";
@@ -11,8 +11,11 @@ import { linkTo } from "@truedat/core/routes";
 import { lineageEventColumns as columns } from "../selectors/getLineageEventColumns";
 import { fetchLineageEvents } from "../routines";
 
-
-export const LineageEventsTable = ({ fetchLineageEvents, lineageEvents, lineageEventsLoading }) => {
+export const LineageEventsTable = ({
+  fetchLineageEvents,
+  lineageEvents,
+  lineageEventsLoading,
+}) => {
   const { formatMessage } = useIntl();
   const history = useHistory();
 

package/src/components/LineageForm.js

@@ -58,7 +58,7 @@ export const LineageForm = ({
     _.isNil(levels)
       ? createGraph({ ids: selectedResources, type })
       : createGraph({ ids: selectedResources, type, levels });
-  }
+  };
   const disabled = _.isEmpty(selectedResources);
   useEffect(() => {
     setOpen(!graphLoading && _.isEmpty(graph));

package/src/components/LineageGraph.js

@@ -98,7 +98,8 @@ export const LineageGraph = ({
                   id:
                     _.toLower(loadingInfo?.data?.status) === "already_started"
                       ? "lineage.graph.loading.background.existing"
-                      : _.toLower(loadingInfo?.data?.status) === "just_started" &&
+                      : _.toLower(loadingInfo?.data?.status) ===
+                          "just_started" &&
                         "lineage.graph.loading.background.launched",
                 })}
               </p>
@@ -182,4 +183,6 @@ const mapStateToProps = (
   };
 };
 
-export default connect(mapStateToProps, { clearNode, cancelPoll })(LineageGraph);
+export default connect(mapStateToProps, { clearNode, cancelPoll })(
+  LineageGraph
+);

package/src/components/LineageRoutes.js

@@ -3,7 +3,7 @@ import { Route } from "react-router-dom";
 import { Unauthorized } from "@truedat/core/components";
 import { useAuthorized } from "@truedat/core/hooks";
 import { GRAPH, GRAPHS, LINEAGE_EVENTS, SAMPLE } from "@truedat/core/routes";
-import LineageEvents from "./LineageEvents"
+import LineageEvents from "./LineageEvents";
 import LineageGraph from "./LineageGraph";
 import LineageSample from "./LineageSample";
 import LineageForm from "./LineageForm";

package/src/components/StructuresBulkUpdate.js

@@ -15,7 +15,6 @@ import { FormattedMessage } from "react-intl";
 import { ConfirmModal, HistoryBackButton } from "@truedat/core/components";
 import { selectTemplate } from "@truedat/df/routines";
 import { applyTemplateWithoutDefaults } from "@truedat/df/utils";
-import { composeProviders } from "redux-saga-test-plan/providers";
 import { compose } from "redux";
 import { withRouter } from "react-router-dom";
 import { bulkUpdateStructures } from "../routines";
@@ -82,7 +81,7 @@ export class StructuresBulkUpdate extends React.Component {
     template: PropTypes.object,
     templateId: PropTypes.number,
     templatesLoaded: PropTypes.bool,
-    location: PropTypes.object,
+    canAutoPublish: PropTypes.bool,
   };
 
   state = initialState;
@@ -123,15 +122,24 @@ export class StructuresBulkUpdate extends React.Component {
   };
 
   render() {
-    const { loading, template, templateId, structureCount, templatesLoaded } =
-      this.props;
+    const {
+      loading,
+      template,
+      templateId,
+      structureCount,
+      templatesLoaded,
+      canAutoPublish,
+      structureActiveFilters,
+    } = this.props;
     const { content } = this.state;
     const disableAction = !template || _.isEmpty(template);
 
     return templatesLoaded ? (
       <Container as={Segment} text>
         <TemplateLoader />
-        {!_.isEmpty(template) && structureCount != 0 ? (
+        {!_.isEmpty(template) &&
+        structureCount != 0 &&
+        _.has("type.raw")(structureActiveFilters) ? (
           <>
             <Header as="h2">
               <Icon name="book" />
@@ -161,10 +169,7 @@ export class StructuresBulkUpdate extends React.Component {
                 />
               )}
               <ConfirmModal
-                actions={actions(
-                  this.handleSubmit,
-                  this.props.location?.state.canAutoPublish
-                )}
+                actions={actions(this.handleSubmit, canAutoPublish)}
                 trigger={
                   <Button
                     primary
@@ -231,6 +236,8 @@ const mapStateToProps = ({
   structures,
   structureTypes,
   structureTypesLoading,
+  userDomains,
+  structureActiveFilters,
 }) => {
   const structure = _.head(structures);
   const templateId = findTemplate(structureTypes, templates, structure);
@@ -242,6 +249,13 @@ const mapStateToProps = ({
     structureCount,
     structureTypesLoaded: !structureTypesLoading,
     templatesLoaded: !templatesLoading && !_.isEmpty(templates),
+    userDomains,
+    canAutoPublish: _.any(
+      ({ domains, permission }) =>
+        !_.isEmpty(domains) &&
+        ["publish_structure_note_from_draft"].includes(permission)
+    )(userDomains),
+    structureActiveFilters,
   };
 };
 

package/src/components/StructuresRoutes.js

@@ -88,6 +88,13 @@ const AuthorizedRoutes = () => {
         path={STRUCTURES_BULK_UPDATE}
         render={() => (
           <>
+            <UserDomainsLoader
+              permissions={[
+                "publish_structure_note_from_draft",
+                "edit_structure_note",
+                "create_structure_note",
+              ]}
+            />
             <StructureCrumbs
               name={formatMessage({ id: "structures.bulkUpdate.crumbs.top" })}
             />

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

@@ -3,7 +3,6 @@ import { render } from "@truedat/test/render";
 import { LineageEventsTable } from "../LineageEventsTable";
 import en from "../../messages/en";
 
-
 describe("<LineageEventsTable />", () => {
   const lineageEvents = [
     {

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

@@ -17,6 +17,7 @@ describe("<StructuresBulkUpdate />", () => {
     templatesLoaded: true,
     structureTypes: [{ name: "doc", template_id: 1 }],
     structureTypesLoaded: true,
+    structureActiveFilters: { "type.raw": "booo" },
   };
 
   it("matches the latest snapshot", () => {
@@ -136,6 +137,7 @@ describe("<StructuresBulkUpdate />", () => {
       selectedDomain: { id: 1 },
       templatesLoaded: true,
       bulkUpdateStructures: jest.fn(),
+      structureActiveFilters: { "type.raw": "booo" },
     };
     const wrapper = shallow(<StructuresBulkUpdate {...props} />);
     wrapper.setState({

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

@@ -250,72 +250,76 @@ exports[`<GrantRequest /> matches the latest snapshot with approve actions 1`] =
                     filters
                   </div>
                 </h3>
-                <table
-                  class="ui very basic table"
+                <div
+                  class="ui segment"
                 >
-                  <thead
-                    class=""
+                  <table
+                    class="ui very basic table"
                   >
-                    <tr
+                    <thead
                       class=""
                     >
-                      <th
+                      <tr
                         class=""
                       >
-                        field
-                      </th>
-                      <th
-                        class=""
-                      >
-                        operator
-                      </th>
-                      <th
-                        class=""
-                      >
-                        values
-                      </th>
-                    </tr>
-                  </thead>
-                  <tbody
-                    class=""
-                  >
-                    <tr
+                        <th
+                          class=""
+                        >
+                          field
+                        </th>
+                        <th
+                          class=""
+                        >
+                          operator
+                        </th>
+                        <th
+                          class=""
+                        >
+                          values
+                        </th>
+                      </tr>
+                    </thead>
+                    <tbody
                       class=""
                     >
-                      <td
-                        class="five wide"
+                      <tr
+                        class=""
                       >
-                        <a
-                          href="/structures/1"
+                        <td
+                          class="five wide"
+                        >
+                          <a
+                            href="/structures/1"
+                          >
+                            <span
+                              class="highlighted"
+                            >
+                              "foo"
+                            </span>
+                          </a>
+                        </td>
+                        <td
+                          class="five wide"
                         >
                           <span
                             class="highlighted"
                           >
-                            "foo"
+                            "eq"
                           </span>
-                        </a>
-                      </td>
-                      <td
-                        class="five wide"
-                      >
-                        <span
-                          class="highlighted"
-                        >
-                          "eq"
-                        </span>
-                      </td>
-                      <td
-                        class="five wide"
-                      >
-                        <span
-                          class="highlighted"
+                        </td>
+                        <td
+                          class="five wide"
                         >
-                          "2021-10-05 08:15:00 "
-                        </span>
-                      </td>
-                    </tr>
-                  </tbody>
-                </table>
+                          <span
+                            class="highlighted"
+                          >
+                            "2021-10-05 08:15:00 "
+                          </span>
+                        </td>
+                      </tr>
+                    </tbody>
+                  </table>
+                </div>
               </div>
             </div>
           </div>
@@ -512,72 +516,76 @@ exports[`<GrantRequest /> matches the latest snapshot with failed status 1`] = `
                     filters
                   </div>
                 </h3>
-                <table
-                  class="ui very basic table"
+                <div
+                  class="ui segment"
                 >
-                  <thead
-                    class=""
+                  <table
+                    class="ui very basic table"
                   >
-                    <tr
+                    <thead
                       class=""
                     >
-                      <th
+                      <tr
                         class=""
                       >
-                        field
-                      </th>
-                      <th
-                        class=""
-                      >
-                        operator
-                      </th>
-                      <th
-                        class=""
-                      >
-                        values
-                      </th>
-                    </tr>
-                  </thead>
-                  <tbody
-                    class=""
-                  >
-                    <tr
+                        <th
+                          class=""
+                        >
+                          field
+                        </th>
+                        <th
+                          class=""
+                        >
+                          operator
+                        </th>
+                        <th
+                          class=""
+                        >
+                          values
+                        </th>
+                      </tr>
+                    </thead>
+                    <tbody
                       class=""
                     >
-                      <td
-                        class="five wide"
+                      <tr
+                        class=""
                       >
-                        <a
-                          href="/structures/1"
+                        <td
+                          class="five wide"
+                        >
+                          <a
+                            href="/structures/1"
+                          >
+                            <span
+                              class="highlighted"
+                            >
+                              "foo"
+                            </span>
+                          </a>
+                        </td>
+                        <td
+                          class="five wide"
                         >
                           <span
                             class="highlighted"
                           >
-                            "foo"
+                            "eq"
                           </span>
-                        </a>
-                      </td>
-                      <td
-                        class="five wide"
-                      >
-                        <span
-                          class="highlighted"
-                        >
-                          "eq"
-                        </span>
-                      </td>
-                      <td
-                        class="five wide"
-                      >
-                        <span
-                          class="highlighted"
+                        </td>
+                        <td
+                          class="five wide"
                         >
-                          "2021-10-05 08:15:00 "
-                        </span>
-                      </td>
-                    </tr>
-                  </tbody>
-                </table>
+                          <span
+                            class="highlighted"
+                          >
+                            "2021-10-05 08:15:00 "
+                          </span>
+                        </td>
+                      </tr>
+                    </tbody>
+                  </table>
+                </div>
               </div>
             </div>
           </div>

package/src/reducers/__tests__/lineageEventsLoading.spec.js

@@ -9,13 +9,15 @@ describe("reducers: lineageEventsLoading", () => {
   });
 
   it("returns to initial state on fetchLineageEvents.FULFILL action", () => {
-    expect(lineageEventsLoading(fooState, { type: fetchLineageEvents.FULFILL })).toBe(
-      false
-    );
+    expect(
+      lineageEventsLoading(fooState, { type: fetchLineageEvents.FULFILL })
+    ).toBe(false);
   });
 
   it("returns true on fetchLineageEvents.REQUEST action", () => {
-    expect(lineageEventsLoading(fooState, { type: fetchLineageEvents.REQUEST })).toBe(true);
+    expect(
+      lineageEventsLoading(fooState, { type: fetchLineageEvents.REQUEST })
+    ).toBe(true);
   });
 
   it("should ignore unhandled actions", () => {

package/src/reducers/graph.js

@@ -4,7 +4,7 @@ import {
   clearStructure,
   fetchGraph,
   fetchStructure,
-  createGraph
+  createGraph,
 } from "../routines";
 
 export const initialState = {};

package/src/sagas/__tests__/createGraph.spec.js

@@ -51,7 +51,7 @@ describe("sagas: createGraphRequestSaga", () => {
         .next()
         .all([
           takeLatest(createGraph.TRIGGER, createGraphSaga),
-          takeLatest(excludeNode.TRIGGER, createGraphSaga)
+          takeLatest(excludeNode.TRIGGER, createGraphSaga),
         ])
         .finish()
         .isDone();
@@ -103,7 +103,6 @@ describe("sagas: createGraphSaga", () => {
   });
 
   it("should fork a polling task when uuids are requested and a 202 (accepted) graph hash response is returned. Then, wait for task cancellation.", () => {
-
     expect(() => {
       testSaga(createGraphSaga)
         .next()
@@ -221,7 +220,7 @@ describe("generator: polling", () => {
         .next()
         .cancelled()
         .next(true)
-        .put(createGraph.failure(POLLING_CANCELLED))
+        .put(createGraph.failure(POLLING_CANCELLED));
     }).not.toThrow();
   });
 });

package/src/sagas/createGraph.js

@@ -120,6 +120,6 @@ export function* polling(graphAccepted) {
 export function* createGraphRequestSaga() {
   yield all([
     takeLatest(createGraph.TRIGGER, createGraphSaga),
-    takeLatest(excludeNode.TRIGGER, createGraphSaga)
+    takeLatest(excludeNode.TRIGGER, createGraphSaga),
   ]);
 }

package/src/selectors/__tests__/getGrantsColumns.spec.js

@@ -1,6 +1,7 @@
 import _ from "lodash/fp";
 import React from "react";
 import { render } from "@truedat/test/render";
+import moment from "moment";
 import {
   defaultGrantsColumns,
   defaultGrantsTableColumns,

package/src/components/StructuresUpdateButton.js

@@ -1,45 +0,0 @@
-import React from "react";
-import PropTypes from "prop-types";
-import { connect } from "react-redux";
-import { Button, Popup } from "semantic-ui-react";
-import { Link } from "react-router-dom";
-import { FormattedMessage } from "react-intl";
-import { useAuthorized } from "@truedat/core/hooks";
-import { STRUCTURES_BULK_UPDATE } from "@truedat/core/routes";
-import { bulkUpdateDisabledSelector } from "../selectors";
-
-export const StructuresUpdateButton = ({ disabled, canAutoPublish }) => {
-  const authorized = useAuthorized();
-
-  return authorized ? (
-    <Popup
-      content={<FormattedMessage id="structures.actions.bulk_update.popup" />}
-      position="top right"
-      trigger={
-        <span className="button-update">
-          <Link
-            className={`ui secondary button ${disabled ? "disabled" : ""}`}
-            role="button"
-            to={{
-              pathname: STRUCTURES_BULK_UPDATE,
-              state: { canAutoPublish },
-            }}
-          >
-            <FormattedMessage id="actions.update" />
-          </Link>
-        </span>
-      }
-    />
-  ) : null;
-};
-
-StructuresUpdateButton.propTypes = {
-  disabled: PropTypes.bool,
-  canAutoPublish: PropTypes.bool,
-};
-
-const mapStateToProps = (state) => ({
-  disabled: bulkUpdateDisabledSelector(state),
-});
-
-export default connect(mapStateToProps)(StructuresUpdateButton);