feat: Taxonomy export menu [FC-0036] (#645)

* feat: System-defined tooltip added
* feat: Taxonomy card menu added. Export menu item added
* feat: Modal for export taxonomy
* feat: Connect with export API
* test: Tests for API and selectors
* feat: Use windows.location.href to call the export endpoint
* test: ExportModal.test added
* style: Delete unnecessary code
* docs: README updated with taxonomy feature
* style: TaxonomyCard updated to a better code style
* style: injectIntl replaced by useIntl on taxonomy pages and components
* refactor: Move and rename taxonomy UI components to match 0002 ADR
* refactor: Move api to data to match with 0002 ADR
* test: Refactor ExportModal tests
* chore: Fix validations
* chore: Lint
* refactor: Moving hooks to apiHooks
* style: Nit on return null

---------

Co-authored-by: Rômulo Penido <[email protected]>
Co-authored-by: Christofer Chavez <[email protected]>

Author: 3 people

README.rst

@@ -250,6 +250,25 @@ Requirements
   * ``contentstore.new_studio_mfe.use_new_export_page``: this feature flag will change the CMS to link to the new export page.
   * ``contentstore.new_studio_mfe.use_new_import_page``: this feature flag will change the CMS to link to the new import page.
 
+Feature: Tagging/Taxonomy Pages
+================================
+
+.. image:: ./docs/readme-images/feature-tagging-taxonomy-pages.png
+
+Requirements
+------------
+
+* ``edx-platform`` Waffle flags:
+
+  * ``contentstore.new_studio_mfe.use_tagging_taxonomy_list_page``: this feature flag must be enabled.
+
+Configuration
+-------------
+
+In additional to the standard settings, the following local configuration items are required:
+
+* ``ENABLE_TAGGING_TAXONOMY_PAGES``: must be enabled in order to actually present the new Tagging/Taxonomy pages.
+
 
 Developing
 **********

docs/readme-images/feature-tagging-taxonomy-pages.png

@@ -18,4 +18,5 @@
 @import "course-updates/CourseUpdates";
 @import "export-page/CourseExportPage";
 @import "import-page/CourseImportPage";
+@import "taxonomy/taxonomy-card/TaxonomyCard";
 @import "files-and-videos";

src/index.scss

@@ -5,15 +5,16 @@ import {
   DataTable,
   Spinner,
 } from '@edx/paragon';
-import { injectIntl, intlShape } from '@edx/frontend-platform/i18n';
 import { StudioFooter } from '@edx/frontend-component-footer';
+import { useIntl } from '@edx/frontend-platform/i18n';
 import Header from '../header';
 import SubHeader from '../generic/sub-header/SubHeader';
 import messages from './messages';
-import TaxonomyCard from './TaxonomyCard';
-import { useTaxonomyListDataResponse, useIsTaxonomyListDataLoaded } from './api/hooks/selectors';
+import TaxonomyCard from './taxonomy-card';
+import { useTaxonomyListDataResponse, useIsTaxonomyListDataLoaded } from './data/apiHooks';
 
-const TaxonomyListPage = ({ intl }) => {
+const TaxonomyListPage = () => {
+  const intl = useIntl();
   const useTaxonomyListData = () => {
     const taxonomyListData = useTaxonomyListDataResponse();
     const isLoaded = useIsTaxonomyListDataLoaded();
@@ -97,8 +98,6 @@ const TaxonomyListPage = ({ intl }) => {
   );
 };
 
-TaxonomyListPage.propTypes = {
-  intl: intlShape.isRequired,
-};
+TaxonomyListPage.propTypes = {};
 
-export default injectIntl(TaxonomyListPage);
+export default TaxonomyListPage;

src/taxonomy/TaxonomyCard.jsx

@@ -7,11 +7,11 @@ import { act, render } from '@testing-library/react';
 import initializeStore from '../store';
 
 import TaxonomyListPage from './TaxonomyListPage';
-import { useTaxonomyListDataResponse, useIsTaxonomyListDataLoaded } from './api/hooks/selectors';
+import { useTaxonomyListDataResponse, useIsTaxonomyListDataLoaded } from './data/apiHooks';
 
 let store;
 
-jest.mock('./api/hooks/selectors', () => ({
+jest.mock('./data/apiHooks', () => ({
   useTaxonomyListDataResponse: jest.fn(),
   useIsTaxonomyListDataLoaded: jest.fn(),
 }));

src/taxonomy/TaxonomyListPage.jsx

@@ -0,0 +1,2 @@
+// eslint-disable-next-line import/prefer-default-export
+export { default as taxonomyListMock } from './taxonomyListMock';

src/taxonomy/TaxonomyListPage.test.jsx

@@ -0,0 +1,50 @@
+module.exports = {
+  next: null,
+  previous: null,
+  count: 4,
+  numPages: 1,
+  currentPage: 1,
+  start: 0,
+  results: [
+    {
+      id: -2,
+      name: 'Content Authors',
+      description: 'Allows tags for any user ID created on the instance.',
+      enabled: true,
+      allowMultiple: false,
+      allowFreeText: false,
+      systemDefined: true,
+      visibleToAuthors: false,
+    },
+    {
+      id: -1,
+      name: 'Languages',
+      description: 'lang lang lang lang lang lang lang lang',
+      enabled: true,
+      allowMultiple: false,
+      allowFreeText: false,
+      systemDefined: true,
+      visibleToAuthors: true,
+    },
+    {
+      id: 1,
+      name: 'Taxonomy',
+      description: 'This is a Description',
+      enabled: true,
+      allowMultiple: false,
+      allowFreeText: false,
+      systemDefined: false,
+      visibleToAuthors: true,
+    },
+    {
+      id: 2,
+      name: 'Taxonomy long long long long long long long long long long long long long long long long long long long',
+      description: 'This is a Description long lon',
+      enabled: true,
+      allowMultiple: false,
+      allowFreeText: false,
+      systemDefined: false,
+      visibleToAuthors: true,
+    },
+  ],
+};