@contentful/field-editor-reference 4.3.5 → 4.3.8

package/CHANGELOG.md

@@ -3,6 +3,24 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [4.3.8](https://github.com/contentful/field-editors/compare/@contentful/field-editor-reference@4.3.7...@contentful/field-editor-reference@4.3.8) (2022-06-15)
+
+### Bug Fixes
+
+- don't register onClick callbacks when isClickable is false ([#1163](https://github.com/contentful/field-editors/issues/1163)) ([fe6fc6b](https://github.com/contentful/field-editors/commit/fe6fc6b872acfde3b097c626d2eef81a7cc24193))
+
+## [4.3.7](https://github.com/contentful/field-editors/compare/@contentful/field-editor-reference@4.3.6...@contentful/field-editor-reference@4.3.7) (2022-06-07)
+
+### Bug Fixes
+
+- **reference:** avoid infinite getEntityScheduledActions calls ([#1159](https://github.com/contentful/field-editors/issues/1159)) ([620111b](https://github.com/contentful/field-editors/commit/620111bdca45a083369bdb42be8e93a16b670937))
+
+## [4.3.6](https://github.com/contentful/field-editors/compare/@contentful/field-editor-reference@4.3.5...@contentful/field-editor-reference@4.3.6) (2022-05-24)
+
+### Bug Fixes
+
+- **reference:** show all content types for reference CT filter [HOMER-836] ([#1152](https://github.com/contentful/field-editors/issues/1152)) ([fe1f38d](https://github.com/contentful/field-editors/commit/fe1f38d093649f31701ead308720a64ffaba44b0))
+
 ## [4.3.5](https://github.com/contentful/field-editors/compare/@contentful/field-editor-reference@4.3.4...@contentful/field-editor-reference@4.3.5) (2022-05-09)
 
 ### Bug Fixes

package/dist/field-editor-reference.cjs.development.js

@@ -1799,8 +1799,10 @@ var ScheduledIconWithTooltip = function ScheduledIconWithTooltip(_ref) {
         type: 'error',
         error: e
       });
-    });
-  }, [getEntityScheduledActions, entityType, entityId]);
+    }); // This should only be ever called once. Following the eslint hint to add used
+    // dependencies will cause page freeze (infinite loop)
+    // eslint-disable-next-line
+  }, []);
 
   if (status.type === 'loading' || status.type === 'error') {
     return null;
@@ -2363,7 +2365,9 @@ function useEditorPermissions(props) {
                 if (entityType === 'Entry') {
                   // Hardcoded `true` value following https://contentful.atlassian.net/browse/DANTE-486
                   // TODO: refine permissions check in order to account for tags in rules
-                  _canRead = readableContentTypes.length > 0 || true;
+                  // TODO: always show every content type (it's just a filter) to avoid people not seeing
+                  // their (partly limited) content types
+                  _canRead = true;
                   setCanLinkEntity(_canRead);
                 }
 
@@ -2564,12 +2568,15 @@ function WrappedEntryCard(props) {
     }, "Move to bottom") : null].filter(function (item) {
       return item;
     }) : [],
-    onClick: function onClick(e) {
+    onClick: // Providing an onClick handler messes up with some rich text
+    // features e.g. pressing ENTER on a card to add a new paragraph
+    // underneath. It's crucial not to pass a custom handler when
+    // isClickable is disabled which in the case of RT it's.
+    props.isClickable ? function (e) {
       e.preventDefault();
-      if (!props.isClickable) return;
       if (props.onClick) return props.onClick(e);
       props.onEdit && props.onEdit();
-    }
+    } : undefined
   });
 }
 WrappedEntryCard.defaultProps = defaultProps;
@@ -3100,11 +3107,14 @@ var WrappedAssetCard = function WrappedAssetCard(props) {
       testId: "schedule-icon"
     })),
     src: entityFile && entityFile.url ? size === 'small' ? entityFile.url + "?w=150&h=150&fit=thumb" : entityFile.url + "?h=300" : '',
-    onClick: function onClick(e) {
+    onClick: // Providing an onClick handler messes up with some rich text
+    // features e.g. pressing ENTER on a card to add a new paragraph
+    // underneath. It's crucial not to pass a custom handler when
+    // isClickable is disabled which in the case of RT it's.
+    isClickable ? function (e) {
       e.preventDefault();
-      if (!isClickable) return;
       onEdit && onEdit();
-    },
+    } : undefined,
     dragHandleRender: props.renderDragHandle,
     withDragHandle: !!props.renderDragHandle,
     actions: [].concat(renderActions({