diff --git a/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypeCollapse/Collapse/index.js b/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypeCollapse/Collapse/index.js index abc78a6161..dcca315763 100644 --- a/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypeCollapse/Collapse/index.js +++ b/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypeCollapse/Collapse/index.js @@ -11,7 +11,7 @@ import Wrapper from './Wrapper'; import RowLabel from '../../RowLabel'; import { getCheckboxState, removeConditionKeyFromData } from '../../utils'; -const Collapse = ({ availableActions, isActive, isGrey, name, onClickToggle, pathToData }) => { +const Collapse = ({ availableActions, isActive, isGrey, label, onClickToggle, pathToData }) => { const { modifiedData, onChangeParentCheckbox, @@ -36,7 +36,7 @@ const Collapse = ({ availableActions, isActive, isGrey, name, onClickToggle, pat @@ -58,6 +59,7 @@ ContentTypeCollapse.propTypes = { contentTypeName: PropTypes.string.isRequired, index: PropTypes.number.isRequired, isActive: PropTypes.bool.isRequired, + label: PropTypes.string.isRequired, onClickToggleCollapse: PropTypes.func.isRequired, pathToData: PropTypes.string.isRequired, properties: PropTypes.array.isRequired, diff --git a/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypeCollapses/index.js b/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypeCollapses/index.js index 07c8629566..883370f625 100644 --- a/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypeCollapses/index.js +++ b/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypeCollapses/index.js @@ -14,17 +14,18 @@ const ContentTypeCollapses = ({ actions, pathToData, subjects }) => { [collapseToOpen] ); - return Object.keys(subjects).map((subject, index) => { + return subjects.map(({ uid, label, properties }, index) => { return ( ); }); @@ -32,13 +33,19 @@ const ContentTypeCollapses = ({ actions, pathToData, subjects }) => { ContentTypeCollapses.defaultProps = { actions: [], - subjects: {}, + subjects: [], }; ContentTypeCollapses.propTypes = { actions: PropTypes.array.isRequired, pathToData: PropTypes.string.isRequired, - subjects: PropTypes.object, + subjects: PropTypes.arrayOf( + PropTypes.shape({ + uid: PropTypes.string.isRequired, + label: PropTypes.string.isRequired, + properties: PropTypes.array.isRequired, + }) + ), }; export default memo(ContentTypeCollapses); diff --git a/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypes/index.js b/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypes/index.js index 753f09424b..1e3823f9d5 100644 --- a/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypes/index.js +++ b/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/ContentTypes/index.js @@ -20,7 +20,13 @@ ContentTypes.propTypes = { kind: PropTypes.string.isRequired, layout: PropTypes.shape({ actions: PropTypes.array, - subjects: PropTypes.object, + subjects: PropTypes.arrayOf( + PropTypes.shape({ + uid: PropTypes.string.isRequired, + label: PropTypes.string.isRequired, + properties: PropTypes.array.isRequired, + }) + ), }).isRequired, }; diff --git a/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/temp/fakeData.js b/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/temp/fakeData.js index ad1a018faf..bab278082b 100644 --- a/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/temp/fakeData.js +++ b/packages/strapi-admin/ee/admin/containers/Roles/CreatePage/temp/fakeData.js @@ -283,8 +283,10 @@ const data = { singleTypes: {}, // same format as under, collectionTypes: { - subjects: { - address: { + subjects: [ + { + uid: 'address', + label: 'Address', properties: [ { label: 'Fields', @@ -315,7 +317,9 @@ const data = { }, ], }, - restaurant: { + { + uid: 'restaurant', + label: 'Restaurant', properties: [ { label: 'Fields', @@ -384,10 +388,12 @@ const data = { }, ], }, - test: { + { + uid: 'test', + label: 'test', properties: [], }, - }, + ], actions: [ { label: 'Create',