🚩(frontend) version MIT only

We have some packages that are not MIT compatible, so if the env var MIT_ONLY is set to true, we don't build the application with features that are not MIT compatible. For the moment, it concerns only the export packages.

Author: AntoLC

.github/workflows/docker-hub.yml

@@ -79,7 +79,9 @@ jobs:
  context: .
  file: ./src/frontend/Dockerfile
  target: frontend-production
- build-args: DOCKER_USER=${{ env.DOCKER_USER }}:-1000
+ build-args: |
+ DOCKER_USER=${{ env.DOCKER_USER }}:-1000
+ MIT_ONLY=false
  push: ${{ github.event_name != 'pull_request' }}
  tags: ${{ steps.meta.outputs.tags }}
  labels: ${{ steps.meta.outputs.labels }}

CHANGELOG.md

@@ -20,6 +20,7 @@ and this project adheres to
 - 🌐(i18n) activate chinese and spanish languages #884
 - 🔧(backend) allow overwriting the data directory #893
 - ➕(backend) add `django-lasuite` dependency #839
+- 🚩(frontend) version MIT only #911
 
 ## Changed
 

docker-compose.yml

@@ -155,8 +155,7 @@ services:
  target: frontend-production
  args:
  API_ORIGIN: "http://localhost:8071"
- Y_PROVIDER_URL: "ws://localhost:4444"
- MEDIA_URL: "http://localhost:8083"
+ MIT_ONLY: "false"
  SW_DEACTIVATED: "true"
  image: impress:frontend-development
  ports:

src/frontend/Dockerfile

@@ -39,6 +39,9 @@ ENV NEXT_PUBLIC_API_ORIGIN=${API_ORIGIN}
 ARG SW_DEACTIVATED
 ENV NEXT_PUBLIC_SW_DEACTIVATED=${SW_DEACTIVATED}
 
+ARG MIT_ONLY
+ENV MIT_ONLY=${MIT_ONLY}
+
 RUN yarn build
 
 # ---- Front-end image ----

src/frontend/apps/impress/.env

@@ -1,2 +1,3 @@
+MIT_ONLY=true
 NEXT_PUBLIC_API_ORIGIN=
 NEXT_PUBLIC_SW_DEACTIVATED=

src/frontend/apps/impress/.env.development

@@ -1,2 +1,3 @@
+MIT_ONLY=false
 NEXT_PUBLIC_API_ORIGIN=http://localhost:8071
 NEXT_PUBLIC_SW_DEACTIVATED=true

src/frontend/apps/impress/next.config.js

@@ -18,6 +18,7 @@ const nextConfig = {
  generateBuildId: () => buildId,
  env: {
  NEXT_PUBLIC_BUILD_ID: buildId,
+ MIT_ONLY: process.env.MIT_ONLY,
  },
  webpack(config, { isServer }) {
  // Grab the existing rule that handles SVG imports

src/frontend/apps/impress/src/custom-next.d.ts

@@ -19,6 +19,7 @@ declare module '*.svg?url' {
 
 namespace NodeJS {
  interface ProcessEnv {
+ MIT_ONLY?: string;
  NEXT_PUBLIC_API_ORIGIN?: string;
  NEXT_PUBLIC_SW_DEACTIVATED?: string;
  }

src/frontend/apps/impress/src/features/docs/doc-header/__tests__/DocToolBox.spec.tsx

@@ -42,6 +42,7 @@ const doc = {
 
 beforeEach(() => {
  Analytics.clearAnalytics();
+ process.env.MIT_ONLY = 'false';
 });
 
 describe('DocToolBox "Copy as HTML" option', () => {
@@ -51,7 +52,9 @@ describe('DocToolBox "Copy as HTML" option', () => {
  render(<DocToolBox doc={doc as any} />, {
  wrapper: AppWrapper,
  });
- const optionsButton = screen.getByLabelText('Open the document options');
+ const optionsButton = await screen.findByLabelText(
+ 'Open the document options',
+ );
  await userEvent.click(optionsButton);
  expect(await screen.findByText('Copy as HTML')).toBeInTheDocument();
  });

src/frontend/apps/impress/src/features/docs/doc-header/__tests__/DocToolBoxAGPL.spec.tsx

@@ -0,0 +1,28 @@
+import { render, screen } from '@testing-library/react';
+import React from 'react';
+
+import { AppWrapper } from '@/tests/utils';
+
+import { DocToolBox } from '../components/DocToolBox';
+
+const doc = {
+ nb_accesses: 1,
+ abilities: {
+ versions_list: true,
+ destroy: true,
+ },
+};
+
+jest.mock('@/features/docs/doc-export/', () => ({
+ ModalExport: () => <span>ModalExport</span>,
+}));
+
+it('DocToolBox dynamic import: loads DocToolBox when MIT_ONLY is false', async () => {
+ process.env.MIT_ONLY = 'false';
+
+ render(<DocToolBox doc={doc as any} />, {
+ wrapper: AppWrapper,
+ });
+
+ expect(await screen.findByText('download')).toBeInTheDocument();
+});