remise à niveau

The modal for document options has been modified to allow navigation
using "tab" and to be closed.

.github/workflows/impress-frontend.yml

@@ -88,6 +88,28 @@ jobs:
       - name: Start Docker services
         run: make bootstrap FLUSH_ARGS='--no-input' cache=
 
+      # Tool to wait for a service to be ready
+      - name: Install Dockerize
+        run: |
+          curl -sSL https://github.com/jwilder/dockerize/releases/download/v0.8.0/dockerize-linux-amd64-v0.8.0.tar.gz | sudo tar -C /usr/local/bin -xzv
+
+      - name: Wait for services to be ready
+        run: |
+          printf "Minio check...\n"
+          dockerize -wait tcp://localhost:9000 -timeout 20s
+          printf "Keyclock check...\n"
+          dockerize -wait tcp://localhost:8080 -timeout 20s
+          printf "Server collaboration check...\n"
+          dockerize -wait tcp://localhost:4444 -timeout 20s
+          printf "Ngnix check...\n"
+          dockerize -wait tcp://localhost:8083 -timeout 20s
+          printf "DRF check...\n"
+          dockerize -wait tcp://localhost:8071 -timeout 20s
+          printf "Postgres Keyclock check...\n"
+          dockerize -wait tcp://localhost:5433 -timeout 20s
+          printf "Postgres back check...\n"
+          dockerize -wait tcp://localhost:15432 -timeout 20s
+
       - name: Run e2e tests
         run: cd src/frontend/ && yarn e2e:test --project='chromium'
 

CHANGELOG.md

@@ -6,30 +6,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0),
 and this project adheres to
 [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+
 ## [Unreleased]
 
-## [2.3.0] - 2025-03-03
-
-## Added
-
-- 💄(frontend) add error pages #643
-- 🔒️ Manage unsafe attachments #663
-- ✨(frontend) Custom block quote with export #646
-- ✨(frontend) add open source section homepage #666
-
-## Changed
-
-- 🛂(frontend) Restore version visibility #629
-- 📝(doc) minor README.md formatting and wording enhancements
-- ♻️Stop setting a default title on doc creation #634
-- ♻️(frontend) misc ui improvements #644
-
-## Fixed
-
-- 🐛(backend) allow any type of extensions for media download #671
-- ♻️(frontend) improve table pdf rendering
-
-
 ## [2.2.0] - 2025-02-10
 
 ## Added
@@ -49,6 +28,7 @@ and this project adheres to
 - 🐛(frontend) fix cursor breakline #609
 - 🐛(frontend) fix style pdf export #609
 
+
 ## [2.1.0] - 2025-01-29
 
 ## Added
@@ -64,7 +44,7 @@ and this project adheres to
 ## Changed
 
 - 💄(frontend) add abilities on doc row #581
-- 💄(frontend) improve DocsGridItem responsive padding #582
+- 💄(frontend) improve DocsGridItem responsive padding  #582
 - 🔧(backend) Bump maximum page size to 200 #516
 - 📝(doc) Improve Read me #558
 
@@ -76,6 +56,7 @@ and this project adheres to
 
 - 🔥(backend) remove "content" field from list serializer # 516
 
+
 ## [2.0.1] - 2025-01-17
 
 ## Fixed
@@ -130,11 +111,12 @@ and this project adheres to
 
 - ⚡️(e2e) reduce flakiness on e2e tests #511
 
-## Fixed
 
+## Fixed
 - 🐛(frontend) update doc editor height #481
 - 💄(frontend) add doc search #485
 
+
 ## [1.9.0] - 2024-12-11
 
 ## Added
@@ -156,18 +138,21 @@ and this project adheres to
 - 🐛(frontend) Fix hidden menu on Firefox #468
 - 🐛(backend) fix sanitize problem IA #490
 
+
 ## [1.8.2] - 2024-11-28
 
 ## Changed
 
 - ♻️(SW) change strategy html caching #460
 
+
 ## [1.8.1] - 2024-11-27
 
 ## Fixed
 
 - 🐛(frontend) link not clickable and flickering firefox #457
 
+
 ## [1.8.0] - 2024-11-25
 
 ## Added
@@ -196,6 +181,7 @@ and this project adheres to
 - 🐛(frontend) users have view access when revoked #387
 - 🐛(frontend) fix placeholder editable when double clicks #454
 
+
 ## [1.7.0] - 2024-10-24
 
 ## Added
@@ -223,6 +209,7 @@ and this project adheres to
 
 - 🔥(helm) remove infra related codes #366
 
+
 ## [1.6.0] - 2024-10-17
 
 ## Added
@@ -245,6 +232,7 @@ and this project adheres to
 - 🐛(backend) fix nginx docker container #340
 - 🐛(frontend) fix copy paste firefox #353
 
+
 ## [1.5.1] - 2024-10-10
 
 ## Fixed
@@ -279,6 +267,7 @@ and this project adheres to
 - 🔧(backend) fix configuration to avoid different ssl warning #297
 - 🐛(frontend) fix editor break line not working #302
 
+
 ## [1.4.0] - 2024-09-17
 
 ## Added
@@ -299,6 +288,7 @@ and this project adheres to
 - 🐛(backend) Fix forcing ID when creating a document via API endpoint #234
 - 🐛 Rebuild frontend dev container from makefile #248
 
+
 ## [1.3.0] - 2024-09-05
 
 ## Added
@@ -323,6 +313,7 @@ and this project adheres to
 
 - 🔥(frontend) remove saving modal #213
 
+
 ## [1.2.1] - 2024-08-23
 
 ## Changed
@@ -330,6 +321,7 @@ and this project adheres to
 - ♻️ Change ordering docs datagrid #195
 - 🔥(helm) use scaleway email #194
 
+
 ## [1.2.0] - 2024-08-22
 
 ## Added
@@ -353,14 +345,14 @@ and this project adheres to
 - ⚡️(CI) only e2e chrome mandatory #177
 
 ## Removed
-
 - 🔥(helm) remove htaccess #181
 
+
 ## [1.1.0] - 2024-07-15
 
 ## Added
 
-- 🤡(demo) generate dummy documents on dev users #120
+- 🤡(demo) generate dummy documents on dev users #120 
 - ✨(frontend) create side modal component #134
 - ✨(frontend) Doc grid actions (update / delete) #136
 - ✨(frontend) Doc editor header information #137
@@ -371,11 +363,12 @@ and this project adheres to
 - ♻️(frontend) create a doc from a modal #132
 - ♻️(frontend) manage members from the share modal #140
 
+
 ## [1.0.0] - 2024-07-02
 
 ## Added
 
-- 🛂(frontend) Manage the document's right (#75)
+- 🛂(frontend) Manage the document's right (#75) 
 - ✨(frontend) Update document (#68)
 - ✨(frontend) Remove document (#68)
 - 🐳(docker) dockerize dev frontend (#63)
@@ -409,6 +402,7 @@ and this project adheres to
 - 💚(CI) Remove trigger workflow on push tags on CI (#68)
 - 🔥(frontend) Remove coming soon page (#121)
 
+
 ## [0.1.0] - 2024-05-24
 
 ## Added
@@ -416,8 +410,8 @@ and this project adheres to
 - ✨(frontend) Coming Soon page (#67)
 - 🚀 Impress, project to manage your documents easily and collaboratively.
 
-[unreleased]: https://github.com/numerique-gouv/impress/compare/v2.3.0...main
-[v2.3.0]: https://github.com/numerique-gouv/impress/releases/v2.3.0
+
+[unreleased]: https://github.com/numerique-gouv/impress/compare/v2.2.0...main
 [v2.2.0]: https://github.com/numerique-gouv/impress/releases/v2.2.0
 [v2.1.0]: https://github.com/numerique-gouv/impress/releases/v2.1.0
 [v2.0.1]: https://github.com/numerique-gouv/impress/releases/v2.0.1

Makefile

@@ -44,6 +44,7 @@ COMPOSE_EXEC_APP    = $(COMPOSE_EXEC) app-dev
 COMPOSE_RUN         = $(COMPOSE) run --rm
 COMPOSE_RUN_APP     = $(COMPOSE_RUN) app-dev
 COMPOSE_RUN_CROWDIN = $(COMPOSE_RUN) crowdin crowdin
+WAIT_DB             = @$(COMPOSE_RUN) dockerize -wait tcp://$(DB_HOST):$(DB_PORT) -timeout 60s
 
 # -- Backend
 MANAGE              = $(COMPOSE_RUN_APP) python manage.py
@@ -80,12 +81,12 @@ bootstrap: \
 	data/static \
 	create-env-files \
 	build \
+	run-with-frontend \
 	migrate \
 	demo \
 	back-i18n-compile \
 	mails-install \
-	mails-build \
-	run
+	mails-build
 .PHONY: bootstrap
 
 # -- Docker/compose
@@ -108,7 +109,7 @@ build-yjs-provider: ## build the y-provider container
 
 build-frontend: cache ?=
 build-frontend: ## build the frontend container
-	@$(COMPOSE) build frontend $(cache)
+	@$(COMPOSE) build frontend-dev $(cache)
 .PHONY: build-frontend
 
 down: ## stop and remove containers, networks, images, and volumes
@@ -119,18 +120,19 @@ logs: ## display app-dev logs (follow mode)
 	@$(COMPOSE) logs -f app-dev
 .PHONY: logs
 
-run-backend: ## Start only the backend application and all needed services
+run: ## start the wsgi (production) and development server
 	@$(COMPOSE) up --force-recreate -d celery-dev
 	@$(COMPOSE) up --force-recreate -d y-provider
 	@$(COMPOSE) up --force-recreate -d nginx
-.PHONY: run-backend
-
-run: ## start the wsgi (production) and development server
-run: 
-	@$(MAKE) run-backend
-	@$(COMPOSE) up --force-recreate -d frontend
+	@echo "Wait for postgresql to be up..."
+	@$(WAIT_DB)
 .PHONY: run
 
+run-with-frontend: ## Start all the containers needed (backend to frontend)
+	@$(MAKE) run
+	@$(COMPOSE) up --force-recreate -d frontend-dev
+.PHONY: run-with-frontend
+
 status: ## an alias for "docker compose ps"
 	@$(COMPOSE) ps
 .PHONY: status
@@ -186,12 +188,14 @@ test-back-parallel: ## run all back-end tests in parallel
 makemigrations:  ## run django makemigrations for the impress project.
 	@echo "$(BOLD)Running makemigrations$(RESET)"
 	@$(COMPOSE) up -d postgresql
+	@$(WAIT_DB)
 	@$(MANAGE) makemigrations
 .PHONY: makemigrations
 
 migrate:  ## run django migrations for the impress project.
 	@echo "$(BOLD)Running migrations$(RESET)"
 	@$(COMPOSE) up -d postgresql
+	@$(WAIT_DB)
 	@$(MANAGE) migrate
 .PHONY: migrate
 
@@ -306,16 +310,16 @@ help:
 .PHONY: help
 
 # Front
-frontend-development-install: ## install the frontend locally
+frontend-install: ## install the frontend locally
 	cd $(PATH_FRONT_IMPRESS) && yarn
-.PHONY: frontend-development-install
+.PHONY: frontend-install
 
 frontend-lint: ## run the frontend linter
 	cd $(PATH_FRONT) && yarn lint
 .PHONY: frontend-lint
 
 run-frontend-development: ## Run the frontend in development mode
-	@$(COMPOSE) stop frontend
+	@$(COMPOSE) stop frontend-dev
 	cd $(PATH_FRONT_IMPRESS) && yarn dev
 .PHONY: run-frontend-development
 

README.md

@@ -23,7 +23,6 @@ Welcome to Docs! The open source document editor where your notes can become kno
 <img src="/docs/assets/docs_live_collaboration_light.gif" width="100%" align="center"/>
 
 ## Why use Docs ❓
-
 Docs is a collaborative text editor designed to address common challenges in knowledge building and sharing.
 
 ### Write
@@ -34,31 +33,23 @@ Docs is a collaborative text editor designed to address common challenges in kno
 *   ✨ Save time thanks to our AI actions (generate, sum up, correct, translate)
 
 ### Collaborate
-*   🤝 Collaborate with your team in real time
-*   🔒 Granular access control to ensure your information is secure and only shared with the right people
+*   🤝 Collaborate in realtime with your team mates
+*   🔒 Granular access control to keep your information secure and shared with the right people
 *   📑 Professional document exports in multiple formats (.odt, .doc, .pdf) with customizable templates
-*   📚 Built-in wiki functionality to turn your team's collaborative work into organized knowledge `ETA 02/2025`
+*   📚 Built-in wiki functionality to transform your team's collaborative work into organized knowledge `ETA 02/2025`
 
 ### Self-host
 *   🚀 Easy to install, scalable and secure alternative to Notion, Outline or Confluence
 
 ## Getting started 🔧
-
 ### Test it
-
 Test Docs on your browser by logging in on this [environment](https://impress-preprod.beta.numerique.gouv.fr/docs/0aa856e9-da41-4d59-b73d-a61cb2c1245f/)
-
 ```
 email: test.docs@yopmail.com
 password: I'd<3ToTestDocs
 ```
-
 ### Run it locally
-
-> ⚠️ Running Docs locally using the methods described below is for testing purposes only.  It is based on building Docs using Minio as the S3 storage solution: if you want to use Minio for production deployment of Docs, you will need to comply with Minio's AGPL-3.0 licence.
-
 **Prerequisite**
-
 Make sure you have a recent version of Docker and [Docker Compose](https://docs.docker.com/compose/install) installed on your laptop:
 
 ```shellscript
@@ -66,22 +57,23 @@ $ docker -v
 
 Docker version 20.10.2, build 2291f61
 
-$ docker compose version
+$ docker compose -v
 
-Docker Compose version v2.32.4
+docker compose version 1.27.4, build 40524192
 ```
 
 > ⚠️ You may need to run the following commands with sudo but this can be avoided by adding your user to the `docker` group.
 
 **Project bootstrap**
-
 The easiest way to start working on the project is to use GNU Make:
 
 ```shellscript
 $ make bootstrap FLUSH_ARGS='--no-input'
 ```
 
-This command builds the `app` container, installs dependencies, performs database migrations and compile translations. It's a good idea to use this command each time you are pulling code from the project repository to avoid dependency-related or migration-related issues.
+This command builds the `app` container, installs dependencies, performs database migrations and compile translations. It's a good idea to use this
+
+command each time you are pulling code from the project repository to avoid dependency-related or migration-related issues.
 
 Your Docker services should now be up and running 🎉
 
@@ -97,7 +89,7 @@ password: impress
 📝 Note that if you need to run them afterwards, you can use the eponym Make rule:
 
 ```shellscript
-$ make run
+$ make run-with-frontend
 ```
 
 ⚠️ For the frontend developer, it is often better to run the frontend in development mode locally.
@@ -105,7 +97,7 @@ $ make run
 To do so, install the frontend dependencies with the following command:
 
 ```shellscript
-$ make frontend-development-install
+$ make frontend-install
 ```
 
 And run the frontend locally in development mode with the following command:
@@ -117,7 +109,7 @@ $ make run-frontend-development
 To start all the services, except the frontend container, you can use the following command:
 
 ```shellscript
-$ make run-backend
+$ make run
 ```
 
 **Adding content**
@@ -134,7 +126,6 @@ $ make help
 ```
 
 **Django admin**
-
 You can access the Django admin site at
 
 <http://localhost:8071/admin>.
@@ -146,21 +137,17 @@ $ make superuser
 ```
 
 ## Feedback 🙋‍♂️🙋‍♀️
-
 We'd love to hear your thoughts and hear about your experiments, so come and say hi on [Matrix](https://matrix.to/#/#docs-official:matrix.org).
 
 ## Roadmap
-
 Want to know where the project is headed? [🗺️ Checkout our roadmap](https://github.com/orgs/numerique-gouv/projects/13/views/11)
 
 ## Licence 📝
-
 This work is released under the MIT License (see [LICENSE](https://github.com/suitenumerique/docs/blob/main/LICENSE)).
 
 While Docs is a public driven initiative our licence choice is an invitation for private sector actors to use, sell and contribute to the project. 
 
 ## Contributing 🙌
-
 This project is intended to be community-driven, so please, do not hesitate to [get in touch](https://matrix.to/#/#docs-official:matrix.org) if you have any question related to our implementation or design decisions.
 
 You can help us with translations on [Crowdin](https://crowdin.com/project/lasuite-docs).
@@ -182,13 +169,10 @@ docs
 ```
 
 ## Credits ❤️
-
 ### Stack
-
-Docs is built on top of [Django Rest Framework](https://www.django-rest-framework.org/), [Next.js](https://nextjs.org/), [BlockNote.js](https://www.blocknotejs.org/), [HocusPocus](https://tiptap.dev/docs/hocuspocus/introduction) and [Yjs](https://yjs.dev/).
+Docs is built on top of [Django Rest Framework](https://www.django-rest-framework.org/), [Next.js](https://nextjs.org/), [MinIO](https://min.io/), [BlockNote.js](https://www.blocknotejs.org/), [HocusPocus](https://tiptap.dev/docs/hocuspocus/introduction) and [Yjs](https://yjs.dev/)
 
 ### Gov ❤️ open source
-
 Docs is the result of a joint effort led by the French 🇫🇷🥖 ([DINUM](https://www.numerique.gouv.fr/dinum/)) and German 🇩🇪🥨 governments ([ZenDiS](https://zendis.de/)). 
 
 We are proud sponsors of [BlockNotejs](https://www.blocknotejs.org/) and [Yjs](https://yjs.dev/).

bin/_config.sh

@@ -7,6 +7,7 @@ UNSET_USER=0
 
 TERRAFORM_DIRECTORY="./env.d/terraform"
 COMPOSE_FILE="${REPO_DIR}/docker-compose.yml"
+COMPOSE_PROJECT="docs"
 
 
 # _set_user: set (or unset) default user id used to run docker commands
@@ -39,8 +40,9 @@ function _set_user() {
 # ARGS   : docker compose command arguments
 function _docker_compose() {
 
-    echo "🐳(compose) file: '${COMPOSE_FILE}'"
+    echo "🐳(compose) project: '${COMPOSE_PROJECT}' file: '${COMPOSE_FILE}'"
     docker compose \
+        -p "${COMPOSE_PROJECT}" \
         -f "${COMPOSE_FILE}" \
         --project-directory "${REPO_DIR}" \
         "$@"

docker-compose.yml

@@ -1,13 +1,6 @@
-name: docs
-
 services:
   postgresql:
     image: postgres:16
-    healthcheck:
-      test: ["CMD-SHELL", "pg_isready"]
-      interval: 1s
-      timeout: 2s
-      retries: 300
     env_file:
       - env.d/development/postgresql
     ports:
@@ -22,7 +15,7 @@ services:
       - "1081:1080"
 
   minio:
-    user: ${DOCKER_USER:-1000}
+    # user: ${DOCKER_USER:-1000}
     image: minio/minio
     environment:
       - MINIO_ROOT_USER=impress
@@ -30,11 +23,6 @@ services:
     ports:
       - '9000:9000'
       - '9001:9001'
-    healthcheck:
-      test: ["CMD", "mc", "ready", "local"]
-      interval: 1s
-      timeout: 20s
-      retries: 300
     entrypoint: ""
     command: minio server --console-address :9001 /data
     volumes:
@@ -43,9 +31,7 @@ services:
   createbuckets:
     image: minio/mc
     depends_on:
-      minio:
-        condition: service_healthy
-        restart: true
+      - minio
     entrypoint: >
       sh -c "
       /usr/bin/mc alias set impress http://minio:9000 impress password && \
@@ -73,15 +59,10 @@ services:
       - ./src/backend:/app
       - ./data/static:/data/static
     depends_on:
-        postgresql:
-            condition: service_healthy
-            restart: true
-        mailcatcher:
-          condition: service_started
-        redis:
-          condition: service_started
-        createbuckets:
-          condition: service_started
+        - postgresql
+        - mailcatcher
+        - redis
+        - createbuckets
   
   celery-dev:
     user: ${DOCKER_USER:-1000}
@@ -112,13 +93,9 @@ services:
       - env.d/development/common
       - env.d/development/postgresql
     depends_on:
-      postgresql:
-        condition: service_healthy
-        restart: true
-      redis:
-        condition: service_started
-      minio:
-        condition: service_started
+      - postgresql
+      - redis
+      - minio
 
   celery:
     user: ${DOCKER_USER:-1000}
@@ -139,15 +116,11 @@ services:
     volumes:
       - ./docker/files/etc/nginx/conf.d:/etc/nginx/conf.d:ro
     depends_on:
-      app-dev:
-        condition: service_started
-      y-provider:
-        condition: service_started
-      keycloak:
-        condition: service_healthy
-        restart: true
+      - keycloak
+      - app-dev
+      - y-provider
 
-  frontend:
+  frontend-dev:
     user: "${DOCKER_USER:-1000}"
     build: 
       context: .
@@ -162,6 +135,9 @@ services:
     ports:
       - "3000:3000"
 
+  dockerize:
+    image: jwilder/dockerize
+
   crowdin:
     image: crowdin/cli:3.16.0
     volumes:
@@ -193,11 +169,6 @@ services:
 
   kc_postgresql:
     image: postgres:14.3
-    healthcheck:
-      test: ["CMD-SHELL", "pg_isready"]
-      interval: 1s
-      timeout: 2s
-      retries: 300
     ports:
       - "5433:5432"
     env_file:
@@ -216,13 +187,6 @@ services:
       - --hostname-admin-url=http://localhost:8083/
       - --hostname-strict=false
       - --hostname-strict-https=false
-      - --health-enabled=true
-      - --metrics-enabled=true
-    healthcheck:
-      test: ["CMD", "curl", "--head", "-fsS", "http://localhost:8080/health/ready"]
-      interval: 1s
-      timeout: 2s
-      retries: 300
     environment:
       KEYCLOAK_ADMIN: admin
       KEYCLOAK_ADMIN_PASSWORD: admin
@@ -236,6 +200,4 @@ services:
     ports:
       - "8080:8080"
     depends_on:
-      kc_postgresql:
-        condition: service_healthy
-        restart: true
+      - kc_postgresql

docker/files/etc/nginx/conf.d/default.conf

@@ -68,8 +68,6 @@ server {
         # Get resource from Minio
         proxy_pass http://minio:9000/impress-media-storage/;
         proxy_set_header Host minio:9000;
-
-        add_header Content-Security-Policy "default-src 'none'" always;
     }
 
     location /media-auth {
@@ -90,11 +88,5 @@ server {
         proxy_set_header Host $host;
         proxy_set_header X-Real-IP $remote_addr;
         proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
-
-        # Increase proxy buffer size to allow keycloak to send large
-        # header responses when a user is created.
-        proxy_buffer_size 128k;
-        proxy_buffers 4 256k;
-        proxy_busy_buffers_size 256k;
     }
 }

package.json

@@ -0,0 +1,6 @@
+{
+  "dependencies": {
+    "@blocknote/core": "^0.23.4",
+    "next": "^15.1.7"
+  }
+}

src/backend/core/api/serializers.py

@@ -418,7 +418,6 @@ class FileUploadSerializer(serializers.Serializer):
 
         self.context["expected_extension"] = extension
         self.context["content_type"] = magic_mime_type
-        self.context["file_name"] = file.name
 
         return file
 
@@ -427,7 +426,6 @@ class FileUploadSerializer(serializers.Serializer):
         attrs["expected_extension"] = self.context["expected_extension"]
         attrs["is_unsafe"] = self.context["is_unsafe"]
         attrs["content_type"] = self.context["content_type"]
-        attrs["file_name"] = self.context["file_name"]
         return attrs
 
 

src/backend/core/api/viewsets.py

@@ -38,10 +38,10 @@ ATTACHMENTS_FOLDER = "attachments"
 UUID_REGEX = (
     r"[a-fA-F0-9]{8}-[a-fA-F0-9]{4}-[a-fA-F0-9]{4}-[a-fA-F0-9]{4}-[a-fA-F0-9]{12}"
 )
-FILE_EXT_REGEX = r"\.[a-zA-Z0-9]{1,10}"
+FILE_EXT_REGEX = r"\.[a-zA-Z]{3,4}"
 MEDIA_STORAGE_URL_PATTERN = re.compile(
     f"{settings.MEDIA_URL:s}(?P<pk>{UUID_REGEX:s})/"
-    f"(?P<key>{ATTACHMENTS_FOLDER:s}/{UUID_REGEX:s}(?:-unsafe)?{FILE_EXT_REGEX:s})$"
+    f"(?P<key>{ATTACHMENTS_FOLDER:s}/{UUID_REGEX:s}{FILE_EXT_REGEX:s})$"
 )
 COLLABORATION_WS_URL_PATTERN = re.compile(rf"(?:^|&)room=(?P<pk>{UUID_REGEX})(?:&|$)")
 
@@ -915,31 +915,15 @@ class DocumentViewSet(
         # Generate a generic yet unique filename to store the image in object storage
         file_id = uuid.uuid4()
         extension = serializer.validated_data["expected_extension"]
+        key = f"{document.key_base}/{ATTACHMENTS_FOLDER:s}/{file_id!s}.{extension:s}"
 
         # Prepare metadata for storage
         extra_args = {
             "Metadata": {"owner": str(request.user.id)},
             "ContentType": serializer.validated_data["content_type"],
         }
-        file_unsafe = ""
         if serializer.validated_data["is_unsafe"]:
             extra_args["Metadata"]["is_unsafe"] = "true"
-            file_unsafe = "-unsafe"
-
-        key = f"{document.key_base}/{ATTACHMENTS_FOLDER:s}/{file_id!s}{file_unsafe}.{extension:s}"
-
-        file_name = serializer.validated_data["file_name"]
-        if (
-            not serializer.validated_data["content_type"].startswith("image/")
-            or serializer.validated_data["is_unsafe"]
-        ):
-            extra_args.update(
-                {"ContentDisposition": f'attachment; filename="{file_name:s}"'}
-            )
-        else:
-            extra_args.update(
-                {"ContentDisposition": f'inline; filename="{file_name:s}"'}
-            )
 
         file = serializer.validated_data["file"]
         default_storage.connection.meta.client.upload_fileobj(

src/backend/core/migrations/0001_initial.py

@@ -1,552 +1,166 @@
 # Generated by Django 5.0.3 on 2024-05-28 20:29
 
-import uuid
-
 import django.contrib.auth.models
 import django.core.validators
 import django.db.models.deletion
+import timezone_field.fields
+import uuid
 from django.conf import settings
 from django.db import migrations, models
 
-import timezone_field.fields
-
 
 class Migration(migrations.Migration):
+
     initial = True
 
     dependencies = [
-        ("auth", "0012_alter_user_first_name_max_length"),
+        ('auth', '0012_alter_user_first_name_max_length'),
     ]
 
     operations = [
         migrations.CreateModel(
-            name="Document",
+            name='Document',
             fields=[
-                (
-                    "id",
-                    models.UUIDField(
-                        default=uuid.uuid4,
-                        editable=False,
-                        help_text="primary key for the record as UUID",
-                        primary_key=True,
-                        serialize=False,
-                        verbose_name="id",
-                    ),
-                ),
-                (
-                    "created_at",
-                    models.DateTimeField(
-                        auto_now_add=True,
-                        help_text="date and time at which a record was created",
-                        verbose_name="created on",
-                    ),
-                ),
-                (
-                    "updated_at",
-                    models.DateTimeField(
-                        auto_now=True,
-                        help_text="date and time at which a record was last updated",
-                        verbose_name="updated on",
-                    ),
-                ),
-                ("title", models.CharField(max_length=255, verbose_name="title")),
-                (
-                    "is_public",
-                    models.BooleanField(
-                        default=False,
-                        help_text="Whether this document is public for anyone to use.",
-                        verbose_name="public",
-                    ),
-                ),
+                ('id', models.UUIDField(default=uuid.uuid4, editable=False, help_text='primary key for the record as UUID', primary_key=True, serialize=False, verbose_name='id')),
+                ('created_at', models.DateTimeField(auto_now_add=True, help_text='date and time at which a record was created', verbose_name='created on')),
+                ('updated_at', models.DateTimeField(auto_now=True, help_text='date and time at which a record was last updated', verbose_name='updated on')),
+                ('title', models.CharField(max_length=255, verbose_name='title')),
+                ('is_public', models.BooleanField(default=False, help_text='Whether this document is public for anyone to use.', verbose_name='public')),
             ],
             options={
-                "verbose_name": "Document",
-                "verbose_name_plural": "Documents",
-                "db_table": "impress_document",
-                "ordering": ("title",),
+                'verbose_name': 'Document',
+                'verbose_name_plural': 'Documents',
+                'db_table': 'impress_document',
+                'ordering': ('title',),
             },
         ),
         migrations.CreateModel(
-            name="Template",
+            name='Template',
             fields=[
-                (
-                    "id",
-                    models.UUIDField(
-                        default=uuid.uuid4,
-                        editable=False,
-                        help_text="primary key for the record as UUID",
-                        primary_key=True,
-                        serialize=False,
-                        verbose_name="id",
-                    ),
-                ),
-                (
-                    "created_at",
-                    models.DateTimeField(
-                        auto_now_add=True,
-                        help_text="date and time at which a record was created",
-                        verbose_name="created on",
-                    ),
-                ),
-                (
-                    "updated_at",
-                    models.DateTimeField(
-                        auto_now=True,
-                        help_text="date and time at which a record was last updated",
-                        verbose_name="updated on",
-                    ),
-                ),
-                ("title", models.CharField(max_length=255, verbose_name="title")),
-                (
-                    "description",
-                    models.TextField(blank=True, verbose_name="description"),
-                ),
-                ("code", models.TextField(blank=True, verbose_name="code")),
-                ("css", models.TextField(blank=True, verbose_name="css")),
-                (
-                    "is_public",
-                    models.BooleanField(
-                        default=False,
-                        help_text="Whether this template is public for anyone to use.",
-                        verbose_name="public",
-                    ),
-                ),
+                ('id', models.UUIDField(default=uuid.uuid4, editable=False, help_text='primary key for the record as UUID', primary_key=True, serialize=False, verbose_name='id')),
+                ('created_at', models.DateTimeField(auto_now_add=True, help_text='date and time at which a record was created', verbose_name='created on')),
+                ('updated_at', models.DateTimeField(auto_now=True, help_text='date and time at which a record was last updated', verbose_name='updated on')),
+                ('title', models.CharField(max_length=255, verbose_name='title')),
+                ('description', models.TextField(blank=True, verbose_name='description')),
+                ('code', models.TextField(blank=True, verbose_name='code')),
+                ('css', models.TextField(blank=True, verbose_name='css')),
+                ('is_public', models.BooleanField(default=False, help_text='Whether this template is public for anyone to use.', verbose_name='public')),
             ],
             options={
-                "verbose_name": "Template",
-                "verbose_name_plural": "Templates",
-                "db_table": "impress_template",
-                "ordering": ("title",),
+                'verbose_name': 'Template',
+                'verbose_name_plural': 'Templates',
+                'db_table': 'impress_template',
+                'ordering': ('title',),
             },
         ),
         migrations.CreateModel(
-            name="User",
+            name='User',
             fields=[
-                ("password", models.CharField(max_length=128, verbose_name="password")),
-                (
-                    "last_login",
-                    models.DateTimeField(
-                        blank=True, null=True, verbose_name="last login"
-                    ),
-                ),
-                (
-                    "is_superuser",
-                    models.BooleanField(
-                        default=False,
-                        help_text="Designates that this user has all permissions without explicitly assigning them.",
-                        verbose_name="superuser status",
-                    ),
-                ),
-                (
-                    "id",
-                    models.UUIDField(
-                        default=uuid.uuid4,
-                        editable=False,
-                        help_text="primary key for the record as UUID",
-                        primary_key=True,
-                        serialize=False,
-                        verbose_name="id",
-                    ),
-                ),
-                (
-                    "created_at",
-                    models.DateTimeField(
-                        auto_now_add=True,
-                        help_text="date and time at which a record was created",
-                        verbose_name="created on",
-                    ),
-                ),
-                (
-                    "updated_at",
-                    models.DateTimeField(
-                        auto_now=True,
-                        help_text="date and time at which a record was last updated",
-                        verbose_name="updated on",
-                    ),
-                ),
-                (
-                    "sub",
-                    models.CharField(
-                        blank=True,
-                        help_text="Required. 255 characters or fewer. Letters, numbers, and @/./+/-/_ characters only.",
-                        max_length=255,
-                        null=True,
-                        unique=True,
-                        validators=[
-                            django.core.validators.RegexValidator(
-                                message="Enter a valid sub. This value may contain only letters, numbers, and @/./+/-/_ characters.",
-                                regex="^[\\w.@+-]+\\Z",
-                            )
-                        ],
-                        verbose_name="sub",
-                    ),
-                ),
-                (
-                    "email",
-                    models.EmailField(
-                        blank=True,
-                        max_length=254,
-                        null=True,
-                        verbose_name="identity email address",
-                    ),
-                ),
-                (
-                    "admin_email",
-                    models.EmailField(
-                        blank=True,
-                        max_length=254,
-                        null=True,
-                        unique=True,
-                        verbose_name="admin email address",
-                    ),
-                ),
-                (
-                    "language",
-                    models.CharField(
-                        choices="(('en-us', 'English'), ('fr-fr', 'French'))",
-                        default="en-us",
-                        help_text="The language in which the user wants to see the interface.",
-                        max_length=10,
-                        verbose_name="language",
-                    ),
-                ),
-                (
-                    "timezone",
-                    timezone_field.fields.TimeZoneField(
-                        choices_display="WITH_GMT_OFFSET",
-                        default="UTC",
-                        help_text="The timezone in which the user wants to see times.",
-                        use_pytz=False,
-                    ),
-                ),
-                (
-                    "is_device",
-                    models.BooleanField(
-                        default=False,
-                        help_text="Whether the user is a device or a real user.",
-                        verbose_name="device",
-                    ),
-                ),
-                (
-                    "is_staff",
-                    models.BooleanField(
-                        default=False,
-                        help_text="Whether the user can log into this admin site.",
-                        verbose_name="staff status",
-                    ),
-                ),
-                (
-                    "is_active",
-                    models.BooleanField(
-                        default=True,
-                        help_text="Whether this user should be treated as active. Unselect this instead of deleting accounts.",
-                        verbose_name="active",
-                    ),
-                ),
-                (
-                    "groups",
-                    models.ManyToManyField(
-                        blank=True,
-                        help_text="The groups this user belongs to. A user will get all permissions granted to each of their groups.",
-                        related_name="user_set",
-                        related_query_name="user",
-                        to="auth.group",
-                        verbose_name="groups",
-                    ),
-                ),
-                (
-                    "user_permissions",
-                    models.ManyToManyField(
-                        blank=True,
-                        help_text="Specific permissions for this user.",
-                        related_name="user_set",
-                        related_query_name="user",
-                        to="auth.permission",
-                        verbose_name="user permissions",
-                    ),
-                ),
+                ('password', models.CharField(max_length=128, verbose_name='password')),
+                ('last_login', models.DateTimeField(blank=True, null=True, verbose_name='last login')),
+                ('is_superuser', models.BooleanField(default=False, help_text='Designates that this user has all permissions without explicitly assigning them.', verbose_name='superuser status')),
+                ('id', models.UUIDField(default=uuid.uuid4, editable=False, help_text='primary key for the record as UUID', primary_key=True, serialize=False, verbose_name='id')),
+                ('created_at', models.DateTimeField(auto_now_add=True, help_text='date and time at which a record was created', verbose_name='created on')),
+                ('updated_at', models.DateTimeField(auto_now=True, help_text='date and time at which a record was last updated', verbose_name='updated on')),
+                ('sub', models.CharField(blank=True, help_text='Required. 255 characters or fewer. Letters, numbers, and @/./+/-/_ characters only.', max_length=255, null=True, unique=True, validators=[django.core.validators.RegexValidator(message='Enter a valid sub. This value may contain only letters, numbers, and @/./+/-/_ characters.', regex='^[\\w.@+-]+\\Z')], verbose_name='sub')),
+                ('email', models.EmailField(blank=True, max_length=254, null=True, verbose_name='identity email address')),
+                ('admin_email', models.EmailField(blank=True, max_length=254, null=True, unique=True, verbose_name='admin email address')),
+                ('language', models.CharField(choices="(('en-us', 'English'), ('fr-fr', 'French'))", default='en-us', help_text='The language in which the user wants to see the interface.', max_length=10, verbose_name='language')),
+                ('timezone', timezone_field.fields.TimeZoneField(choices_display='WITH_GMT_OFFSET', default='UTC', help_text='The timezone in which the user wants to see times.', use_pytz=False)),
+                ('is_device', models.BooleanField(default=False, help_text='Whether the user is a device or a real user.', verbose_name='device')),
+                ('is_staff', models.BooleanField(default=False, help_text='Whether the user can log into this admin site.', verbose_name='staff status')),
+                ('is_active', models.BooleanField(default=True, help_text='Whether this user should be treated as active. Unselect this instead of deleting accounts.', verbose_name='active')),
+                ('groups', models.ManyToManyField(blank=True, help_text='The groups this user belongs to. A user will get all permissions granted to each of their groups.', related_name='user_set', related_query_name='user', to='auth.group', verbose_name='groups')),
+                ('user_permissions', models.ManyToManyField(blank=True, help_text='Specific permissions for this user.', related_name='user_set', related_query_name='user', to='auth.permission', verbose_name='user permissions')),
             ],
             options={
-                "verbose_name": "user",
-                "verbose_name_plural": "users",
-                "db_table": "impress_user",
+                'verbose_name': 'user',
+                'verbose_name_plural': 'users',
+                'db_table': 'impress_user',
             },
             managers=[
-                ("objects", django.contrib.auth.models.UserManager()),
+                ('objects', django.contrib.auth.models.UserManager()),
             ],
         ),
         migrations.CreateModel(
-            name="DocumentAccess",
+            name='DocumentAccess',
             fields=[
-                (
-                    "id",
-                    models.UUIDField(
-                        default=uuid.uuid4,
-                        editable=False,
-                        help_text="primary key for the record as UUID",
-                        primary_key=True,
-                        serialize=False,
-                        verbose_name="id",
-                    ),
-                ),
-                (
-                    "created_at",
-                    models.DateTimeField(
-                        auto_now_add=True,
-                        help_text="date and time at which a record was created",
-                        verbose_name="created on",
-                    ),
-                ),
-                (
-                    "updated_at",
-                    models.DateTimeField(
-                        auto_now=True,
-                        help_text="date and time at which a record was last updated",
-                        verbose_name="updated on",
-                    ),
-                ),
-                ("team", models.CharField(blank=True, max_length=100)),
-                (
-                    "role",
-                    models.CharField(
-                        choices=[
-                            ("reader", "Reader"),
-                            ("editor", "Editor"),
-                            ("administrator", "Administrator"),
-                            ("owner", "Owner"),
-                        ],
-                        default="reader",
-                        max_length=20,
-                    ),
-                ),
-                (
-                    "document",
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name="accesses",
-                        to="core.document",
-                    ),
-                ),
-                (
-                    "user",
-                    models.ForeignKey(
-                        blank=True,
-                        null=True,
-                        on_delete=django.db.models.deletion.CASCADE,
-                        to=settings.AUTH_USER_MODEL,
-                    ),
-                ),
+                ('id', models.UUIDField(default=uuid.uuid4, editable=False, help_text='primary key for the record as UUID', primary_key=True, serialize=False, verbose_name='id')),
+                ('created_at', models.DateTimeField(auto_now_add=True, help_text='date and time at which a record was created', verbose_name='created on')),
+                ('updated_at', models.DateTimeField(auto_now=True, help_text='date and time at which a record was last updated', verbose_name='updated on')),
+                ('team', models.CharField(blank=True, max_length=100)),
+                ('role', models.CharField(choices=[('reader', 'Reader'), ('editor', 'Editor'), ('administrator', 'Administrator'), ('owner', 'Owner')], default='reader', max_length=20)),
+                ('document', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='accesses', to='core.document')),
+                ('user', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
             ],
             options={
-                "verbose_name": "Document/user relation",
-                "verbose_name_plural": "Document/user relations",
-                "db_table": "impress_document_access",
-                "ordering": ("-created_at",),
+                'verbose_name': 'Document/user relation',
+                'verbose_name_plural': 'Document/user relations',
+                'db_table': 'impress_document_access',
+                'ordering': ('-created_at',),
             },
         ),
         migrations.CreateModel(
-            name="Invitation",
+            name='Invitation',
             fields=[
-                (
-                    "id",
-                    models.UUIDField(
-                        default=uuid.uuid4,
-                        editable=False,
-                        help_text="primary key for the record as UUID",
-                        primary_key=True,
-                        serialize=False,
-                        verbose_name="id",
-                    ),
-                ),
-                (
-                    "created_at",
-                    models.DateTimeField(
-                        auto_now_add=True,
-                        help_text="date and time at which a record was created",
-                        verbose_name="created on",
-                    ),
-                ),
-                (
-                    "updated_at",
-                    models.DateTimeField(
-                        auto_now=True,
-                        help_text="date and time at which a record was last updated",
-                        verbose_name="updated on",
-                    ),
-                ),
-                (
-                    "email",
-                    models.EmailField(max_length=254, verbose_name="email address"),
-                ),
-                (
-                    "role",
-                    models.CharField(
-                        choices=[
-                            ("reader", "Reader"),
-                            ("editor", "Editor"),
-                            ("administrator", "Administrator"),
-                            ("owner", "Owner"),
-                        ],
-                        default="reader",
-                        max_length=20,
-                    ),
-                ),
-                (
-                    "document",
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name="invitations",
-                        to="core.document",
-                    ),
-                ),
-                (
-                    "issuer",
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name="invitations",
-                        to=settings.AUTH_USER_MODEL,
-                    ),
-                ),
+                ('id', models.UUIDField(default=uuid.uuid4, editable=False, help_text='primary key for the record as UUID', primary_key=True, serialize=False, verbose_name='id')),
+                ('created_at', models.DateTimeField(auto_now_add=True, help_text='date and time at which a record was created', verbose_name='created on')),
+                ('updated_at', models.DateTimeField(auto_now=True, help_text='date and time at which a record was last updated', verbose_name='updated on')),
+                ('email', models.EmailField(max_length=254, verbose_name='email address')),
+                ('role', models.CharField(choices=[('reader', 'Reader'), ('editor', 'Editor'), ('administrator', 'Administrator'), ('owner', 'Owner')], default='reader', max_length=20)),
+                ('document', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='invitations', to='core.document')),
+                ('issuer', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='invitations', to=settings.AUTH_USER_MODEL)),
             ],
             options={
-                "verbose_name": "Document invitation",
-                "verbose_name_plural": "Document invitations",
-                "db_table": "impress_invitation",
+                'verbose_name': 'Document invitation',
+                'verbose_name_plural': 'Document invitations',
+                'db_table': 'impress_invitation',
             },
         ),
         migrations.CreateModel(
-            name="TemplateAccess",
+            name='TemplateAccess',
             fields=[
-                (
-                    "id",
-                    models.UUIDField(
-                        default=uuid.uuid4,
-                        editable=False,
-                        help_text="primary key for the record as UUID",
-                        primary_key=True,
-                        serialize=False,
-                        verbose_name="id",
-                    ),
-                ),
-                (
-                    "created_at",
-                    models.DateTimeField(
-                        auto_now_add=True,
-                        help_text="date and time at which a record was created",
-                        verbose_name="created on",
-                    ),
-                ),
-                (
-                    "updated_at",
-                    models.DateTimeField(
-                        auto_now=True,
-                        help_text="date and time at which a record was last updated",
-                        verbose_name="updated on",
-                    ),
-                ),
-                ("team", models.CharField(blank=True, max_length=100)),
-                (
-                    "role",
-                    models.CharField(
-                        choices=[
-                            ("reader", "Reader"),
-                            ("editor", "Editor"),
-                            ("administrator", "Administrator"),
-                            ("owner", "Owner"),
-                        ],
-                        default="reader",
-                        max_length=20,
-                    ),
-                ),
-                (
-                    "template",
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name="accesses",
-                        to="core.template",
-                    ),
-                ),
-                (
-                    "user",
-                    models.ForeignKey(
-                        blank=True,
-                        null=True,
-                        on_delete=django.db.models.deletion.CASCADE,
-                        to=settings.AUTH_USER_MODEL,
-                    ),
-                ),
+                ('id', models.UUIDField(default=uuid.uuid4, editable=False, help_text='primary key for the record as UUID', primary_key=True, serialize=False, verbose_name='id')),
+                ('created_at', models.DateTimeField(auto_now_add=True, help_text='date and time at which a record was created', verbose_name='created on')),
+                ('updated_at', models.DateTimeField(auto_now=True, help_text='date and time at which a record was last updated', verbose_name='updated on')),
+                ('team', models.CharField(blank=True, max_length=100)),
+                ('role', models.CharField(choices=[('reader', 'Reader'), ('editor', 'Editor'), ('administrator', 'Administrator'), ('owner', 'Owner')], default='reader', max_length=20)),
+                ('template', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='accesses', to='core.template')),
+                ('user', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
             ],
             options={
-                "verbose_name": "Template/user relation",
-                "verbose_name_plural": "Template/user relations",
-                "db_table": "impress_template_access",
-                "ordering": ("-created_at",),
+                'verbose_name': 'Template/user relation',
+                'verbose_name_plural': 'Template/user relations',
+                'db_table': 'impress_template_access',
+                'ordering': ('-created_at',),
             },
         ),
         migrations.AddConstraint(
-            model_name="documentaccess",
-            constraint=models.UniqueConstraint(
-                condition=models.Q(("user__isnull", False)),
-                fields=("user", "document"),
-                name="unique_document_user",
-                violation_error_message="This user is already in this document.",
-            ),
+            model_name='documentaccess',
+            constraint=models.UniqueConstraint(condition=models.Q(('user__isnull', False)), fields=('user', 'document'), name='unique_document_user', violation_error_message='This user is already in this document.'),
         ),
         migrations.AddConstraint(
-            model_name="documentaccess",
-            constraint=models.UniqueConstraint(
-                condition=models.Q(("team__gt", "")),
-                fields=("team", "document"),
-                name="unique_document_team",
-                violation_error_message="This team is already in this document.",
-            ),
+            model_name='documentaccess',
+            constraint=models.UniqueConstraint(condition=models.Q(('team__gt', '')), fields=('team', 'document'), name='unique_document_team', violation_error_message='This team is already in this document.'),
         ),
         migrations.AddConstraint(
-            model_name="documentaccess",
-            constraint=models.CheckConstraint(
-                check=models.Q(
-                    models.Q(("team", ""), ("user__isnull", False)),
-                    models.Q(("team__gt", ""), ("user__isnull", True)),
-                    _connector="OR",
-                ),
-                name="check_document_access_either_user_or_team",
-                violation_error_message="Either user or team must be set, not both.",
-            ),
+            model_name='documentaccess',
+            constraint=models.CheckConstraint(check=models.Q(models.Q(('team', ''), ('user__isnull', False)), models.Q(('team__gt', ''), ('user__isnull', True)), _connector='OR'), name='check_document_access_either_user_or_team', violation_error_message='Either user or team must be set, not both.'),
         ),
         migrations.AddConstraint(
-            model_name="invitation",
-            constraint=models.UniqueConstraint(
-                fields=("email", "document"), name="email_and_document_unique_together"
-            ),
+            model_name='invitation',
+            constraint=models.UniqueConstraint(fields=('email', 'document'), name='email_and_document_unique_together'),
         ),
         migrations.AddConstraint(
-            model_name="templateaccess",
-            constraint=models.UniqueConstraint(
-                condition=models.Q(("user__isnull", False)),
-                fields=("user", "template"),
-                name="unique_template_user",
-                violation_error_message="This user is already in this template.",
-            ),
+            model_name='templateaccess',
+            constraint=models.UniqueConstraint(condition=models.Q(('user__isnull', False)), fields=('user', 'template'), name='unique_template_user', violation_error_message='This user is already in this template.'),
         ),
         migrations.AddConstraint(
-            model_name="templateaccess",
-            constraint=models.UniqueConstraint(
-                condition=models.Q(("team__gt", "")),
-                fields=("team", "template"),
-                name="unique_template_team",
-                violation_error_message="This team is already in this template.",
-            ),
+            model_name='templateaccess',
+            constraint=models.UniqueConstraint(condition=models.Q(('team__gt', '')), fields=('team', 'template'), name='unique_template_team', violation_error_message='This team is already in this template.'),
         ),
         migrations.AddConstraint(
-            model_name="templateaccess",
-            constraint=models.CheckConstraint(
-                check=models.Q(
-                    models.Q(("team", ""), ("user__isnull", False)),
-                    models.Q(("team__gt", ""), ("user__isnull", True)),
-                    _connector="OR",
-                ),
-                name="check_template_access_either_user_or_team",
-                violation_error_message="Either user or team must be set, not both.",
-            ),
+            model_name='templateaccess',
+            constraint=models.CheckConstraint(check=models.Q(models.Q(('team', ''), ('user__isnull', False)), models.Q(('team__gt', ''), ('user__isnull', True)), _connector='OR'), name='check_template_access_either_user_or_team', violation_error_message='Either user or team must be set, not both.'),
         ),
     ]

src/backend/core/migrations/0002_create_pg_trgm_extension.py

@@ -1,9 +1,9 @@
 from django.db import migrations
 
-
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0001_initial"),
+        ('core', '0001_initial'),
     ]
 
     operations = [

src/backend/core/migrations/0003_document_link_reach_document_link_role_and_more.py

@@ -1,114 +1,52 @@
 # Generated by Django 5.1 on 2024-09-08 16:55
 
-import uuid
-
 import django.db.models.deletion
+import uuid
 from django.conf import settings
 from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0002_create_pg_trgm_extension"),
+        ('core', '0002_create_pg_trgm_extension'),
     ]
 
     operations = [
         migrations.AddField(
-            model_name="document",
-            name="link_reach",
-            field=models.CharField(
-                choices=[
-                    ("restricted", "Restricted"),
-                    ("authenticated", "Authenticated"),
-                    ("public", "Public"),
-                ],
-                default="authenticated",
-                max_length=20,
-            ),
+            model_name='document',
+            name='link_reach',
+            field=models.CharField(choices=[('restricted', 'Restricted'), ('authenticated', 'Authenticated'), ('public', 'Public')], default='authenticated', max_length=20),
         ),
         migrations.AddField(
-            model_name="document",
-            name="link_role",
-            field=models.CharField(
-                choices=[("reader", "Reader"), ("editor", "Editor")],
-                default="reader",
-                max_length=20,
-            ),
+            model_name='document',
+            name='link_role',
+            field=models.CharField(choices=[('reader', 'Reader'), ('editor', 'Editor')], default='reader', max_length=20),
         ),
         migrations.AlterField(
-            model_name="document",
-            name="is_public",
+            model_name='document',
+            name='is_public',
             field=models.BooleanField(null=True),
         ),
         migrations.AlterField(
-            model_name="user",
-            name="language",
-            field=models.CharField(
-                choices="(('en-us', 'English'), ('fr-fr', 'French'))",
-                default="en-us",
-                help_text="The language in which the user wants to see the interface.",
-                max_length=10,
-                verbose_name="language",
-            ),
+            model_name='user',
+            name='language',
+            field=models.CharField(choices="(('en-us', 'English'), ('fr-fr', 'French'))", default='en-us', help_text='The language in which the user wants to see the interface.', max_length=10, verbose_name='language'),
         ),
         migrations.CreateModel(
-            name="LinkTrace",
+            name='LinkTrace',
             fields=[
-                (
-                    "id",
-                    models.UUIDField(
-                        default=uuid.uuid4,
-                        editable=False,
-                        help_text="primary key for the record as UUID",
-                        primary_key=True,
-                        serialize=False,
-                        verbose_name="id",
-                    ),
-                ),
-                (
-                    "created_at",
-                    models.DateTimeField(
-                        auto_now_add=True,
-                        help_text="date and time at which a record was created",
-                        verbose_name="created on",
-                    ),
-                ),
-                (
-                    "updated_at",
-                    models.DateTimeField(
-                        auto_now=True,
-                        help_text="date and time at which a record was last updated",
-                        verbose_name="updated on",
-                    ),
-                ),
-                (
-                    "document",
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name="link_traces",
-                        to="core.document",
-                    ),
-                ),
-                (
-                    "user",
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name="link_traces",
-                        to=settings.AUTH_USER_MODEL,
-                    ),
-                ),
+                ('id', models.UUIDField(default=uuid.uuid4, editable=False, help_text='primary key for the record as UUID', primary_key=True, serialize=False, verbose_name='id')),
+                ('created_at', models.DateTimeField(auto_now_add=True, help_text='date and time at which a record was created', verbose_name='created on')),
+                ('updated_at', models.DateTimeField(auto_now=True, help_text='date and time at which a record was last updated', verbose_name='updated on')),
+                ('document', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='link_traces', to='core.document')),
+                ('user', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='link_traces', to=settings.AUTH_USER_MODEL)),
             ],
             options={
-                "verbose_name": "Document/user link trace",
-                "verbose_name_plural": "Document/user link traces",
-                "db_table": "impress_link_trace",
-                "constraints": [
-                    models.UniqueConstraint(
-                        fields=("user", "document"),
-                        name="unique_link_trace_document_user",
-                        violation_error_message="A link trace already exists for this document/user.",
-                    )
-                ],
+                'verbose_name': 'Document/user link trace',
+                'verbose_name_plural': 'Document/user link traces',
+                'db_table': 'impress_link_trace',
+                'constraints': [models.UniqueConstraint(fields=('user', 'document'), name='unique_link_trace_document_user', violation_error_message='A link trace already exists for this document/user.')],
             },
         ),
     ]

src/backend/core/migrations/0004_migrate_is_public_to_link_reach.py

@@ -1,14 +1,13 @@
 # Generated by Django 5.1 on 2024-09-08 17:04
 from django.db import migrations
 
-
 def migrate_is_public_to_link_reach(apps, schema_editor):
     """
     Forward migration: Migrate 'is_public' to 'link_reach'.
     If is_public == True, set link_reach to 'public'
     """
-    Document = apps.get_model("core", "Document")
-    Document.objects.filter(is_public=True).update(link_reach="public")
+    Document = apps.get_model('core', 'Document')
+    Document.objects.filter(is_public=True).update(link_reach='public')
 
 
 def reverse_migrate_link_reach_to_is_public(apps, schema_editor):
@@ -17,20 +16,20 @@ def reverse_migrate_link_reach_to_is_public(apps, schema_editor):
     - If link_reach == 'public', set is_public to True
     - Else set is_public to False
     """
-    Document = apps.get_model("core", "Document")
-    Document.objects.filter(link_reach="public").update(is_public=True)
-    Document.objects.filter(link_reach__in=["restricted", "authenticated"]).update(
-        is_public=False
-    )
+    Document = apps.get_model('core', 'Document')
+    Document.objects.filter(link_reach='public').update(is_public=True)
+    Document.objects.filter(link_reach__in=['restricted', "authenticated"]).update(is_public=False)
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0003_document_link_reach_document_link_role_and_more"),
+        ('core', '0003_document_link_reach_document_link_role_and_more'),
     ]
 
     operations = [
         migrations.RunPython(
-            migrate_is_public_to_link_reach, reverse_migrate_link_reach_to_is_public
+            migrate_is_public_to_link_reach, 
+            reverse_migrate_link_reach_to_is_public
         ),
     ]

src/backend/core/migrations/0005_remove_document_is_public_alter_document_link_reach_and_more.py

@@ -4,16 +4,15 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0004_migrate_is_public_to_link_reach"),
+        ('core', '0004_migrate_is_public_to_link_reach'),
     ]
 
     operations = [
         migrations.AlterField(
-            model_name="document",
-            name="title",
-            field=models.CharField(
-                blank=True, max_length=255, null=True, verbose_name="title"
-            ),
+            model_name='document',
+            name='title',
+            field=models.CharField(blank=True, max_length=255, null=True, verbose_name='title'),
         ),
     ]

src/backend/core/migrations/0006_add_user_full_name_and_short_name.py

@@ -4,34 +4,25 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0005_remove_document_is_public_alter_document_link_reach_and_more"),
+        ('core', '0005_remove_document_is_public_alter_document_link_reach_and_more'),
     ]
 
     operations = [
         migrations.AddField(
-            model_name="user",
-            name="full_name",
-            field=models.CharField(
-                blank=True, max_length=100, null=True, verbose_name="full name"
-            ),
+            model_name='user',
+            name='full_name',
+            field=models.CharField(blank=True, max_length=100, null=True, verbose_name='full name'),
         ),
         migrations.AddField(
-            model_name="user",
-            name="short_name",
-            field=models.CharField(
-                blank=True, max_length=20, null=True, verbose_name="short name"
-            ),
+            model_name='user',
+            name='short_name',
+            field=models.CharField(blank=True, max_length=20, null=True, verbose_name='short name'),
         ),
         migrations.AlterField(
-            model_name="user",
-            name="language",
-            field=models.CharField(
-                choices="(('en-us', 'English'), ('fr-fr', 'French'))",
-                default="en-us",
-                help_text="The language in which the user wants to see the interface.",
-                max_length=10,
-                verbose_name="language",
-            ),
+            model_name='user',
+            name='language',
+            field=models.CharField(choices="(('en-us', 'English'), ('fr-fr', 'French'))", default='en-us', help_text='The language in which the user wants to see the interface.', max_length=10, verbose_name='language'),
         ),
     ]

src/backend/core/migrations/0007_fix_users_duplicate.py

@@ -117,10 +117,10 @@ BEGIN
 END $$;
 """
 
-
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0006_add_user_full_name_and_short_name"),
+        ('core', '0006_add_user_full_name_and_short_name'),
     ]
 
     operations = [

src/backend/core/migrations/0008_alter_document_link_reach.py

@@ -4,22 +4,15 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0007_fix_users_duplicate"),
+        ('core', '0007_fix_users_duplicate'),
     ]
 
     operations = [
         migrations.AlterField(
-            model_name="document",
-            name="link_reach",
-            field=models.CharField(
-                choices=[
-                    ("restricted", "Restricted"),
-                    ("authenticated", "Authenticated"),
-                    ("public", "Public"),
-                ],
-                default="restricted",
-                max_length=20,
-            ),
+            model_name='document',
+            name='link_reach',
+            field=models.CharField(choices=[('restricted', 'Restricted'), ('authenticated', 'Authenticated'), ('public', 'Public')], default='restricted', max_length=20),
         ),
     ]

src/backend/core/migrations/0009_add_document_favorite.py

@@ -1,87 +1,37 @@
 # Generated by Django 5.1.2 on 2024-11-08 07:59
 
-import uuid
-
 import django.db.models.deletion
+import uuid
 from django.conf import settings
 from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0008_alter_document_link_reach"),
+        ('core', '0008_alter_document_link_reach'),
     ]
 
     operations = [
         migrations.AlterField(
-            model_name="user",
-            name="language",
-            field=models.CharField(
-                choices="(('en-us', 'English'), ('fr-fr', 'French'), ('de-de', 'German'))",
-                default="en-us",
-                help_text="The language in which the user wants to see the interface.",
-                max_length=10,
-                verbose_name="language",
-            ),
+            model_name='user',
+            name='language',
+            field=models.CharField(choices="(('en-us', 'English'), ('fr-fr', 'French'), ('de-de', 'German'))", default='en-us', help_text='The language in which the user wants to see the interface.', max_length=10, verbose_name='language'),
         ),
         migrations.CreateModel(
-            name="DocumentFavorite",
+            name='DocumentFavorite',
             fields=[
-                (
-                    "id",
-                    models.UUIDField(
-                        default=uuid.uuid4,
-                        editable=False,
-                        help_text="primary key for the record as UUID",
-                        primary_key=True,
-                        serialize=False,
-                        verbose_name="id",
-                    ),
-                ),
-                (
-                    "created_at",
-                    models.DateTimeField(
-                        auto_now_add=True,
-                        help_text="date and time at which a record was created",
-                        verbose_name="created on",
-                    ),
-                ),
-                (
-                    "updated_at",
-                    models.DateTimeField(
-                        auto_now=True,
-                        help_text="date and time at which a record was last updated",
-                        verbose_name="updated on",
-                    ),
-                ),
-                (
-                    "document",
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name="favorited_by_users",
-                        to="core.document",
-                    ),
-                ),
-                (
-                    "user",
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name="favorite_documents",
-                        to=settings.AUTH_USER_MODEL,
-                    ),
-                ),
+                ('id', models.UUIDField(default=uuid.uuid4, editable=False, help_text='primary key for the record as UUID', primary_key=True, serialize=False, verbose_name='id')),
+                ('created_at', models.DateTimeField(auto_now_add=True, help_text='date and time at which a record was created', verbose_name='created on')),
+                ('updated_at', models.DateTimeField(auto_now=True, help_text='date and time at which a record was last updated', verbose_name='updated on')),
+                ('document', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='favorited_by_users', to='core.document')),
+                ('user', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='favorite_documents', to=settings.AUTH_USER_MODEL)),
             ],
             options={
-                "verbose_name": "Document favorite",
-                "verbose_name_plural": "Document favorites",
-                "db_table": "impress_document_favorite",
-                "constraints": [
-                    models.UniqueConstraint(
-                        fields=("user", "document"),
-                        name="unique_document_favorite_user",
-                        violation_error_message="This document is already targeted by a favorite relation instance for the same user.",
-                    )
-                ],
+                'verbose_name': 'Document favorite',
+                'verbose_name_plural': 'Document favorites',
+                'db_table': 'impress_document_favorite',
+                'constraints': [models.UniqueConstraint(fields=('user', 'document'), name='unique_document_favorite_user', violation_error_message='This document is already targeted by a favorite relation instance for the same user.')],
             },
         ),
     ]

src/backend/core/migrations/0010_add_field_creator_to_document.py

@@ -7,48 +7,25 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0009_add_document_favorite"),
+        ('core', '0009_add_document_favorite'),
     ]
 
     operations = [
         migrations.AddField(
-            model_name="document",
-            name="creator",
-            field=models.ForeignKey(
-                null=True,
-                on_delete=django.db.models.deletion.RESTRICT,
-                related_name="documents_created",
-                to=settings.AUTH_USER_MODEL,
-            ),
+            model_name='document',
+            name='creator',
+            field=models.ForeignKey(null=True, on_delete=django.db.models.deletion.RESTRICT, related_name='documents_created', to=settings.AUTH_USER_MODEL),
         ),
         migrations.AlterField(
-            model_name="user",
-            name="language",
-            field=models.CharField(
-                choices="(('en-us', 'English'), ('fr-fr', 'French'), ('de-de', 'German'))",
-                default="en-us",
-                help_text="The language in which the user wants to see the interface.",
-                max_length=10,
-                verbose_name="language",
-            ),
+            model_name='user',
+            name='language',
+            field=models.CharField(choices="(('en-us', 'English'), ('fr-fr', 'French'), ('de-de', 'German'))", default='en-us', help_text='The language in which the user wants to see the interface.', max_length=10, verbose_name='language'),
         ),
         migrations.AlterField(
-            model_name="user",
-            name="sub",
-            field=models.CharField(
-                blank=True,
-                help_text="Required. 255 characters or fewer. Letters, numbers, and @/./+/-/_/: characters only.",
-                max_length=255,
-                null=True,
-                unique=True,
-                validators=[
-                    django.core.validators.RegexValidator(
-                        message="Enter a valid sub. This value may contain only letters, numbers, and @/./+/-/_/: characters.",
-                        regex="^[\\w.@+-:]+\\Z",
-                    )
-                ],
-                verbose_name="sub",
-            ),
+            model_name='user',
+            name='sub',
+            field=models.CharField(blank=True, help_text='Required. 255 characters or fewer. Letters, numbers, and @/./+/-/_/: characters only.', max_length=255, null=True, unique=True, validators=[django.core.validators.RegexValidator(message='Enter a valid sub. This value may contain only letters, numbers, and @/./+/-/_/: characters.', regex='^[\\w.@+-:]+\\Z')], verbose_name='sub'),
         ),
     ]

src/backend/core/migrations/0011_populate_creator_field_and_make_it_required.py

@@ -3,7 +3,7 @@
 import django.db.models.deletion
 from django.conf import settings
 from django.db import migrations
-from django.db.models import F, ForeignKey, OuterRef, Q, Subquery
+from django.db.models import F, ForeignKey, Subquery, OuterRef, Q
 
 
 def set_creator_from_document_access(apps, schema_editor):
@@ -25,37 +25,28 @@ def set_creator_from_document_access(apps, schema_editor):
     DocumentAccess = apps.get_model("core", "DocumentAccess")
 
     # Update `creator` using the "owner" role
-    owner_subquery = (
-        DocumentAccess.objects.filter(
-            document=OuterRef("pk"),
-            user__isnull=False,
-            role="owner",
-        )
-        .order_by("created_at")
-        .values("user_id")[:1]
-    )
+    owner_subquery = DocumentAccess.objects.filter(
+        document=OuterRef('pk'),
+        user__isnull=False,
+        role='owner',
+    ).order_by('created_at').values('user_id')[:1]
 
-    Document.objects.filter(creator__isnull=True).update(
-        creator=Subquery(owner_subquery)
-    )
+    Document.objects.filter(
+        creator__isnull=True
+    ).update(creator=Subquery(owner_subquery))
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0010_add_field_creator_to_document"),
+        ('core', '0010_add_field_creator_to_document'),
     ]
 
     operations = [
-        migrations.RunPython(
-            set_creator_from_document_access, reverse_code=migrations.RunPython.noop
-        ),
+        migrations.RunPython(set_creator_from_document_access, reverse_code=migrations.RunPython.noop),
         migrations.AlterField(
-            model_name="document",
-            name="creator",
-            field=ForeignKey(
-                on_delete=django.db.models.deletion.RESTRICT,
-                related_name="documents_created",
-                to=settings.AUTH_USER_MODEL,
-            ),
+            model_name='document',
+            name='creator',
+            field=ForeignKey(on_delete=django.db.models.deletion.RESTRICT, related_name='documents_created', to=settings.AUTH_USER_MODEL),
         ),
     ]

src/backend/core/migrations/0012_make_document_creator_and_invitation_issuer_optional.py

@@ -6,42 +6,25 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0011_populate_creator_field_and_make_it_required"),
+        ('core', '0011_populate_creator_field_and_make_it_required'),
     ]
 
     operations = [
         migrations.AlterField(
-            model_name="document",
-            name="creator",
-            field=models.ForeignKey(
-                blank=True,
-                null=True,
-                on_delete=django.db.models.deletion.RESTRICT,
-                related_name="documents_created",
-                to=settings.AUTH_USER_MODEL,
-            ),
+            model_name='document',
+            name='creator',
+            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.RESTRICT, related_name='documents_created', to=settings.AUTH_USER_MODEL),
         ),
         migrations.AlterField(
-            model_name="invitation",
-            name="issuer",
-            field=models.ForeignKey(
-                blank=True,
-                null=True,
-                on_delete=django.db.models.deletion.CASCADE,
-                related_name="invitations",
-                to=settings.AUTH_USER_MODEL,
-            ),
+            model_name='invitation',
+            name='issuer',
+            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.CASCADE, related_name='invitations', to=settings.AUTH_USER_MODEL),
         ),
         migrations.AlterField(
-            model_name="user",
-            name="language",
-            field=models.CharField(
-                choices="(('en-us', 'English'), ('fr-fr', 'French'), ('de-de', 'German'))",
-                default="en-us",
-                help_text="The language in which the user wants to see the interface.",
-                max_length=10,
-                verbose_name="language",
-            ),
+            model_name='user',
+            name='language',
+            field=models.CharField(choices="(('en-us', 'English'), ('fr-fr', 'French'), ('de-de', 'German'))", default='en-us', help_text='The language in which the user wants to see the interface.', max_length=10, verbose_name='language'),
         ),
     ]

src/backend/core/migrations/0013_activate_fuzzystrmatch_extension.py

@@ -2,10 +2,10 @@
 
 from django.db import migrations
 
-
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0012_make_document_creator_and_invitation_issuer_optional"),
+        ('core', '0012_make_document_creator_and_invitation_issuer_optional'),
     ]
 
     operations = [

src/backend/core/migrations/0014_add_tree_structure_to_documents.py

@@ -4,29 +4,28 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0013_activate_fuzzystrmatch_extension"),
+        ('core', '0013_activate_fuzzystrmatch_extension'),
     ]
 
     operations = [
         migrations.AddField(
-            model_name="document",
-            name="depth",
+            model_name='document',
+            name='depth',
             field=models.PositiveIntegerField(default=0),
             preserve_default=False,
         ),
         migrations.AddField(
-            model_name="document",
-            name="numchild",
+            model_name='document',
+            name='numchild',
             field=models.PositiveIntegerField(default=0),
         ),
         migrations.AddField(
-            model_name="document",
-            name="path",
+            model_name='document',
+            name='path',
             # Allow null values pending the next datamigration to populate the field
-            field=models.CharField(
-                db_collation="C", max_length=252, null=True, unique=True
-            ),
+            field=models.CharField(db_collation='C', max_length=252, null=True, unique=True),
             preserve_default=False,
         ),
     ]

src/backend/core/migrations/0015_set_path_on_existing_documents.py

@@ -7,10 +7,9 @@ from treebeard.numconv import NumConv
 ALPHABET = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz"
 STEPLEN = 7
 
-
 def set_path_on_existing_documents(apps, schema_editor):
     """
-    Updates the `path` and `depth` fields for all existing Document records
+    Updates the `path` and `depth` fields for all existing Document records 
     to ensure valid materialized paths.
 
     This function assigns a unique `path` to each Document as a root node
@@ -27,25 +26,27 @@ def set_path_on_existing_documents(apps, schema_editor):
     updates = []
     for i, pk in enumerate(documents):
         key = numconv.int2str(i)
-        path = "{0}{1}".format(ALPHABET[0] * (STEPLEN - len(key)), key)
+        path = "{0}{1}".format(
+            ALPHABET[0] * (STEPLEN - len(key)),
+            key
+        )
         updates.append(Document(pk=pk, path=path, depth=1))
 
     # Bulk update using the prepared updates list
-    Document.objects.bulk_update(updates, ["depth", "path"])
+    Document.objects.bulk_update(updates, ['depth', 'path'])
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0014_add_tree_structure_to_documents"),
+        ('core', '0014_add_tree_structure_to_documents'),
     ]
 
     operations = [
-        migrations.RunPython(
-            set_path_on_existing_documents, reverse_code=migrations.RunPython.noop
-        ),
+        migrations.RunPython(set_path_on_existing_documents, reverse_code=migrations.RunPython.noop),
         migrations.AlterField(
-            model_name="document",
-            name="path",
-            field=models.CharField(db_collation="C", max_length=252, unique=True),
+            model_name='document',
+            name='path',
+            field=models.CharField(db_collation='C', max_length=252, unique=True),
         ),
     ]

src/backend/core/migrations/0016_add_document_excerpt.py

@@ -4,27 +4,20 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0015_set_path_on_existing_documents"),
+        ('core', '0015_set_path_on_existing_documents'),
     ]
 
     operations = [
         migrations.AddField(
-            model_name="document",
-            name="excerpt",
-            field=models.TextField(
-                blank=True, max_length=300, null=True, verbose_name="excerpt"
-            ),
+            model_name='document',
+            name='excerpt',
+            field=models.TextField(blank=True, max_length=300, null=True, verbose_name='excerpt'),
         ),
         migrations.AlterField(
-            model_name="user",
-            name="language",
-            field=models.CharField(
-                choices="(('en-us', 'English'), ('fr-fr', 'French'), ('de-de', 'German'))",
-                default="en-us",
-                help_text="The language in which the user wants to see the interface.",
-                max_length=10,
-                verbose_name="language",
-            ),
+            model_name='user',
+            name='language',
+            field=models.CharField(choices="(('en-us', 'English'), ('fr-fr', 'French'), ('de-de', 'German'))", default='en-us', help_text='The language in which the user wants to see the interface.', max_length=10, verbose_name='language'),
         ),
     ]

src/backend/core/migrations/0017_add_fields_for_soft_delete.py

@@ -4,49 +4,33 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     dependencies = [
-        ("core", "0016_add_document_excerpt"),
+        ('core', '0016_add_document_excerpt'),
     ]
 
     operations = [
         migrations.AlterModelOptions(
-            name="document",
-            options={
-                "ordering": ("path",),
-                "verbose_name": "Document",
-                "verbose_name_plural": "Documents",
-            },
+            name='document',
+            options={'ordering': ('path',), 'verbose_name': 'Document', 'verbose_name_plural': 'Documents'},
         ),
         migrations.AddField(
-            model_name="document",
-            name="ancestors_deleted_at",
+            model_name='document',
+            name='ancestors_deleted_at',
             field=models.DateTimeField(blank=True, null=True),
         ),
         migrations.AddField(
-            model_name="document",
-            name="deleted_at",
+            model_name='document',
+            name='deleted_at',
             field=models.DateTimeField(blank=True, null=True),
         ),
         migrations.AlterField(
-            model_name="user",
-            name="language",
-            field=models.CharField(
-                choices="(('en-us', 'English'), ('fr-fr', 'French'), ('de-de', 'German'))",
-                default="en-us",
-                help_text="The language in which the user wants to see the interface.",
-                max_length=10,
-                verbose_name="language",
-            ),
+            model_name='user',
+            name='language',
+            field=models.CharField(choices="(('en-us', 'English'), ('fr-fr', 'French'), ('de-de', 'German'))", default='en-us', help_text='The language in which the user wants to see the interface.', max_length=10, verbose_name='language'),
         ),
         migrations.AddConstraint(
-            model_name="document",
-            constraint=models.CheckConstraint(
-                condition=models.Q(
-                    ("deleted_at__isnull", True),
-                    ("deleted_at", models.F("ancestors_deleted_at")),
-                    _connector="OR",
-                ),
-                name="check_deleted_at_matches_ancestors_deleted_at_when_set",
-            ),
+            model_name='document',
+            constraint=models.CheckConstraint(condition=models.Q(('deleted_at__isnull', True), ('deleted_at', models.F('ancestors_deleted_at')), _connector='OR'), name='check_deleted_at_matches_ancestors_deleted_at_when_set'),
         ),
     ]

src/backend/core/migrations/0018_update_blank_title.py

@@ -1,24 +0,0 @@
-from django.db import migrations
-
-
-def update_titles_to_null(apps, schema_editor):
-    """
-    If the titles are "Untitled document" or "Unbenanntes Dokument" or "Document sans titre"
-    we set them to Null
-    """
-    Document = apps.get_model("core", "Document")
-    Document.objects.filter(
-        title__in=["Untitled document", "Unbenanntes Dokument", "Document sans titre"]
-    ).update(title=None)
-
-
-class Migration(migrations.Migration):
-    dependencies = [
-        ("core", "0017_add_fields_for_soft_delete"),
-    ]
-
-    operations = [
-        migrations.RunPython(
-            update_titles_to_null, reverse_code=migrations.RunPython.noop
-        ),
-    ]

src/backend/core/models.py

@@ -697,7 +697,6 @@ class Document(MP_Node, BaseModel):
                 "document": self,
                 "domain": domain,
                 "link": f"{domain}/docs/{self.id}/",
-                "document_title": self.title or str(_("Untitled Document")),
                 "logo_img": settings.EMAIL_LOGO_IMG,
             }
         )
@@ -739,12 +738,8 @@ class Document(MP_Node, BaseModel):
                     '{name} invited you with the role "{role}" on the following document:'
                 ).format(name=sender_name_email, role=role.lower()),
             }
-            subject = (
-                context["title"]
-                if not self.title
-                else _("{name} shared a document with you: {title}").format(
-                    name=sender_name, title=self.title
-                )
+            subject = _("{name} shared a document with you: {title}").format(
+                name=sender_name, title=self.title
             )
 
         self.send_email(subject, [email], context, language)
@@ -799,11 +794,9 @@ class Document(MP_Node, BaseModel):
         ancestors_deleted_at = (
             self.get_ancestors()
             .filter(deleted_at__isnull=False)
-            .order_by("deleted_at")
             .values_list("deleted_at", flat=True)
-            .first()
         )
-        self.ancestors_deleted_at = ancestors_deleted_at
+        self.ancestors_deleted_at = min(ancestors_deleted_at, default=None)
         self.save()
 
         # Update descendants excluding those who were deleted prior to the deletion of the

src/backend/core/tests/documents/test_api_documents_attachment_upload.py

@@ -79,7 +79,6 @@ def test_api_documents_attachment_upload_anonymous_success():
 
     assert file_head["Metadata"] == {"owner": "None"}
     assert file_head["ContentType"] == "image/png"
-    assert file_head["ContentDisposition"] == 'inline; filename="test.png"'
 
 
 @pytest.mark.parametrize(
@@ -218,7 +217,6 @@ def test_api_documents_attachment_upload_success(via, role, mock_user_teams):
     )
     assert file_head["Metadata"] == {"owner": str(user.id)}
     assert file_head["ContentType"] == "image/png"
-    assert file_head["ContentDisposition"] == 'inline; filename="test.png"'
 
 
 def test_api_documents_attachment_upload_invalid(client):
@@ -293,9 +291,7 @@ def test_api_documents_attachment_upload_fix_extension(
     match = pattern.search(file_path)
     file_id = match.group(1)
 
-    assert "-unsafe" in file_id
     # Validate that file_id is a valid UUID
-    file_id = file_id.replace("-unsafe", "")
     uuid.UUID(file_id)
 
     # Now, check the metadata of the uploaded file
@@ -305,7 +301,6 @@ def test_api_documents_attachment_upload_fix_extension(
     )
     assert file_head["Metadata"] == {"owner": str(user.id), "is_unsafe": "true"}
     assert file_head["ContentType"] == content_type
-    assert file_head["ContentDisposition"] == f'attachment; filename="{name:s}"'
 
 
 def test_api_documents_attachment_upload_empty_file():
@@ -345,9 +340,7 @@ def test_api_documents_attachment_upload_unsafe():
     match = pattern.search(file_path)
     file_id = match.group(1)
 
-    assert "-unsafe" in file_id
     # Validate that file_id is a valid UUID
-    file_id = file_id.replace("-unsafe", "")
     uuid.UUID(file_id)
 
     # Now, check the metadata of the uploaded file
@@ -357,4 +350,3 @@ def test_api_documents_attachment_upload_unsafe():
     )
     assert file_head["Metadata"] == {"owner": str(user.id), "is_unsafe": "true"}
     assert file_head["ContentType"] == "application/octet-stream"
-    assert file_head["ContentDisposition"] == 'attachment; filename="script.exe"'

src/backend/core/tests/documents/test_api_documents_media_auth.py

@@ -64,30 +64,6 @@ def test_api_documents_media_auth_anonymous_public():
     assert response.content.decode("utf-8") == "my prose"
 
 
-def test_api_documents_media_auth_extensions():
-    """Files with extensions of any format should work."""
-    document = factories.DocumentFactory(link_reach="public")
-
-    extensions = [
-        "c",
-        "go",
-        "gif",
-        "mp4",
-        "woff2",
-        "appimage",
-    ]
-    for ext in extensions:
-        filename = f"{uuid.uuid4()!s}.{ext:s}"
-        key = f"{document.pk!s}/attachments/{filename:s}"
-
-        original_url = f"http://localhost/media/{key:s}"
-        response = APIClient().get(
-            "/api/v1.0/documents/media-auth/", HTTP_X_ORIGINAL_URL=original_url
-        )
-
-        assert response.status_code == 200
-
-
 @pytest.mark.parametrize("reach", ["authenticated", "restricted"])
 def test_api_documents_media_auth_anonymous_authenticated_or_restricted(reach):
     """

src/backend/core/tests/migrations/test_0018_update_blank_title.py

@@ -1,35 +0,0 @@
-import pytest
-
-from core import factories
-
-
-@pytest.mark.django_db
-def test_update_blank_title_migration(migrator):
-    """
-    Test that the migration fixes the titles of documents that are
-    "Untitled document", "Unbenanntes Dokument" or "Document sans titre"
-    """
-    migrator.apply_initial_migration(("core", "0017_add_fields_for_soft_delete"))
-
-    english_doc = factories.DocumentFactory(title="Untitled document")
-    german_doc = factories.DocumentFactory(title="Unbenanntes Dokument")
-    french_doc = factories.DocumentFactory(title="Document sans titre")
-    other_doc = factories.DocumentFactory(title="My document")
-
-    assert english_doc.title == "Untitled document"
-    assert german_doc.title == "Unbenanntes Dokument"
-    assert french_doc.title == "Document sans titre"
-    assert other_doc.title == "My document"
-
-    # Apply the migration
-    migrator.apply_tested_migration(("core", "0018_update_blank_title"))
-
-    english_doc.refresh_from_db()
-    german_doc.refresh_from_db()
-    french_doc.refresh_from_db()
-    other_doc.refresh_from_db()
-
-    assert english_doc.title == None
-    assert german_doc.title == None
-    assert french_doc.title == None
-    assert other_doc.title == "My document"

src/backend/core/tests/test_models_documents.py

@@ -636,37 +636,6 @@ def test_models_documents__email_invitation__success():
     assert f"docs/{document.id}/" in email_content
 
 
-def test_models_documents__email_invitation__success_empty_title():
-    """
-    The email invitation is sent successfully.
-    """
-    document = factories.DocumentFactory(title=None)
-
-    # pylint: disable-next=no-member
-    assert len(mail.outbox) == 0
-
-    sender = factories.UserFactory(full_name="Test Sender", email="sender@example.com")
-    document.send_invitation_email(
-        "guest@example.com", models.RoleChoices.EDITOR, sender, "en"
-    )
-
-    # pylint: disable-next=no-member
-    assert len(mail.outbox) == 1
-
-    # pylint: disable-next=no-member
-    email = mail.outbox[0]
-
-    assert email.to == ["guest@example.com"]
-    email_content = " ".join(email.body.split())
-
-    assert "Test sender shared a document with you!" in email.subject
-    assert (
-        "Test Sender (sender@example.com) invited you with the role "editor" "
-        "on the following document: Untitled Document" in email_content
-    )
-    assert f"docs/{document.id}/" in email_content
-
-
 def test_models_documents__email_invitation__success_fr():
     """
     The email invitation is sent successfully in french.

src/backend/impress/settings.py

@@ -210,6 +210,7 @@ class Base(Configuration):
         "application/x-ms-regedit",
         "application/x-msdownload",
         "application/xml",
+        "image/svg+xml",
     ]
 
     # Document versions

src/backend/locale/de_DE/LC_MESSAGES/django.po

@@ -2,8 +2,8 @@ msgid ""
 msgstr ""
 "Project-Id-Version: lasuite-docs\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2025-03-03 12:20+0000\n"
-"PO-Revision-Date: 2025-03-03 12:22\n"
+"POT-Creation-Date: 2025-02-06 15:30+0000\n"
+"PO-Revision-Date: 2025-02-10 14:14\n"
 "Last-Translator: \n"
 "Language-Team: German\n"
 "Language: de_DE\n"
@@ -54,15 +54,15 @@ msgstr "Ein neues Dokument wurde in Ihrem Namen erstellt!"
 msgid "You have been granted ownership of a new document:"
 msgstr "Sie sind Besitzer eines neuen Dokuments:"
 
-#: build/lib/core/api/serializers.py:455 core/api/serializers.py:455
+#: build/lib/core/api/serializers.py:453 core/api/serializers.py:453
 msgid "Body"
 msgstr "Inhalt"
 
-#: build/lib/core/api/serializers.py:458 core/api/serializers.py:458
+#: build/lib/core/api/serializers.py:456 core/api/serializers.py:456
 msgid "Body type"
 msgstr "Typ"
 
-#: build/lib/core/api/serializers.py:464 core/api/serializers.py:464
+#: build/lib/core/api/serializers.py:462 core/api/serializers.py:462
 msgid "Format"
 msgstr ""
 
@@ -230,8 +230,8 @@ msgstr "Benutzer"
 msgid "users"
 msgstr "Benutzer"
 
-#: build/lib/core/models.py:373 build/lib/core/models.py:942 core/models.py:373
-#: core/models.py:942
+#: build/lib/core/models.py:373 build/lib/core/models.py:925 core/models.py:373
+#: core/models.py:925
 msgid "title"
 msgstr "Titel"
 
@@ -251,143 +251,143 @@ msgstr "Dokumente"
 msgid "Untitled Document"
 msgstr "Unbenanntes Dokument"
 
-#: build/lib/core/models.py:734 core/models.py:734
+#: build/lib/core/models.py:719 core/models.py:719
 #, python-brace-format
 msgid "{name} shared a document with you!"
 msgstr "{name} hat ein Dokument mit Ihnen geteilt!"
 
-#: build/lib/core/models.py:738 core/models.py:738
+#: build/lib/core/models.py:723 core/models.py:723
 #, python-brace-format
 msgid "{name} invited you with the role \"{role}\" on the following document:"
 msgstr "{name} hat Sie mit der Rolle \"{role}\" zu folgendem Dokument eingeladen:"
 
-#: build/lib/core/models.py:741 core/models.py:741
+#: build/lib/core/models.py:726 core/models.py:726
 #, python-brace-format
 msgid "{name} shared a document with you: {title}"
 msgstr "{name} hat ein Dokument mit Ihnen geteilt: {title}"
 
-#: build/lib/core/models.py:777 core/models.py:777
+#: build/lib/core/models.py:762 core/models.py:762
 msgid "This document is not deleted."
 msgstr ""
 
-#: build/lib/core/models.py:784 core/models.py:784
+#: build/lib/core/models.py:769 core/models.py:769
 msgid "This document was permanently deleted and cannot be restored."
 msgstr ""
 
-#: build/lib/core/models.py:837 core/models.py:837
+#: build/lib/core/models.py:820 core/models.py:820
 msgid "Document/user link trace"
 msgstr "Dokument/Benutzer Linkverfolgung"
 
-#: build/lib/core/models.py:838 core/models.py:838
+#: build/lib/core/models.py:821 core/models.py:821
 msgid "Document/user link traces"
 msgstr "Dokument/Benutzer Linkverfolgung"
 
-#: build/lib/core/models.py:844 core/models.py:844
+#: build/lib/core/models.py:827 core/models.py:827
 msgid "A link trace already exists for this document/user."
 msgstr ""
 
-#: build/lib/core/models.py:867 core/models.py:867
+#: build/lib/core/models.py:850 core/models.py:850
 msgid "Document favorite"
 msgstr "Dokumentenfavorit"
 
-#: build/lib/core/models.py:868 core/models.py:868
+#: build/lib/core/models.py:851 core/models.py:851
 msgid "Document favorites"
 msgstr "Dokumentfavoriten"
 
-#: build/lib/core/models.py:874 core/models.py:874
+#: build/lib/core/models.py:857 core/models.py:857
 msgid "This document is already targeted by a favorite relation instance for the same user."
 msgstr "Dieses Dokument ist bereits durch den gleichen Benutzer favorisiert worden."
 
-#: build/lib/core/models.py:896 core/models.py:896
+#: build/lib/core/models.py:879 core/models.py:879
 msgid "Document/user relation"
 msgstr "Dokument/Benutzerbeziehung"
 
-#: build/lib/core/models.py:897 core/models.py:897
+#: build/lib/core/models.py:880 core/models.py:880
 msgid "Document/user relations"
 msgstr "Dokument/Benutzerbeziehungen"
 
-#: build/lib/core/models.py:903 core/models.py:903
+#: build/lib/core/models.py:886 core/models.py:886
 msgid "This user is already in this document."
 msgstr "Dieser Benutzer befindet sich bereits in diesem Dokument."
 
-#: build/lib/core/models.py:909 core/models.py:909
+#: build/lib/core/models.py:892 core/models.py:892
 msgid "This team is already in this document."
 msgstr "Dieses Team befindet sich bereits in diesem Dokument."
 
-#: build/lib/core/models.py:915 build/lib/core/models.py:1029
-#: core/models.py:915 core/models.py:1029
+#: build/lib/core/models.py:898 build/lib/core/models.py:1012
+#: core/models.py:898 core/models.py:1012
 msgid "Either user or team must be set, not both."
 msgstr "Benutzer oder Team müssen gesetzt werden, nicht beides."
 
-#: build/lib/core/models.py:943 core/models.py:943
+#: build/lib/core/models.py:926 core/models.py:926
 msgid "description"
 msgstr "Beschreibung"
 
-#: build/lib/core/models.py:944 core/models.py:944
+#: build/lib/core/models.py:927 core/models.py:927
 msgid "code"
 msgstr "Code"
 
-#: build/lib/core/models.py:945 core/models.py:945
+#: build/lib/core/models.py:928 core/models.py:928
 msgid "css"
 msgstr "CSS"
 
-#: build/lib/core/models.py:947 core/models.py:947
+#: build/lib/core/models.py:930 core/models.py:930
 msgid "public"
 msgstr "öffentlich"
 
-#: build/lib/core/models.py:949 core/models.py:949
+#: build/lib/core/models.py:932 core/models.py:932
 msgid "Whether this template is public for anyone to use."
 msgstr "Ob diese Vorlage für jedermann öffentlich ist."
 
-#: build/lib/core/models.py:955 core/models.py:955
+#: build/lib/core/models.py:938 core/models.py:938
 msgid "Template"
 msgstr "Vorlage"
 
-#: build/lib/core/models.py:956 core/models.py:956
+#: build/lib/core/models.py:939 core/models.py:939
 msgid "Templates"
 msgstr "Vorlagen"
 
-#: build/lib/core/models.py:1010 core/models.py:1010
+#: build/lib/core/models.py:993 core/models.py:993
 msgid "Template/user relation"
 msgstr "Vorlage/Benutzer-Beziehung"
 
-#: build/lib/core/models.py:1011 core/models.py:1011
+#: build/lib/core/models.py:994 core/models.py:994
 msgid "Template/user relations"
 msgstr "Vorlage/Benutzerbeziehungen"
 
-#: build/lib/core/models.py:1017 core/models.py:1017
+#: build/lib/core/models.py:1000 core/models.py:1000
 msgid "This user is already in this template."
 msgstr "Dieser Benutzer ist bereits in dieser Vorlage."
 
-#: build/lib/core/models.py:1023 core/models.py:1023
+#: build/lib/core/models.py:1006 core/models.py:1006
 msgid "This team is already in this template."
 msgstr "Dieses Team ist bereits in diesem Template."
 
-#: build/lib/core/models.py:1046 core/models.py:1046
+#: build/lib/core/models.py:1029 core/models.py:1029
 msgid "email address"
 msgstr "E-Mail-Adresse"
 
-#: build/lib/core/models.py:1065 core/models.py:1065
+#: build/lib/core/models.py:1048 core/models.py:1048
 msgid "Document invitation"
 msgstr "Einladung zum Dokument"
 
-#: build/lib/core/models.py:1066 core/models.py:1066
+#: build/lib/core/models.py:1049 core/models.py:1049
 msgid "Document invitations"
 msgstr "Dokumenteinladungen"
 
-#: build/lib/core/models.py:1086 core/models.py:1086
+#: build/lib/core/models.py:1069 core/models.py:1069
 msgid "This email is already associated to a registered user."
 msgstr "Diese E-Mail ist bereits einem registrierten Benutzer zugeordnet."
 
-#: build/lib/impress/settings.py:235 impress/settings.py:235
+#: build/lib/impress/settings.py:236 impress/settings.py:236
 msgid "English"
 msgstr "Englisch"
 
-#: build/lib/impress/settings.py:236 impress/settings.py:236
+#: build/lib/impress/settings.py:237 impress/settings.py:237
 msgid "French"
 msgstr "Französisch"
 
-#: build/lib/impress/settings.py:237 impress/settings.py:237
+#: build/lib/impress/settings.py:238 impress/settings.py:238
 msgid "German"
 msgstr "Deutsch"
 

src/backend/locale/en_US/LC_MESSAGES/django.po

@@ -2,8 +2,8 @@ msgid ""
 msgstr ""
 "Project-Id-Version: lasuite-docs\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2025-03-03 12:20+0000\n"
-"PO-Revision-Date: 2025-03-03 12:22\n"
+"POT-Creation-Date: 2025-02-06 15:30+0000\n"
+"PO-Revision-Date: 2025-02-10 14:14\n"
 "Last-Translator: \n"
 "Language-Team: English\n"
 "Language: en_US\n"
@@ -54,15 +54,15 @@ msgstr ""
 msgid "You have been granted ownership of a new document:"
 msgstr ""
 
-#: build/lib/core/api/serializers.py:455 core/api/serializers.py:455
+#: build/lib/core/api/serializers.py:453 core/api/serializers.py:453
 msgid "Body"
 msgstr ""
 
-#: build/lib/core/api/serializers.py:458 core/api/serializers.py:458
+#: build/lib/core/api/serializers.py:456 core/api/serializers.py:456
 msgid "Body type"
 msgstr ""
 
-#: build/lib/core/api/serializers.py:464 core/api/serializers.py:464
+#: build/lib/core/api/serializers.py:462 core/api/serializers.py:462
 msgid "Format"
 msgstr ""
 
@@ -230,8 +230,8 @@ msgstr ""
 msgid "users"
 msgstr ""
 
-#: build/lib/core/models.py:373 build/lib/core/models.py:942 core/models.py:373
-#: core/models.py:942
+#: build/lib/core/models.py:373 build/lib/core/models.py:925 core/models.py:373
+#: core/models.py:925
 msgid "title"
 msgstr ""
 
@@ -251,143 +251,143 @@ msgstr ""
 msgid "Untitled Document"
 msgstr ""
 
-#: build/lib/core/models.py:734 core/models.py:734
+#: build/lib/core/models.py:719 core/models.py:719
 #, python-brace-format
 msgid "{name} shared a document with you!"
 msgstr ""
 
-#: build/lib/core/models.py:738 core/models.py:738
+#: build/lib/core/models.py:723 core/models.py:723
 #, python-brace-format
 msgid "{name} invited you with the role \"{role}\" on the following document:"
 msgstr ""
 
-#: build/lib/core/models.py:741 core/models.py:741
+#: build/lib/core/models.py:726 core/models.py:726
 #, python-brace-format
 msgid "{name} shared a document with you: {title}"
 msgstr ""
 
-#: build/lib/core/models.py:777 core/models.py:777
+#: build/lib/core/models.py:762 core/models.py:762
 msgid "This document is not deleted."
 msgstr ""
 
-#: build/lib/core/models.py:784 core/models.py:784
+#: build/lib/core/models.py:769 core/models.py:769
 msgid "This document was permanently deleted and cannot be restored."
 msgstr ""
 
-#: build/lib/core/models.py:837 core/models.py:837
+#: build/lib/core/models.py:820 core/models.py:820
 msgid "Document/user link trace"
 msgstr ""
 
-#: build/lib/core/models.py:838 core/models.py:838
+#: build/lib/core/models.py:821 core/models.py:821
 msgid "Document/user link traces"
 msgstr ""
 
-#: build/lib/core/models.py:844 core/models.py:844
+#: build/lib/core/models.py:827 core/models.py:827
 msgid "A link trace already exists for this document/user."
 msgstr ""
 
-#: build/lib/core/models.py:867 core/models.py:867
+#: build/lib/core/models.py:850 core/models.py:850
 msgid "Document favorite"
 msgstr ""
 
-#: build/lib/core/models.py:868 core/models.py:868
+#: build/lib/core/models.py:851 core/models.py:851
 msgid "Document favorites"
 msgstr ""
 
-#: build/lib/core/models.py:874 core/models.py:874
+#: build/lib/core/models.py:857 core/models.py:857
 msgid "This document is already targeted by a favorite relation instance for the same user."
 msgstr ""
 
-#: build/lib/core/models.py:896 core/models.py:896
+#: build/lib/core/models.py:879 core/models.py:879
 msgid "Document/user relation"
 msgstr ""
 
-#: build/lib/core/models.py:897 core/models.py:897
+#: build/lib/core/models.py:880 core/models.py:880
 msgid "Document/user relations"
 msgstr ""
 
-#: build/lib/core/models.py:903 core/models.py:903
+#: build/lib/core/models.py:886 core/models.py:886
 msgid "This user is already in this document."
 msgstr ""
 
-#: build/lib/core/models.py:909 core/models.py:909
+#: build/lib/core/models.py:892 core/models.py:892
 msgid "This team is already in this document."
 msgstr ""
 
-#: build/lib/core/models.py:915 build/lib/core/models.py:1029
-#: core/models.py:915 core/models.py:1029
+#: build/lib/core/models.py:898 build/lib/core/models.py:1012
+#: core/models.py:898 core/models.py:1012
 msgid "Either user or team must be set, not both."
 msgstr ""
 
-#: build/lib/core/models.py:943 core/models.py:943
+#: build/lib/core/models.py:926 core/models.py:926
 msgid "description"
 msgstr ""
 
-#: build/lib/core/models.py:944 core/models.py:944
+#: build/lib/core/models.py:927 core/models.py:927
 msgid "code"
 msgstr ""
 
-#: build/lib/core/models.py:945 core/models.py:945
+#: build/lib/core/models.py:928 core/models.py:928
 msgid "css"
 msgstr ""
 
-#: build/lib/core/models.py:947 core/models.py:947
+#: build/lib/core/models.py:930 core/models.py:930
 msgid "public"
 msgstr ""
 
-#: build/lib/core/models.py:949 core/models.py:949
+#: build/lib/core/models.py:932 core/models.py:932
 msgid "Whether this template is public for anyone to use."
 msgstr ""
 
-#: build/lib/core/models.py:955 core/models.py:955
+#: build/lib/core/models.py:938 core/models.py:938
 msgid "Template"
 msgstr ""
 
-#: build/lib/core/models.py:956 core/models.py:956
+#: build/lib/core/models.py:939 core/models.py:939
 msgid "Templates"
 msgstr ""
 
-#: build/lib/core/models.py:1010 core/models.py:1010
+#: build/lib/core/models.py:993 core/models.py:993
 msgid "Template/user relation"
 msgstr ""
 
-#: build/lib/core/models.py:1011 core/models.py:1011
+#: build/lib/core/models.py:994 core/models.py:994
 msgid "Template/user relations"
 msgstr ""
 
-#: build/lib/core/models.py:1017 core/models.py:1017
+#: build/lib/core/models.py:1000 core/models.py:1000
 msgid "This user is already in this template."
 msgstr ""
 
-#: build/lib/core/models.py:1023 core/models.py:1023
+#: build/lib/core/models.py:1006 core/models.py:1006
 msgid "This team is already in this template."
 msgstr ""
 
-#: build/lib/core/models.py:1046 core/models.py:1046
+#: build/lib/core/models.py:1029 core/models.py:1029
 msgid "email address"
 msgstr ""
 
-#: build/lib/core/models.py:1065 core/models.py:1065
+#: build/lib/core/models.py:1048 core/models.py:1048
 msgid "Document invitation"
 msgstr ""
 
-#: build/lib/core/models.py:1066 core/models.py:1066
+#: build/lib/core/models.py:1049 core/models.py:1049
 msgid "Document invitations"
 msgstr ""
 
-#: build/lib/core/models.py:1086 core/models.py:1086
+#: build/lib/core/models.py:1069 core/models.py:1069
 msgid "This email is already associated to a registered user."
 msgstr ""
 
-#: build/lib/impress/settings.py:235 impress/settings.py:235
+#: build/lib/impress/settings.py:236 impress/settings.py:236
 msgid "English"
 msgstr ""
 
-#: build/lib/impress/settings.py:236 impress/settings.py:236
+#: build/lib/impress/settings.py:237 impress/settings.py:237
 msgid "French"
 msgstr ""
 
-#: build/lib/impress/settings.py:237 impress/settings.py:237
+#: build/lib/impress/settings.py:238 impress/settings.py:238
 msgid "German"
 msgstr ""
 

src/backend/locale/fr_FR/LC_MESSAGES/django.po

@@ -2,8 +2,8 @@ msgid ""
 msgstr ""
 "Project-Id-Version: lasuite-docs\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2025-03-03 12:20+0000\n"
-"PO-Revision-Date: 2025-03-04 08:20\n"
+"POT-Creation-Date: 2025-02-06 15:30+0000\n"
+"PO-Revision-Date: 2025-02-10 14:14\n"
 "Last-Translator: \n"
 "Language-Team: French\n"
 "Language: fr_FR\n"
@@ -54,15 +54,15 @@ msgstr "Un nouveau document a été créé pour vous !"
 msgid "You have been granted ownership of a new document:"
 msgstr "Vous avez été déclaré propriétaire d'un nouveau document :"
 
-#: build/lib/core/api/serializers.py:455 core/api/serializers.py:455
+#: build/lib/core/api/serializers.py:453 core/api/serializers.py:453
 msgid "Body"
 msgstr ""
 
-#: build/lib/core/api/serializers.py:458 core/api/serializers.py:458
+#: build/lib/core/api/serializers.py:456 core/api/serializers.py:456
 msgid "Body type"
 msgstr ""
 
-#: build/lib/core/api/serializers.py:464 core/api/serializers.py:464
+#: build/lib/core/api/serializers.py:462 core/api/serializers.py:462
 msgid "Format"
 msgstr ""
 
@@ -230,8 +230,8 @@ msgstr ""
 msgid "users"
 msgstr ""
 
-#: build/lib/core/models.py:373 build/lib/core/models.py:942 core/models.py:373
-#: core/models.py:942
+#: build/lib/core/models.py:373 build/lib/core/models.py:925 core/models.py:373
+#: core/models.py:925
 msgid "title"
 msgstr ""
 
@@ -249,145 +249,145 @@ msgstr ""
 
 #: build/lib/core/models.py:418 core/models.py:418
 msgid "Untitled Document"
-msgstr "Document sans titre"
+msgstr ""
 
-#: build/lib/core/models.py:734 core/models.py:734
+#: build/lib/core/models.py:719 core/models.py:719
 #, python-brace-format
 msgid "{name} shared a document with you!"
 msgstr "{name} a partagé un document avec vous!"
 
-#: build/lib/core/models.py:738 core/models.py:738
+#: build/lib/core/models.py:723 core/models.py:723
 #, python-brace-format
 msgid "{name} invited you with the role \"{role}\" on the following document:"
 msgstr "{name} vous a invité avec le rôle \"{role}\" sur le document suivant:"
 
-#: build/lib/core/models.py:741 core/models.py:741
+#: build/lib/core/models.py:726 core/models.py:726
 #, python-brace-format
 msgid "{name} shared a document with you: {title}"
 msgstr "{name} a partagé un document avec vous: {title}"
 
-#: build/lib/core/models.py:777 core/models.py:777
+#: build/lib/core/models.py:762 core/models.py:762
 msgid "This document is not deleted."
 msgstr ""
 
-#: build/lib/core/models.py:784 core/models.py:784
+#: build/lib/core/models.py:769 core/models.py:769
 msgid "This document was permanently deleted and cannot be restored."
 msgstr ""
 
-#: build/lib/core/models.py:837 core/models.py:837
+#: build/lib/core/models.py:820 core/models.py:820
 msgid "Document/user link trace"
 msgstr ""
 
-#: build/lib/core/models.py:838 core/models.py:838
+#: build/lib/core/models.py:821 core/models.py:821
 msgid "Document/user link traces"
 msgstr ""
 
-#: build/lib/core/models.py:844 core/models.py:844
+#: build/lib/core/models.py:827 core/models.py:827
 msgid "A link trace already exists for this document/user."
 msgstr ""
 
-#: build/lib/core/models.py:867 core/models.py:867
+#: build/lib/core/models.py:850 core/models.py:850
 msgid "Document favorite"
 msgstr ""
 
-#: build/lib/core/models.py:868 core/models.py:868
+#: build/lib/core/models.py:851 core/models.py:851
 msgid "Document favorites"
 msgstr ""
 
-#: build/lib/core/models.py:874 core/models.py:874
+#: build/lib/core/models.py:857 core/models.py:857
 msgid "This document is already targeted by a favorite relation instance for the same user."
 msgstr ""
 
-#: build/lib/core/models.py:896 core/models.py:896
+#: build/lib/core/models.py:879 core/models.py:879
 msgid "Document/user relation"
 msgstr ""
 
-#: build/lib/core/models.py:897 core/models.py:897
+#: build/lib/core/models.py:880 core/models.py:880
 msgid "Document/user relations"
 msgstr ""
 
-#: build/lib/core/models.py:903 core/models.py:903
+#: build/lib/core/models.py:886 core/models.py:886
 msgid "This user is already in this document."
 msgstr ""
 
-#: build/lib/core/models.py:909 core/models.py:909
+#: build/lib/core/models.py:892 core/models.py:892
 msgid "This team is already in this document."
 msgstr ""
 
-#: build/lib/core/models.py:915 build/lib/core/models.py:1029
-#: core/models.py:915 core/models.py:1029
+#: build/lib/core/models.py:898 build/lib/core/models.py:1012
+#: core/models.py:898 core/models.py:1012
 msgid "Either user or team must be set, not both."
 msgstr ""
 
-#: build/lib/core/models.py:943 core/models.py:943
+#: build/lib/core/models.py:926 core/models.py:926
 msgid "description"
 msgstr ""
 
-#: build/lib/core/models.py:944 core/models.py:944
+#: build/lib/core/models.py:927 core/models.py:927
 msgid "code"
 msgstr ""
 
-#: build/lib/core/models.py:945 core/models.py:945
+#: build/lib/core/models.py:928 core/models.py:928
 msgid "css"
 msgstr ""
 
-#: build/lib/core/models.py:947 core/models.py:947
+#: build/lib/core/models.py:930 core/models.py:930
 msgid "public"
 msgstr ""
 
-#: build/lib/core/models.py:949 core/models.py:949
+#: build/lib/core/models.py:932 core/models.py:932
 msgid "Whether this template is public for anyone to use."
 msgstr ""
 
-#: build/lib/core/models.py:955 core/models.py:955
+#: build/lib/core/models.py:938 core/models.py:938
 msgid "Template"
 msgstr ""
 
-#: build/lib/core/models.py:956 core/models.py:956
+#: build/lib/core/models.py:939 core/models.py:939
 msgid "Templates"
 msgstr ""
 
-#: build/lib/core/models.py:1010 core/models.py:1010
+#: build/lib/core/models.py:993 core/models.py:993
 msgid "Template/user relation"
 msgstr ""
 
-#: build/lib/core/models.py:1011 core/models.py:1011
+#: build/lib/core/models.py:994 core/models.py:994
 msgid "Template/user relations"
 msgstr ""
 
-#: build/lib/core/models.py:1017 core/models.py:1017
+#: build/lib/core/models.py:1000 core/models.py:1000
 msgid "This user is already in this template."
 msgstr ""
 
-#: build/lib/core/models.py:1023 core/models.py:1023
+#: build/lib/core/models.py:1006 core/models.py:1006
 msgid "This team is already in this template."
 msgstr ""
 
-#: build/lib/core/models.py:1046 core/models.py:1046
+#: build/lib/core/models.py:1029 core/models.py:1029
 msgid "email address"
 msgstr ""
 
-#: build/lib/core/models.py:1065 core/models.py:1065
+#: build/lib/core/models.py:1048 core/models.py:1048
 msgid "Document invitation"
 msgstr ""
 
-#: build/lib/core/models.py:1066 core/models.py:1066
+#: build/lib/core/models.py:1049 core/models.py:1049
 msgid "Document invitations"
 msgstr ""
 
-#: build/lib/core/models.py:1086 core/models.py:1086
+#: build/lib/core/models.py:1069 core/models.py:1069
 msgid "This email is already associated to a registered user."
 msgstr ""
 
-#: build/lib/impress/settings.py:235 impress/settings.py:235
+#: build/lib/impress/settings.py:236 impress/settings.py:236
 msgid "English"
 msgstr ""
 
-#: build/lib/impress/settings.py:236 impress/settings.py:236
+#: build/lib/impress/settings.py:237 impress/settings.py:237
 msgid "French"
 msgstr ""
 
-#: build/lib/impress/settings.py:237 impress/settings.py:237
+#: build/lib/impress/settings.py:238 impress/settings.py:238
 msgid "German"
 msgstr ""
 

src/backend/locale/nl_NL/LC_MESSAGES/django.po

@@ -2,8 +2,8 @@ msgid ""
 msgstr ""
 "Project-Id-Version: lasuite-docs\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2025-03-03 12:20+0000\n"
-"PO-Revision-Date: 2025-03-03 12:22\n"
+"POT-Creation-Date: 2025-02-06 15:30+0000\n"
+"PO-Revision-Date: 2025-02-10 14:14\n"
 "Last-Translator: \n"
 "Language-Team: Dutch\n"
 "Language: nl_NL\n"
@@ -54,15 +54,15 @@ msgstr ""
 msgid "You have been granted ownership of a new document:"
 msgstr ""
 
-#: build/lib/core/api/serializers.py:455 core/api/serializers.py:455
+#: build/lib/core/api/serializers.py:453 core/api/serializers.py:453
 msgid "Body"
 msgstr ""
 
-#: build/lib/core/api/serializers.py:458 core/api/serializers.py:458
+#: build/lib/core/api/serializers.py:456 core/api/serializers.py:456
 msgid "Body type"
 msgstr ""
 
-#: build/lib/core/api/serializers.py:464 core/api/serializers.py:464
+#: build/lib/core/api/serializers.py:462 core/api/serializers.py:462
 msgid "Format"
 msgstr ""
 
@@ -230,8 +230,8 @@ msgstr ""
 msgid "users"
 msgstr ""
 
-#: build/lib/core/models.py:373 build/lib/core/models.py:942 core/models.py:373
-#: core/models.py:942
+#: build/lib/core/models.py:373 build/lib/core/models.py:925 core/models.py:373
+#: core/models.py:925
 msgid "title"
 msgstr ""
 
@@ -251,143 +251,143 @@ msgstr ""
 msgid "Untitled Document"
 msgstr ""
 
-#: build/lib/core/models.py:734 core/models.py:734
+#: build/lib/core/models.py:719 core/models.py:719
 #, python-brace-format
 msgid "{name} shared a document with you!"
 msgstr ""
 
-#: build/lib/core/models.py:738 core/models.py:738
+#: build/lib/core/models.py:723 core/models.py:723
 #, python-brace-format
 msgid "{name} invited you with the role \"{role}\" on the following document:"
 msgstr ""
 
-#: build/lib/core/models.py:741 core/models.py:741
+#: build/lib/core/models.py:726 core/models.py:726
 #, python-brace-format
 msgid "{name} shared a document with you: {title}"
 msgstr ""
 
-#: build/lib/core/models.py:777 core/models.py:777
+#: build/lib/core/models.py:762 core/models.py:762
 msgid "This document is not deleted."
 msgstr ""
 
-#: build/lib/core/models.py:784 core/models.py:784
+#: build/lib/core/models.py:769 core/models.py:769
 msgid "This document was permanently deleted and cannot be restored."
 msgstr ""
 
-#: build/lib/core/models.py:837 core/models.py:837
+#: build/lib/core/models.py:820 core/models.py:820
 msgid "Document/user link trace"
 msgstr ""
 
-#: build/lib/core/models.py:838 core/models.py:838
+#: build/lib/core/models.py:821 core/models.py:821
 msgid "Document/user link traces"
 msgstr ""
 
-#: build/lib/core/models.py:844 core/models.py:844
+#: build/lib/core/models.py:827 core/models.py:827
 msgid "A link trace already exists for this document/user."
 msgstr ""
 
-#: build/lib/core/models.py:867 core/models.py:867
+#: build/lib/core/models.py:850 core/models.py:850
 msgid "Document favorite"
 msgstr ""
 
-#: build/lib/core/models.py:868 core/models.py:868
+#: build/lib/core/models.py:851 core/models.py:851
 msgid "Document favorites"
 msgstr ""
 
-#: build/lib/core/models.py:874 core/models.py:874
+#: build/lib/core/models.py:857 core/models.py:857
 msgid "This document is already targeted by a favorite relation instance for the same user."
 msgstr ""
 
-#: build/lib/core/models.py:896 core/models.py:896
+#: build/lib/core/models.py:879 core/models.py:879
 msgid "Document/user relation"
 msgstr ""
 
-#: build/lib/core/models.py:897 core/models.py:897
+#: build/lib/core/models.py:880 core/models.py:880
 msgid "Document/user relations"
 msgstr ""
 
-#: build/lib/core/models.py:903 core/models.py:903
+#: build/lib/core/models.py:886 core/models.py:886
 msgid "This user is already in this document."
 msgstr ""
 
-#: build/lib/core/models.py:909 core/models.py:909
+#: build/lib/core/models.py:892 core/models.py:892
 msgid "This team is already in this document."
 msgstr ""
 
-#: build/lib/core/models.py:915 build/lib/core/models.py:1029
-#: core/models.py:915 core/models.py:1029
+#: build/lib/core/models.py:898 build/lib/core/models.py:1012
+#: core/models.py:898 core/models.py:1012
 msgid "Either user or team must be set, not both."
 msgstr ""
 
-#: build/lib/core/models.py:943 core/models.py:943
+#: build/lib/core/models.py:926 core/models.py:926
 msgid "description"
 msgstr ""
 
-#: build/lib/core/models.py:944 core/models.py:944
+#: build/lib/core/models.py:927 core/models.py:927
 msgid "code"
 msgstr ""
 
-#: build/lib/core/models.py:945 core/models.py:945
+#: build/lib/core/models.py:928 core/models.py:928
 msgid "css"
 msgstr ""
 
-#: build/lib/core/models.py:947 core/models.py:947
+#: build/lib/core/models.py:930 core/models.py:930
 msgid "public"
 msgstr ""
 
-#: build/lib/core/models.py:949 core/models.py:949
+#: build/lib/core/models.py:932 core/models.py:932
 msgid "Whether this template is public for anyone to use."
 msgstr ""
 
-#: build/lib/core/models.py:955 core/models.py:955
+#: build/lib/core/models.py:938 core/models.py:938
 msgid "Template"
 msgstr ""
 
-#: build/lib/core/models.py:956 core/models.py:956
+#: build/lib/core/models.py:939 core/models.py:939
 msgid "Templates"
 msgstr ""
 
-#: build/lib/core/models.py:1010 core/models.py:1010
+#: build/lib/core/models.py:993 core/models.py:993
 msgid "Template/user relation"
 msgstr ""
 
-#: build/lib/core/models.py:1011 core/models.py:1011
+#: build/lib/core/models.py:994 core/models.py:994
 msgid "Template/user relations"
 msgstr ""
 
-#: build/lib/core/models.py:1017 core/models.py:1017
+#: build/lib/core/models.py:1000 core/models.py:1000
 msgid "This user is already in this template."
 msgstr ""
 
-#: build/lib/core/models.py:1023 core/models.py:1023
+#: build/lib/core/models.py:1006 core/models.py:1006
 msgid "This team is already in this template."
 msgstr ""
 
-#: build/lib/core/models.py:1046 core/models.py:1046
+#: build/lib/core/models.py:1029 core/models.py:1029
 msgid "email address"
 msgstr ""
 
-#: build/lib/core/models.py:1065 core/models.py:1065
+#: build/lib/core/models.py:1048 core/models.py:1048
 msgid "Document invitation"
 msgstr ""
 
-#: build/lib/core/models.py:1066 core/models.py:1066
+#: build/lib/core/models.py:1049 core/models.py:1049
 msgid "Document invitations"
 msgstr ""
 
-#: build/lib/core/models.py:1086 core/models.py:1086
+#: build/lib/core/models.py:1069 core/models.py:1069
 msgid "This email is already associated to a registered user."
 msgstr ""
 
-#: build/lib/impress/settings.py:235 impress/settings.py:235
+#: build/lib/impress/settings.py:236 impress/settings.py:236
 msgid "English"
 msgstr ""
 
-#: build/lib/impress/settings.py:236 impress/settings.py:236
+#: build/lib/impress/settings.py:237 impress/settings.py:237
 msgid "French"
 msgstr ""
 
-#: build/lib/impress/settings.py:237 impress/settings.py:237
+#: build/lib/impress/settings.py:238 impress/settings.py:238
 msgid "German"
 msgstr ""
 

src/backend/pyproject.toml

@@ -7,7 +7,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "impress"
-version = "2.3.0"
+version = "2.2.0"
 authors = [{ "name" = "DINUM", "email" = "dev@mail.numerique.gouv.fr" }]
 classifiers = [
     "Development Status :: 5 - Production/Stable",
@@ -68,7 +68,6 @@ dependencies = [
 [project.optional-dependencies]
 dev = [
     "django-extensions==3.2.3",
-    "django-test-migrations==1.4.0",
     "drf-spectacular-sidecar==2024.12.1",
     "freezegun==1.5.1",
     "ipdb==0.13.13",
@@ -100,6 +99,7 @@ exclude = [
     "build",
     "venv",
     "__pycache__",
+    "*/migrations/*",
 ]
 line-length = 88
 

src/frontend/apps/e2e/__tests__/app-impress/404.spec.ts

@@ -17,13 +17,13 @@ test.describe('404', () => {
         'It seems that the page you are looking for does not exist or cannot be displayed correctly.',
       ),
     ).toBeVisible();
-    await expect(page.getByText('Home')).toBeVisible();
+    await expect(page.getByText('Back to home page')).toBeVisible();
   });
 
   test('checks go back to home page redirects to home page', async ({
     page,
   }) => {
-    await page.getByText('Home').click();
+    await page.getByText('Back to home page').click();
     await expect(page).toHaveURL('/');
   });
 });

src/frontend/apps/e2e/__tests__/app-impress/assets/test.html

@@ -1,22 +0,0 @@
-<html>
-  <head>
-    <title>Test unsafe file</title>
-  </head>
-  <body>
-    <h1>Hello svg</h1>
-    <img src="test.jpg" alt="test" />
-    <svg
-      xmlns="http://www.w3.org/2000/svg"
-      width="100"
-      height="100"
-      viewBox="0 0 100 100"
-    >
-      <circle cx="50" cy="30" r="20" fill="#3498db" />
-      <polygon
-        points="50,10 55,20 65,20 58,30 60,40 50,35 40,40 42,30 35,20 45,20"
-        fill="#f1c40f"
-      />
-      <text x="50" y="70" text-anchor="middle" fill="white">Hello svg</text>
-    </svg>
-  </body>
-</html>

src/frontend/apps/e2e/__tests__/app-impress/auth.setup.ts

@@ -1,46 +1,27 @@
-import { FullConfig, FullProject, chromium, expect } from '@playwright/test';
+import { test as setup } from '@playwright/test';
 
 import { keyCloakSignIn } from './common';
 
-const saveStorageState = async (
-  browserConfig: FullProject<unknown, unknown>,
-) => {
-  const browserName = browserConfig?.name || 'chromium';
+setup('authenticate-chromium', async ({ page }) => {
+  await page.goto('/');
+  await keyCloakSignIn(page, 'chromium');
+  await page
+    .context()
+    .storageState({ path: `playwright/.auth/user-chromium.json` });
+});
 
-  const { storageState, ...useConfig } = browserConfig?.use;
-  const browser = await chromium.launch();
-  const context = await browser.newContext(useConfig);
-  const page = await context.newPage();
+setup('authenticate-webkit', async ({ page }) => {
+  await page.goto('/');
+  await keyCloakSignIn(page, 'webkit');
+  await page
+    .context()
+    .storageState({ path: `playwright/.auth/user-webkit.json` });
+});
 
-  await page.goto('/', { waitUntil: 'networkidle' });
-  await page.content();
-  await expect(page.getByText('Docs').first()).toBeVisible();
-
-  await keyCloakSignIn(page, browserName);
-
-  await expect(
-    page.locator('header').first().getByRole('button', {
-      name: 'Logout',
-    }),
-  ).toBeVisible();
-
-  await page.context().storageState({
-    path: storageState as string,
-  });
-
-  await browser.close();
-};
-
-async function globalSetup(config: FullConfig) {
-  /* eslint-disable @typescript-eslint/no-non-null-assertion */
-  const chromeConfig = config.projects.find((p) => p.name === 'chromium')!;
-  const firefoxConfig = config.projects.find((p) => p.name === 'firefox')!;
-  const webkitConfig = config.projects.find((p) => p.name === 'webkit')!;
-  /* eslint-enable @typescript-eslint/no-non-null-assertion */
-
-  await saveStorageState(chromeConfig);
-  await saveStorageState(webkitConfig);
-  await saveStorageState(firefoxConfig);
-}
-
-export default globalSetup;
+setup('authenticate-firefox', async ({ page }) => {
+  await page.goto('/');
+  await keyCloakSignIn(page, 'firefox');
+  await page
+    .context()
+    .storageState({ path: `playwright/.auth/user-firefox.json` });
+});

src/frontend/apps/e2e/__tests__/app-impress/common.ts

@@ -97,7 +97,7 @@ export const addNewMember = async (
 
   // Choose a role
   await page.getByLabel('doc-role-dropdown').click();
-  await page.getByRole('menuitem', { name: role }).click();
+  await page.getByRole('button', { name: role }).click();
   await page.getByRole('button', { name: 'Invite' }).click();
 
   return users[index].email;

src/frontend/apps/e2e/__tests__/app-impress/doc-editor.spec.ts

@@ -1,7 +1,8 @@
+/* eslint-disable playwright/no-conditional-expect */
+/* eslint-disable playwright/no-conditional-in-test */
 import path from 'path';
 
 import { expect, test } from '@playwright/test';
-import cs from 'convert-stream';
 
 import {
   createDoc,
@@ -36,10 +37,10 @@ test.describe('Doc Editor', () => {
 
     // Change language to French
     await header.click();
-    await header.getByRole('button', { name: /Language/ }).click();
-    await page.getByRole('menuitem', { name: 'Français' }).click();
+    await header.getByRole('combobox').getByText('English').click();
+    await header.getByRole('option', { name: 'Français' }).click();
     await expect(
-      header.getByRole('button').getByText('Français'),
+      header.getByRole('combobox').getByText('Français'),
     ).toBeVisible();
 
     // Trigger slash menu to show french menu
@@ -129,7 +130,7 @@ test.describe('Doc Editor', () => {
 
     await selectVisibility.click();
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Connected',
       })
       .click();
@@ -414,8 +415,6 @@ test.describe('Doc Editor', () => {
       const editor = page.locator('.ProseMirror');
       await editor.getByText('Hello').dblclick();
 
-      /* eslint-disable playwright/no-conditional-expect */
-      /* eslint-disable playwright/no-conditional-in-test */
       if (!ai_transform && !ai_translate) {
         await expect(page.getByRole('button', { name: 'AI' })).toBeHidden();
         return;
@@ -442,45 +441,6 @@ test.describe('Doc Editor', () => {
           page.getByRole('menuitem', { name: 'Language' }),
         ).toBeHidden();
       }
-      /* eslint-enable playwright/no-conditional-expect */
-      /* eslint-enable playwright/no-conditional-in-test */
     });
   });
-
-  test('it downloads unsafe files', async ({ page, browserName }) => {
-    const [randomDoc] = await createDoc(page, 'doc-editor', browserName, 1);
-
-    const fileChooserPromise = page.waitForEvent('filechooser');
-    const downloadPromise = page.waitForEvent('download', (download) => {
-      return download.suggestedFilename().includes(`html`);
-    });
-
-    await verifyDocName(page, randomDoc);
-
-    await page.locator('.ProseMirror.bn-editor').click();
-    await page.locator('.ProseMirror.bn-editor').fill('Hello World');
-
-    await page.keyboard.press('Enter');
-    await page.locator('.bn-block-outer').last().fill('/');
-    await page.getByText('Embedded file').click();
-    await page.getByText('Upload file').click();
-
-    const fileChooser = await fileChooserPromise;
-    await fileChooser.setFiles(path.join(__dirname, 'assets/test.html'));
-
-    await page.locator('.bn-block-content[data-name="test.html"]').click();
-    await page.getByRole('button', { name: 'Download file' }).click();
-
-    await expect(
-      page.getByText('This file is flagged as unsafe.'),
-    ).toBeVisible();
-
-    await page.getByRole('button', { name: 'Download' }).click();
-
-    const download = await downloadPromise;
-    expect(download.suggestedFilename()).toContain(`-unsafe.html`);
-
-    const svgBuffer = await cs.toBuffer(await download.createReadStream());
-    expect(svgBuffer.toString()).toContain('Hello svg');
-  });
 });

src/frontend/apps/e2e/__tests__/app-impress/doc-export.spec.ts

@@ -197,49 +197,4 @@ test.describe('Doc Export', () => {
 
     expect(pdfText).toContain('Hello World');
   });
-
-  test('it exports the doc with quotes', async ({ page, browserName }) => {
-    const [randomDoc] = await createDoc(page, 'export-quotes', browserName, 1);
-
-    const downloadPromise = page.waitForEvent('download', (download) => {
-      return download.suggestedFilename().includes(`${randomDoc}.pdf`);
-    });
-
-    const editor = page.locator('.ProseMirror');
-    // Trigger slash menu to show menu
-    await editor.click();
-    await editor.fill('/');
-    await page.getByText('Add a quote block').click();
-
-    await expect(
-      editor.locator('.bn-block-content[data-content-type="quote"]'),
-    ).toBeVisible();
-
-    await editor.fill('Hello World');
-
-    await expect(editor.getByText('Hello World')).toHaveCSS(
-      'font-style',
-      'italic',
-    );
-
-    await page
-      .getByRole('button', {
-        name: 'download',
-      })
-      .click();
-
-    await page
-      .getByRole('button', {
-        name: 'Download',
-      })
-      .click();
-
-    const download = await downloadPromise;
-    expect(download.suggestedFilename()).toBe(`${randomDoc}.pdf`);
-
-    const pdfBuffer = await cs.toBuffer(await download.createReadStream());
-    const pdfData = await pdf(pdfBuffer);
-
-    expect(pdfData.text).toContain('Hello World'); // This is the pdf text
-  });
 });

src/frontend/apps/e2e/__tests__/app-impress/doc-grid.spec.ts

@@ -7,9 +7,17 @@ type SmallDoc = {
   title: string;
 };
 
+test.beforeEach(async ({ page }) => {
+  await page.goto('/');
+});
+
 test.describe('Documents Grid mobile', () => {
   test.use({ viewport: { width: 500, height: 1200 } });
 
+  test.beforeEach(async ({ page }) => {
+    await page.goto('/');
+  });
+
   test('it checks the grid when mobile', async ({ page }) => {
     await page.route('**/documents/**', async (route) => {
       const request = route.request();
@@ -86,10 +94,6 @@ test.describe('Documents Grid mobile', () => {
 });
 
 test.describe('Document grid item options', () => {
-  test.beforeEach(async ({ page }) => {
-    await page.goto('/');
-  });
-
   test('it pins a document', async ({ page, browserName }) => {
     const [docTitle] = await createDoc(page, `Favorite doc`, browserName);
 
@@ -208,8 +212,6 @@ test.describe('Document grid item options', () => {
 
 test.describe('Documents filters', () => {
   test('it checks the prebuild left panel filters', async ({ page }) => {
-    await page.goto('/');
-
     // All Docs
     const response = await page.waitForResponse(
       (response) =>
@@ -280,9 +282,11 @@ test.describe('Documents filters', () => {
 });
 
 test.describe('Documents Grid', () => {
-  test('checks all the elements are visible', async ({ page }) => {
+  test.beforeEach(async ({ page }) => {
     await page.goto('/');
+  });
 
+  test('checks all the elements are visible', async ({ page }) => {
     let docs: SmallDoc[] = [];
     const response = await page.waitForResponse(
       (response) =>
@@ -310,12 +314,11 @@ test.describe('Documents Grid', () => {
 
   test('checks the infinite scroll', async ({ page }) => {
     let docs: SmallDoc[] = [];
-    const responsePromisePage1 = page.waitForResponse((response) => {
-      return (
+    const responsePromisePage1 = page.waitForResponse(
+      (response) =>
         response.url().endsWith(`/documents/?page=1`) &&
-        response.status() === 200
-      );
-    });
+        response.status() === 200,
+    );
 
     const responsePromisePage2 = page.waitForResponse(
       (response) =>
@@ -323,8 +326,6 @@ test.describe('Documents Grid', () => {
         response.status() === 200,
     );
 
-    await page.goto('/');
-
     const responsePage1 = await responsePromisePage1;
     expect(responsePage1.ok()).toBeTruthy();
     let result = await responsePage1.json();

src/frontend/apps/e2e/__tests__/app-impress/doc-header.spec.ts

@@ -89,7 +89,7 @@ test.describe('Doc Header', () => {
 
     await page.getByLabel('Open the document options').click();
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Delete document',
       })
       .click();
@@ -153,7 +153,7 @@ test.describe('Doc Header', () => {
     await page.getByLabel('Open the document options').click();
 
     await expect(
-      page.getByRole('menuitem', { name: 'Delete document' }),
+      page.getByRole('button', { name: 'Delete document' }),
     ).toBeDisabled();
 
     // Click somewhere else to close the options
@@ -177,7 +177,7 @@ test.describe('Doc Header', () => {
     await invitationCard.getByRole('button', { name: 'more_horiz' }).click();
 
     await expect(
-      page.getByRole('menuitem', {
+      page.getByRole('button', {
         name: 'delete',
       }),
     ).toBeEnabled();
@@ -195,7 +195,7 @@ test.describe('Doc Header', () => {
     await memberCard.getByRole('button', { name: 'more_horiz' }).click();
 
     await expect(
-      page.getByRole('menuitem', {
+      page.getByRole('button', {
         name: 'delete',
       }),
     ).toBeEnabled();
@@ -233,7 +233,7 @@ test.describe('Doc Header', () => {
     await page.getByLabel('Open the document options').click();
 
     await expect(
-      page.getByRole('menuitem', { name: 'Delete document' }),
+      page.getByRole('button', { name: 'Delete document' }),
     ).toBeDisabled();
 
     // Click somewhere else to close the options
@@ -295,7 +295,7 @@ test.describe('Doc Header', () => {
     await page.getByLabel('Open the document options').click();
 
     await expect(
-      page.getByRole('menuitem', { name: 'Delete document' }),
+      page.getByRole('button', { name: 'Delete document' }),
     ).toBeDisabled();
 
     // Click somewhere else to close the options
@@ -352,7 +352,7 @@ test.describe('Doc Header', () => {
 
     // Copy content to clipboard
     await page.getByLabel('Open the document options').click();
-    await page.getByRole('menuitem', { name: 'Copy as Markdown' }).click();
+    await page.getByRole('button', { name: 'Copy as Markdown' }).click();
     await expect(page.getByText('Copied to clipboard')).toBeVisible();
 
     // Test that clipboard is in Markdown format
@@ -387,7 +387,7 @@ test.describe('Doc Header', () => {
 
     // Copy content to clipboard
     await page.getByLabel('Open the document options').click();
-    await page.getByRole('menuitem', { name: 'Copy as HTML' }).click();
+    await page.getByRole('button', { name: 'Copy as HTML' }).click();
     await expect(page.getByText('Copied to clipboard')).toBeVisible();
 
     // Test that clipboard is in HTML format
@@ -460,7 +460,7 @@ test.describe('Documents Header mobile', () => {
 
     await expect(page.getByRole('button', { name: 'Copy link' })).toBeHidden();
     await page.getByLabel('Open the document options').click();
-    await page.getByRole('menuitem', { name: 'Share' }).click();
+    await page.getByRole('button', { name: 'Share' }).click();
     await page.getByRole('button', { name: 'Copy link' }).click();
     await expect(page.getByText('Link Copied !')).toBeVisible();
     // Test that clipboard is in HTML format
@@ -494,7 +494,7 @@ test.describe('Documents Header mobile', () => {
     await goToGridDoc(page);
 
     await page.getByLabel('Open the document options').click();
-    await page.getByRole('menuitem', { name: 'Share' }).click();
+    await page.getByRole('button', { name: 'Share' }).click();
 
     await expect(page.getByLabel('Share modal')).toBeVisible();
     await page.getByRole('button', { name: 'close' }).click();

src/frontend/apps/e2e/__tests__/app-impress/doc-member-create.spec.ts

@@ -65,15 +65,15 @@ test.describe('Document create member', () => {
 
     // Check roles are displayed
     await list.getByLabel('doc-role-dropdown').click();
-    await expect(page.getByRole('menuitem', { name: 'Reader' })).toBeVisible();
-    await expect(page.getByRole('menuitem', { name: 'Editor' })).toBeVisible();
-    await expect(page.getByRole('menuitem', { name: 'Owner' })).toBeVisible();
+    await expect(page.getByRole('button', { name: 'Reader' })).toBeVisible();
+    await expect(page.getByRole('button', { name: 'Editor' })).toBeVisible();
+    await expect(page.getByRole('button', { name: 'Owner' })).toBeVisible();
     await expect(
-      page.getByRole('menuitem', { name: 'Administrator' }),
+      page.getByRole('button', { name: 'Administrator' }),
     ).toBeVisible();
 
     // Validate
-    await page.getByRole('menuitem', { name: 'Administrator' }).click();
+    await page.getByRole('button', { name: 'Administrator' }).click();
     await page.getByRole('button', { name: 'Invite' }).click();
 
     // Check invitation added
@@ -121,7 +121,7 @@ test.describe('Document create member', () => {
     // Choose a role
     const container = page.getByTestId('doc-share-add-member-list');
     await container.getByLabel('doc-role-dropdown').click();
-    await page.getByRole('menuitem', { name: 'Owner' }).click();
+    await page.getByRole('button', { name: 'Owner' }).click();
 
     const responsePromiseCreateInvitation = page.waitForResponse(
       (response) =>
@@ -139,7 +139,7 @@ test.describe('Document create member', () => {
 
     // Choose a role
     await container.getByLabel('doc-role-dropdown').click();
-    await page.getByRole('menuitem', { name: 'Owner' }).click();
+    await page.getByRole('button', { name: 'Owner' }).click();
 
     const responsePromiseCreateInvitationFail = page.waitForResponse(
       (response) =>
@@ -162,8 +162,8 @@ test.describe('Document create member', () => {
     await createDoc(page, 'user-invitation', browserName, 1);
 
     const header = page.locator('header').first();
-    await header.getByRole('button', { name: /Language/ }).click();
-    await page.getByRole('menuitem', { name: 'Français' }).click();
+    await header.getByRole('combobox').getByText('EN').click();
+    await header.getByRole('option', { name: 'translate Français' }).click();
 
     await page.getByRole('button', { name: 'Partager' }).click();
 
@@ -178,7 +178,7 @@ test.describe('Document create member', () => {
     // Choose a role
     const container = page.getByTestId('doc-share-add-member-list');
     await container.getByLabel('doc-role-dropdown').click();
-    await page.getByRole('menuitem', { name: 'Administrateur' }).click();
+    await page.getByRole('button', { name: 'Administrateur' }).click();
 
     const responsePromiseCreateInvitation = page.waitForResponse(
       (response) =>
@@ -212,7 +212,7 @@ test.describe('Document create member', () => {
     // Choose a role
     const container = page.getByTestId('doc-share-add-member-list');
     await container.getByLabel('doc-role-dropdown').click();
-    await page.getByRole('menuitem', { name: 'Administrator' }).click();
+    await page.getByRole('button', { name: 'Administrator' }).click();
 
     const responsePromiseCreateInvitation = page.waitForResponse(
       (response) =>
@@ -232,14 +232,14 @@ test.describe('Document create member', () => {
     await expect(userInvitation).toBeVisible();
 
     await userInvitation.getByLabel('doc-role-dropdown').click();
-    await page.getByRole('menuitem', { name: 'Reader' }).click();
+    await page.getByRole('button', { name: 'Reader' }).click();
 
     const moreActions = userInvitation.getByRole('button', {
       name: 'more_horiz',
     });
     await moreActions.click();
 
-    await page.getByRole('menuitem', { name: 'Delete' }).click();
+    await page.getByRole('button', { name: 'Delete' }).click();
 
     await expect(userInvitation).toBeHidden();
   });

src/frontend/apps/e2e/__tests__/app-impress/doc-member-list.spec.ts

@@ -161,12 +161,12 @@ test.describe('Document list members', () => {
     await list.click();
 
     await currentUserRole.click();
-    await page.getByRole('menuitem', { name: 'Administrator' }).click();
+    await page.getByRole('button', { name: 'Administrator' }).click();
     await list.click();
     await expect(currentUserRole).toBeVisible();
 
     await currentUserRole.click();
-    await page.getByRole('menuitem', { name: 'Reader' }).click();
+    await page.getByRole('button', { name: 'Reader' }).click();
     await list.click();
     await expect(currentUserRole).toBeHidden();
   });
@@ -215,13 +215,13 @@ test.describe('Document list members', () => {
     await expect(mySelfMoreActions).toBeVisible();
 
     await userReaderMoreActions.click();
-    await page.getByRole('menuitem', { name: 'Delete' }).click();
+    await page.getByRole('button', { name: 'Delete' }).click();
     await expect(userReader).toBeHidden();
 
     await mySelfMoreActions.click();
-    await page.getByRole('menuitem', { name: 'Delete' }).click();
+    await page.getByRole('button', { name: 'Delete' }).click();
     await expect(
-      page.getByText('You do not have permission to view this document.'),
+      page.getByText('You do not have permission to perform this action.'),
     ).toBeVisible();
   });
 });

src/frontend/apps/e2e/__tests__/app-impress/doc-version.spec.ts

@@ -19,7 +19,7 @@ test.describe('Doc Version', () => {
 
     await page.getByLabel('Open the document options').click();
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Version history',
       })
       .click();
@@ -59,7 +59,7 @@ test.describe('Doc Version', () => {
 
     await page.getByLabel('Open the document options').click();
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Version history',
       })
       .click();
@@ -91,7 +91,7 @@ test.describe('Doc Version', () => {
 
     await page.getByLabel('Open the document options').click();
     await expect(
-      page.getByRole('menuitem', { name: 'Version history' }),
+      page.getByRole('button', { name: 'Version history' }),
     ).toBeDisabled();
   });
 
@@ -120,7 +120,7 @@ test.describe('Doc Version', () => {
 
     await page.getByLabel('Open the document options').click();
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Version history',
       })
       .click();

src/frontend/apps/e2e/__tests__/app-impress/doc-visibility.spec.ts

@@ -50,7 +50,7 @@ test.describe('Doc Visibility', () => {
 
     await selectVisibility.click();
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Connected',
       })
       .click();
@@ -60,7 +60,7 @@ test.describe('Doc Visibility', () => {
     await selectVisibility.click();
 
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Public',
       })
       .click();
@@ -100,9 +100,7 @@ test.describe('Doc Visibility: Restricted', () => {
 
     await page.goto(urlDoc);
 
-    await expect(
-      page.getByText('Log in to access the document.'),
-    ).toBeVisible();
+    await expect(page.getByRole('textbox', { name: 'password' })).toBeVisible();
   });
 
   test('A doc is not accessible when authentified but not member.', async ({
@@ -135,7 +133,7 @@ test.describe('Doc Visibility: Restricted', () => {
     await page.goto(urlDoc);
 
     await expect(
-      page.getByText('You do not have permission to view this document.'),
+      page.getByText('You do not have permission to perform this action.'),
     ).toBeVisible();
   });
 
@@ -162,7 +160,7 @@ test.describe('Doc Visibility: Restricted', () => {
     // Choose a role
     const container = page.getByTestId('doc-share-add-member-list');
     await container.getByLabel('doc-role-dropdown').click();
-    await page.getByRole('menuitem', { name: 'Administrator' }).click();
+    await page.getByRole('button', { name: 'Administrator' }).click();
 
     await page.getByRole('button', { name: 'Invite' }).click();
 
@@ -215,7 +213,7 @@ test.describe('Doc Visibility: Public', () => {
     await selectVisibility.click();
 
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Public',
       })
       .click();
@@ -227,7 +225,7 @@ test.describe('Doc Visibility: Public', () => {
     await expect(page.getByLabel('Visibility mode')).toBeVisible();
     await page.getByLabel('Visibility mode').click();
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Reading',
       })
       .click();
@@ -289,7 +287,7 @@ test.describe('Doc Visibility: Public', () => {
     await selectVisibility.click();
 
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Public',
       })
       .click();
@@ -357,7 +355,7 @@ test.describe('Doc Visibility: Authenticated', () => {
     const selectVisibility = page.getByLabel('Visibility', { exact: true });
     await selectVisibility.click();
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Connected',
       })
       .click();
@@ -381,10 +379,7 @@ test.describe('Doc Visibility: Authenticated', () => {
     await page.goto(urlDoc);
 
     await expect(page.locator('h2').getByText(docTitle)).toBeHidden();
-
-    await expect(
-      page.getByText('Log in to access the document.'),
-    ).toBeVisible();
+    await expect(page.getByRole('textbox', { name: 'password' })).toBeVisible();
   });
 
   test('It checks a authenticated doc in read only mode', async ({
@@ -407,7 +402,7 @@ test.describe('Doc Visibility: Authenticated', () => {
     const selectVisibility = page.getByLabel('Visibility', { exact: true });
     await selectVisibility.click();
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Connected',
       })
       .click();
@@ -416,14 +411,6 @@ test.describe('Doc Visibility: Authenticated', () => {
       page.getByText('The document visibility has been updated.'),
     ).toBeVisible();
 
-    await expect(
-      page
-        .getByLabel('It is the card information about the document.')
-        .getByText('Document accessible to any connected person', {
-          exact: true,
-        }),
-    ).toBeVisible();
-
     await page.getByRole('button', { name: 'close' }).click();
 
     const urlDoc = page.url();
@@ -469,7 +456,7 @@ test.describe('Doc Visibility: Authenticated', () => {
     const selectVisibility = page.getByLabel('Visibility', { exact: true });
     await selectVisibility.click();
     await page
-      .getByRole('menuitem', {
+      .getByRole('button', {
         name: 'Connected',
       })
       .click();

src/frontend/apps/e2e/__tests__/app-impress/header.spec.ts

@@ -88,7 +88,6 @@ test.describe('Header mobile', () => {
 test.describe('Header: Log out', () => {
   test.use({ storageState: { cookies: [], origins: [] } });
 
-  // eslint-disable-next-line playwright/expect-expect
   test('checks logout button', async ({ page, browserName }) => {
     await page.goto('/');
     await keyCloakSignIn(page, browserName);

src/frontend/apps/e2e/__tests__/app-impress/home.spec.ts

@@ -12,7 +12,7 @@ test.describe('Home page', () => {
     const footer = page.locator('footer').first();
     await expect(header).toBeVisible();
     await expect(
-      header.getByRole('button', { name: /Language/ }),
+      header.getByRole('combobox', { name: 'Language' }),
     ).toBeVisible();
     await expect(
       header.getByRole('button', { name: 'Les services de La Suite numé' }),
@@ -26,7 +26,6 @@ test.describe('Home page', () => {
 
     // Check the titles
     const h2 = page.locator('h2');
-    await expect(h2.getByText('Govs ❤️ Open Source.')).toBeVisible();
     await expect(
       h2.getByText('Collaborative writing, Simplified.'),
     ).toBeVisible();

src/frontend/apps/e2e/__tests__/app-impress/language.spec.ts

@@ -9,20 +9,19 @@ test.describe('Language', () => {
     await expect(page.getByLabel('Logout')).toBeVisible();
 
     const header = page.locator('header').first();
-    await header
-      .getByRole('button', { name: /Language/ })
-      .getByText('English')
-      .click();
-    await page.getByRole('menuitem', { name: 'Français' }).click();
+    await header.getByRole('combobox').getByText('English').click();
+    await header.getByRole('option', { name: 'Français' }).click();
     await expect(
-      header.getByRole('button').getByText('Français'),
+      header.getByRole('combobox').getByText('Français'),
     ).toBeVisible();
 
     await expect(page.getByLabel('Se déconnecter')).toBeVisible();
 
-    await header.getByRole('button').getByText('Français').click();
-    await page.getByRole('menuitem', { name: 'Deutsch' }).click();
-    await expect(header.getByRole('button').getByText('Deutsch')).toBeVisible();
+    await header.getByRole('combobox').getByText('Français').click();
+    await header.getByRole('option', { name: 'Deutsch' }).click();
+    await expect(
+      header.getByRole('combobox').getByText('Deutsch'),
+    ).toBeVisible();
 
     await expect(page.getByLabel('Abmelden')).toBeVisible();
   });
@@ -54,11 +53,8 @@ test.describe('Language', () => {
 
     // Switch language to French
     const header = page.locator('header').first();
-    await header
-      .getByRole('button', { name: /Language/ })
-      .getByText('English')
-      .click();
-    await page.getByRole('menuitem', { name: 'Français' }).click();
+    await header.getByRole('combobox').getByText('English').click();
+    await header.getByRole('option', { name: 'Français' }).click();
 
     // Check for French 404 response
     await check404Response('Pas trouvé.');

src/frontend/apps/e2e/__tests__/app-impress/left-panel.spec.ts

@@ -29,7 +29,7 @@ test.describe('Left panel mobile', () => {
     const header = page.locator('header').first();
     const homeButton = page.getByRole('button', { name: 'house' });
     const newDocButton = page.getByRole('button', { name: 'New doc' });
-    const languageButton = page.getByRole('button', { name: /Language/ });
+    const languageButton = page.getByRole('combobox', { name: 'Language' });
     const logoutButton = page.getByRole('button', { name: 'Logout' });
 
     await expect(homeButton).not.toBeInViewport();

src/frontend/apps/e2e/package.json

@@ -1,6 +1,6 @@
 {
   "name": "app-e2e",
-  "version": "2.3.0",
+  "version": "2.2.0",
   "private": true,
   "scripts": {
     "lint": "eslint . --ext .ts",

src/frontend/apps/e2e/playwright.config.ts

@@ -38,9 +38,10 @@ export default defineConfig({
     timeout: 120 * 1000,
     reuseExistingServer: true,
   },
-  globalSetup: require.resolve('./__tests__/app-impress/auth.setup'),
+
   /* Configure projects for major browsers */
   projects: [
+    { name: 'setup', testMatch: /.*\.setup\.ts/ },
     {
       name: 'chromium',
       use: {
@@ -52,6 +53,7 @@ export default defineConfig({
           permissions: ['clipboard-read', 'clipboard-write'],
         },
       },
+      dependencies: ['setup'],
     },
     {
       name: 'webkit',
@@ -61,6 +63,7 @@ export default defineConfig({
         timezoneId: 'Europe/Paris',
         storageState: 'playwright/.auth/user-webkit.json',
       },
+      dependencies: ['setup'],
     },
     {
       name: 'firefox',
@@ -76,6 +79,7 @@ export default defineConfig({
           },
         },
       },
+      dependencies: ['setup'],
     },
   ],
 });

src/frontend/apps/impress/cunningham.ts

@@ -357,15 +357,6 @@ const config = {
       components: {
         alert: {
           'border-radius': '0',
-          error: {
-            'background-color': 'var(--c--theme--colors--danger-100)',
-            'border-left-color': 'var(--c--theme--colors--danger-400)',
-            close: {
-              color: 'white',
-              'background-color': 'var(--c--theme--colors--danger-400)',
-              'background-color-hover': 'var(--c--theme--colors--danger-600)',
-            },
-          },
         },
         modal: {
           'width-small': '342px',
@@ -390,7 +381,6 @@ const config = {
               'color-active': 'var(--c--theme--colors--primary-100)',
             },
             'color-hover': 'var(--c--theme--colors--primary-text)',
-            color: 'var(--c--theme--colors--primary-800)',
           },
           secondary: {
             background: {

src/frontend/apps/impress/package.json

@@ -1,6 +1,6 @@
 {
   "name": "app-impress",
-  "version": "2.3.0",
+  "version": "2.2.0",
   "private": true,
   "scripts": {
     "dev": "next dev",
@@ -15,7 +15,6 @@
     "test:watch": "jest --watch"
   },
   "dependencies": {
-    "@ag-media/react-pdf-table": "2.0.1",
     "@blocknote/core": "0.23.2",
     "@blocknote/mantine": "0.23.2",
     "@blocknote/react": "0.23.2",

src/frontend/apps/impress/src/assets/icons/icon-docs.svg

@@ -1,4 +1,10 @@
-<svg viewBox="0 0 32 33" fill="none" xmlns="http://www.w3.org/2000/svg">
+<svg
+  width="32"
+  height="33"
+  viewBox="0 0 32 33"
+  fill="none"
+  xmlns="http://www.w3.org/2000/svg"
+>
   <path
     d="M21.6305 29.5812C22.7983 29.2538 23.9166 28.6562 24.6505 27.6003C25.3749 26.5663 25.5789 25.2547 25.5789 23.9925V5.50099C25.5789 5.17358 25.5611 4.84557 25.5216 4.52148C26.1016 4.74961 26.5486 5.12658 26.8626 5.65239C27.2331 6.25024 27.4184 7.03757 27.4184 8.01435V26.7964C27.4184 28.1184 27.0942 29.1078 26.4458 29.7646C25.7974 30.4214 24.8207 30.7498 23.5155 30.7498H16.4209C16.5889 30.7204 16.7574 30.6901 16.9262 30.659C18.4067 30.3944 19.9713 30.0354 21.6185 29.5846L21.6305 29.5812Z"
     fill="#C9191E"

src/frontend/apps/impress/src/components/DropButton.tsx

@@ -1,3 +1,4 @@
+import { FocusScope } from '@react-aria/focus';
 import {
   PropsWithChildren,
   ReactNode,
@@ -6,22 +7,19 @@ import {
   useState,
 } from 'react';
 import { Button, Popover } from 'react-aria-components';
+import { useTranslation } from 'react-i18next';
 import styled from 'styled-components';
 
-import { BoxProps } from './Box';
-
 const StyledPopover = styled(Popover)`
   background-color: white;
   border-radius: 4px;
   box-shadow: 1px 1px 5px rgba(0, 0, 0, 0.1);
   border: 1px solid #dddddd;
   transition: opacity 0.2s ease-in-out;
+  padding: 1rem;
 `;
 
-interface StyledButtonProps {
-  $css?: BoxProps['$css'];
-}
-const StyledButton = styled(Button)<StyledButtonProps>`
+const StyledButton = styled(Button)`
   cursor: pointer;
   border: none;
   background: none;
@@ -32,12 +30,14 @@ const StyledButton = styled(Button)<StyledButtonProps>`
   font-size: 0.938rem;
   padding: 0;
   text-wrap: nowrap;
-  ${({ $css }) => $css};
+
+  &:focus-within {
+    outline: 2px solid #007bff;
+  }
 `;
 
 export interface DropButtonProps {
   button: ReactNode;
-  buttonCss?: BoxProps['$css'];
   isOpen?: boolean;
   onOpenChange?: (isOpen: boolean) => void;
   label?: string;
@@ -45,19 +45,20 @@ export interface DropButtonProps {
 
 export const DropButton = ({
   button,
-  buttonCss,
   isOpen = false,
   onOpenChange,
   children,
-  label,
 }: PropsWithChildren<DropButtonProps>) => {
+  const { t } = useTranslation();
   const [isLocalOpen, setIsLocalOpen] = useState(isOpen);
-
-  const triggerRef = useRef(null);
+  const triggerRef = useRef<HTMLButtonElement>(null);
+  const firstFocusableRef = useRef<HTMLButtonElement>(null);
 
   useEffect(() => {
-    setIsLocalOpen(isOpen);
-  }, [isOpen]);
+    if (isLocalOpen && firstFocusableRef.current) {
+      firstFocusableRef.current.focus();
+    }
+  }, [isLocalOpen]);
 
   const onOpenChangeHandler = (isOpen: boolean) => {
     setIsLocalOpen(isOpen);
@@ -69,19 +70,30 @@ export const DropButton = ({
       <StyledButton
         ref={triggerRef}
         onPress={() => onOpenChangeHandler(true)}
-        aria-label={label}
-        $css={buttonCss}
+        aria-haspopup="true"
+        aria-expanded={isLocalOpen}
+        aria-label={t('Open the document options')}
       >
-        {button}
+        <span aria-hidden="true">{button}</span>
       </StyledButton>
 
-      <StyledPopover
-        triggerRef={triggerRef}
-        isOpen={isLocalOpen}
-        onOpenChange={onOpenChangeHandler}
-      >
-        {children}
-      </StyledPopover>
+      {isLocalOpen && (
+        <StyledPopover
+          triggerRef={triggerRef}
+          isOpen={isLocalOpen}
+          onOpenChange={onOpenChangeHandler}
+        >
+          <FocusScope contain restoreFocus>
+            {children}
+            <button
+              ref={firstFocusableRef}
+              onClick={() => setIsLocalOpen(false)}
+            >
+              {t('Close the modal')}
+            </button>
+          </FocusScope>
+        </StyledPopover>
+      )}
     </>
   );
 };

src/frontend/apps/impress/src/components/DropdownMenu.tsx

@@ -1,4 +1,5 @@
-import { PropsWithChildren, useRef, useState } from 'react';
+//import { t } from 'i18next';
+import { PropsWithChildren, useState } from 'react';
 import { css } from 'styled-components';
 
 import { Box, BoxButton, BoxProps, DropButton, Icon, Text } from '@/components';
@@ -20,7 +21,6 @@ export type DropdownMenuProps = {
   showArrow?: boolean;
   label?: string;
   arrowCss?: BoxProps['$css'];
-  buttonCss?: BoxProps['$css'];
   disabled?: boolean;
   topMessage?: string;
 };
@@ -31,7 +31,6 @@ export const DropdownMenu = ({
   disabled = false,
   showArrow = false,
   arrowCss,
-  buttonCss,
   label,
   topMessage,
 }: PropsWithChildren<DropdownMenuProps>) => {
@@ -39,7 +38,6 @@ export const DropdownMenu = ({
   const spacings = theme.spacingsTokens();
   const colors = theme.colorsTokens();
   const [isOpen, setIsOpen] = useState(false);
-  const blockButtonRef = useRef<HTMLDivElement>(null);
 
   const onOpenChange = (isOpen: boolean) => {
     setIsOpen(isOpen);
@@ -54,17 +52,10 @@ export const DropdownMenu = ({
       isOpen={isOpen}
       onOpenChange={onOpenChange}
       label={label}
-      buttonCss={buttonCss}
       button={
         showArrow ? (
-          <Box
-            ref={blockButtonRef}
-            $direction="row"
-            $align="center"
-            $position="relative"
-            aria-controls="menu"
-          >
-            <Box>{children}</Box>
+          <Box $direction="row" $align="center">
+            <div>{children}</div>
             <Icon
               $variation="600"
               $css={
@@ -77,17 +68,11 @@ export const DropdownMenu = ({
             />
           </Box>
         ) : (
-          <Box ref={blockButtonRef} aria-controls="menu">
-            {children}
-          </Box>
+          children
         )
       }
     >
-      <Box
-        $maxWidth="320px"
-        $minWidth={`${blockButtonRef.current?.clientWidth}px`}
-        role="menu"
-      >
+      <Box $maxWidth="320px">
         {topMessage && (
           <Text
             $variation="700"
@@ -106,7 +91,6 @@ export const DropdownMenu = ({
           const isDisabled = option.disabled !== undefined && option.disabled;
           return (
             <BoxButton
-              role="menuitem"
               aria-label={option.label}
               data-testid={option.testId}
               $direction="row"

src/frontend/apps/impress/src/components/Text.tsx

@@ -25,9 +25,7 @@ export interface TextProps extends BoxProps {
   $size?: TextSizes | (string & {});
   $theme?:
     | 'primary'
-    | 'primary-text'
     | 'secondary'
-    | 'secondary-text'
     | 'info'
     | 'success'
     | 'warning'

src/frontend/apps/impress/src/components/TextErrors.tsx

@@ -35,7 +35,6 @@ export const TextErrors = ({
             <Text
               key={`causes-${i}`}
               $theme="danger"
-              $variation="600"
               $textAlign="center"
               {...textProps}
             >
@@ -44,12 +43,7 @@ export const TextErrors = ({
           ))}
 
         {!causes && (
-          <Text
-            $theme="danger"
-            $variation="600"
-            $textAlign="center"
-            {...textProps}
-          >
+          <Text $theme="danger" $textAlign="center" {...textProps}>
             {defaultMessage || t('Something bad happens, please retry.')}
           </Text>
         )}

src/frontend/apps/impress/src/components/quick-search/QuickSearchInput.tsx

@@ -47,7 +47,11 @@ export const QuickSearchInput = ({
         $gap={spacing['2xs']}
         $padding={{ all: 'base' }}
       >
-        {!loading && <Icon iconName="search" $variation="600" />}
+        {!loading && (
+          <span aria-hidden="true">
+            <Icon iconName="search" $variation="600" />
+          </span>
+        )}
         {loading && (
           <div>
             <Loader size="small" />

src/frontend/apps/impress/src/core/config/ConfigProvider.tsx

@@ -3,7 +3,7 @@ import { PropsWithChildren, useEffect } from 'react';
 
 import { Box } from '@/components';
 import { useCunninghamTheme } from '@/cunningham';
-import { CrispProvider, PostHogProvider } from '@/services';
+import { PostHogProvider, configureCrispSession } from '@/services';
 import { useSentryStore } from '@/stores/useSentryStore';
 
 import { useConfig } from './api/useConfig';
@@ -29,6 +29,14 @@ export const ConfigProvider = ({ children }: PropsWithChildren) => {
     setTheme(conf.FRONTEND_THEME);
   }, [conf?.FRONTEND_THEME, setTheme]);
 
+  useEffect(() => {
+    if (!conf?.CRISP_WEBSITE_ID) {
+      return;
+    }
+
+    configureCrispSession(conf.CRISP_WEBSITE_ID);
+  }, [conf?.CRISP_WEBSITE_ID]);
+
   if (!conf) {
     return (
       <Box $height="100vh" $width="100vw" $align="center" $justify="center">
@@ -37,11 +45,5 @@ export const ConfigProvider = ({ children }: PropsWithChildren) => {
     );
   }
 
-  return (
-    <PostHogProvider conf={conf.POSTHOG_KEY}>
-      <CrispProvider websiteId={conf?.CRISP_WEBSITE_ID}>
-        {children}
-      </CrispProvider>
-    </PostHogProvider>
-  );
+  return <PostHogProvider conf={conf.POSTHOG_KEY}>{children}</PostHogProvider>;
 };

src/frontend/apps/impress/src/cunningham/cunningham-style.css

@@ -203,6 +203,7 @@ input:-webkit-autofill:focus {
 
 .c__select__wrapper .c__select__inner__actions__open:focus {
   outline: none;
+  
 }
 
 .c__select__wrapper .labelled-box__label.c__offscreen {
@@ -310,7 +311,7 @@ input:-webkit-autofill:focus {
 }
 
 /**
- * Checkbox
+ * Others
 */
 .c__checkbox:focus-within {
   border-color: transparent;
@@ -365,8 +366,7 @@ input:-webkit-autofill:focus {
 }
 
 .c__button--medium {
-  height: auto;
-  min-height: var(--c--components--button--medium-height);
+  padding: 0.9rem var(--c--theme--spacings--s);
 }
 
 .c__button--small {
@@ -407,10 +407,6 @@ input:-webkit-autofill:focus {
   );
 }
 
-.c__button--primary-text {
-  color: var(--c--components--button--primary-text--color);
-}
-
 .c__button--primary-text:hover,
 .c__button--primary-text:focus-visible {
   background-color: var(
@@ -552,8 +548,6 @@ input:-webkit-autofill:focus {
 
 .c__modal__close .c__button {
   padding: 0 !important;
-  top: -0.65rem;
-  right: -0.65rem;
 }
 
 .c__modal--full .c__modal__content {
@@ -614,20 +608,29 @@ input:-webkit-autofill:focus {
 }
 
 /**
- * Alert
-*/
-.c__alert--error {
-  background-color: var(--c--components--alert--error--background-color);
-  border-left-color: var(--c--components--alert--error--border-left-color);
+ * lecture ou non des icons
+ */
+ .sr-only {
+  position: absolute;
+  width: 1px;
+  height: 1px;
+  padding: 0;
+  margin: -1px;
+  overflow: hidden;
+  clip: rect(0, 0, 0, 0);
+  white-space: nowrap;
+  border: 0;
 }
 
-.c__alert--error .c__button--tertiary {
-  background-color: var(--c--components--alert--error--close--background-color);
-  color: var(--c--components--alert--error--close--color);
+[data-icon]:before {
+  font-family: 'Material Icons';
+  content: attr(data-icon);
 }
 
-.c__alert.c__alert--error .c__button--tertiary:hover {
+button:focus {
   background-color: var(
-    --c--components--alert--error--close--background-color-hover
+    --c--components--button--primary-text--background--color-hover
   );
-}
+  border-radius: var(--c--components--button--border-radius--focus);
+  box-shadow: 0 0 0 2px var(--c--theme--colors--primary-400)
+}

src/frontend/apps/impress/src/cunningham/cunningham-tokens.css

@@ -484,19 +484,6 @@
   --c--theme--logo--widthFooter: 220px;
   --c--theme--logo--alt: gouvernement logo;
   --c--components--alert--border-radius: 0;
-  --c--components--alert--error--background-color: var(
-    --c--theme--colors--danger-100
-  );
-  --c--components--alert--error--border-left-color: var(
-    --c--theme--colors--danger-400
-  );
-  --c--components--alert--error--close--color: white;
-  --c--components--alert--error--close--background-color: var(
-    --c--theme--colors--danger-400
-  );
-  --c--components--alert--error--close--background-color-hover: var(
-    --c--theme--colors--danger-600
-  );
   --c--components--modal--width-small: 342px;
   --c--components--button--medium-height: 40px;
   --c--components--button--medium-text-height: 40px;
@@ -518,9 +505,6 @@
   --c--components--button--primary-text--color-hover: var(
     --c--theme--colors--primary-text
   );
-  --c--components--button--primary-text--color: var(
-    --c--theme--colors--primary-800
-  );
   --c--components--button--secondary--background--color-hover: #f6f6f6;
   --c--components--button--secondary--background--color-active: #ededed;
   --c--components--button--secondary--border--color: var(

src/frontend/apps/impress/src/cunningham/cunningham-tokens.ts

@@ -483,18 +483,7 @@ export const tokens = {
         },
       },
       components: {
-        alert: {
-          'border-radius': '0',
-          error: {
-            'background-color': 'var(--c--theme--colors--danger-100)',
-            'border-left-color': 'var(--c--theme--colors--danger-400)',
-            close: {
-              color: 'white',
-              'background-color': 'var(--c--theme--colors--danger-400)',
-              'background-color-hover': 'var(--c--theme--colors--danger-600)',
-            },
-          },
-        },
+        alert: { 'border-radius': '0' },
         modal: { 'width-small': '342px' },
         button: {
           'medium-height': '40px',
@@ -516,7 +505,6 @@ export const tokens = {
               'color-active': 'var(--c--theme--colors--primary-100)',
             },
             'color-hover': 'var(--c--theme--colors--primary-text)',
-            color: 'var(--c--theme--colors--primary-800)',
           },
           secondary: {
             background: { 'color-hover': '#F6F6F6', 'color-active': '#EDEDED' },

src/frontend/apps/impress/src/features/auth/api/useAuthQuery.tsx

@@ -1,6 +1,6 @@
 import { UseQueryOptions, useQuery } from '@tanstack/react-query';
 
-import { APIError, errorCauses, fetchAPI } from '@/api';
+import { APIError, fetchAPI } from '@/api';
 
 import { User } from './types';
 
@@ -17,10 +17,7 @@ import { User } from './types';
 export const getMe = async (): Promise<User> => {
   const response = await fetchAPI(`users/me/`);
   if (!response.ok) {
-    throw new APIError(
-      `Couldn't fetch user data: ${response.statusText}`,
-      await errorCauses(response),
-    );
+    throw new Error(`Couldn't fetch user data: ${response.statusText}`);
   }
   return response.json() as Promise<User>;
 };

src/frontend/apps/impress/src/features/auth/components/Auth.tsx

@@ -5,7 +5,6 @@ import { PropsWithChildren } from 'react';
 import { Box } from '@/components';
 
 import { useAuth } from '../hooks';
-import { getAuthUrl } from '../utils';
 
 export const Auth = ({ children }: PropsWithChildren) => {
   const { isLoading, pathAllowed, isFetchedAfterMount, authenticated } =
@@ -20,22 +19,6 @@ export const Auth = ({ children }: PropsWithChildren) => {
     );
   }
 
-  /**
-   * If the user is authenticated and wanted initially to access a document,
-   * we redirect to the document page.
-   */
-  if (authenticated) {
-    const authUrl = getAuthUrl();
-    if (authUrl) {
-      void replace(authUrl);
-      return (
-        <Box $height="100vh" $width="100vw" $align="center" $justify="center">
-          <Loader />
-        </Box>
-      );
-    }
-  }
-
   /**
    * If the user is not authenticated and the path is not allowed, we redirect to the login page.
    */

src/frontend/apps/impress/src/features/auth/components/ButtonLogin.tsx

@@ -14,11 +14,7 @@ export const ButtonLogin = () => {
 
   if (!authenticated) {
     return (
-      <Button
-        onClick={() => gotoLogin()}
-        color="primary-text"
-        aria-label={t('Login')}
-      >
+      <Button onClick={gotoLogin} color="primary-text" aria-label={t('Login')}>
         {t('Login')}
       </Button>
     );
@@ -36,7 +32,7 @@ export const ProConnectButton = () => {
 
   return (
     <BoxButton
-      onClick={() => gotoLogin()}
+      onClick={gotoLogin}
       aria-label={t('Proconnect Login')}
       $css={css`
         background-color: var(--c--theme--colors--primary-text);

src/frontend/apps/impress/src/features/auth/hooks/useAuth.tsx

@@ -2,12 +2,13 @@ import { useRouter } from 'next/router';
 import { useEffect, useState } from 'react';
 
 import { useAuthQuery } from '../api';
+import { getAuthUrl } from '../utils';
 
 const regexpUrlsAuth = [/\/docs\/$/g, /\/docs$/g, /^\/$/g];
 
 export const useAuth = () => {
   const { data: user, ...authStates } = useAuthQuery();
-  const { pathname } = useRouter();
+  const { pathname, replace } = useRouter();
 
   const [pathAllowed, setPathAllowed] = useState<boolean>(
     !regexpUrlsAuth.some((regexp) => !!pathname.match(regexp)),
@@ -17,10 +18,17 @@ export const useAuth = () => {
     setPathAllowed(!regexpUrlsAuth.some((regexp) => !!pathname.match(regexp)));
   }, [pathname]);
 
-  return {
-    user,
-    authenticated: !!user && authStates.isSuccess,
-    pathAllowed,
-    ...authStates,
-  };
+  // Redirect to the path before login
+  useEffect(() => {
+    if (!user) {
+      return;
+    }
+
+    const authUrl = getAuthUrl();
+    if (authUrl) {
+      void replace(authUrl);
+    }
+  }, [user, replace]);
+
+  return { user, authenticated: !!user, pathAllowed, ...authStates };
 };

src/frontend/apps/impress/src/features/auth/index.ts

@@ -1,4 +1,4 @@
-export * from './api';
+export * from './api/types';
 export * from './components';
 export * from './hooks';
 export * from './utils';

src/frontend/apps/impress/src/features/auth/utils.ts

@@ -16,11 +16,8 @@ export const setAuthUrl = () => {
   }
 };
 
-export const gotoLogin = (withRedirect = true) => {
-  if (withRedirect) {
-    setAuthUrl();
-  }
-
+export const gotoLogin = () => {
+  setAuthUrl();
   window.location.replace(LOGIN_URL);
 };
 

src/frontend/apps/impress/src/features/docs/doc-editor/components/AIButton.tsx

@@ -20,7 +20,7 @@ import {
   AITransformActions,
   useDocAITransform,
   useDocAITranslate,
-} from '../../api';
+} from '../api/';
 
 type LanguageTranslate = {
   value: string;

src/frontend/apps/impress/src/features/docs/doc-editor/components/BlockNoteEditor.tsx

@@ -1,7 +1,6 @@
 import {
   BlockNoteSchema,
   Dictionary,
-  defaultBlockSpecs,
   locales,
   withPageBreak,
 } from '@blocknote/core';
@@ -26,17 +25,9 @@ import { cssEditor } from '../styles';
 import { randomColor } from '../utils';
 
 import { BlockNoteSuggestionMenu } from './BlockNoteSuggestionMenu';
-import { BlockNoteToolbar } from './BlockNoteToolBar/BlockNoteToolbar';
-import { QuoteBlock } from './custom-blocks';
+import { BlockNoteToolbar } from './BlockNoteToolbar';
 
-export const blockNoteSchema = withPageBreak(
-  BlockNoteSchema.create({
-    blockSpecs: {
-      ...defaultBlockSpecs,
-      quote: QuoteBlock,
-    },
-  }),
-);
+export const blockNoteSchema = withPageBreak(BlockNoteSchema.create());
 
 interface BlockNoteEditorProps {
   doc: Doc;
@@ -134,7 +125,7 @@ export const BlockNoteEditor = ({ doc, provider }: BlockNoteEditorProps) => {
       $css={cssEditor(readOnly)}
     >
       {errorAttachment && (
-        <Box $margin={{ bottom: 'big', top: 'none', horizontal: 'large' }}>
+        <Box $margin={{ bottom: 'big' }}>
           <TextErrors
             causes={errorAttachment.cause}
             canClose
@@ -150,8 +141,8 @@ export const BlockNoteEditor = ({ doc, provider }: BlockNoteEditorProps) => {
         editable={!readOnly}
         theme="light"
       >
-        <BlockNoteSuggestionMenu />
         <BlockNoteToolbar />
+        <BlockNoteSuggestionMenu />
       </BlockNoteView>
     </Box>
   );

src/frontend/apps/impress/src/features/docs/doc-editor/components/BlockNoteSuggestionMenu.tsx

@@ -5,19 +5,13 @@ import {
   getDefaultReactSlashMenuItems,
   getPageBreakReactSlashMenuItems,
   useBlockNoteEditor,
-  useDictionary,
 } from '@blocknote/react';
 import React, { useMemo } from 'react';
-import { useTranslation } from 'react-i18next';
 
-import { DocsBlockSchema } from '../types';
-
-import { getQuoteReactSlashMenuItems } from './custom-blocks';
+import { DocsBlockNoteEditor } from '../types';
 
 export const BlockNoteSuggestionMenu = () => {
-  const editor = useBlockNoteEditor<DocsBlockSchema>();
-  const { t } = useTranslation();
-  const basicBlocksName = useDictionary().slash_menu.page_break.group;
+  const editor = useBlockNoteEditor() as DocsBlockNoteEditor;
 
   const getSlashMenuItems = useMemo(() => {
     return async (query: string) =>
@@ -26,12 +20,11 @@ export const BlockNoteSuggestionMenu = () => {
           combineByGroup(
             getDefaultReactSlashMenuItems(editor),
             getPageBreakReactSlashMenuItems(editor),
-            getQuoteReactSlashMenuItems(editor, t, basicBlocksName),
           ),
           query,
         ),
       );
-  }, [basicBlocksName, editor, t]);
+  }, [editor]);
 
   return (
     <SuggestionMenuController

src/frontend/apps/impress/src/features/docs/doc-editor/components/BlockNoteToolBar/BlockNoteToolbar.tsx

@@ -1,75 +0,0 @@
-import '@blocknote/mantine/style.css';
-import {
-  FormattingToolbar,
-  FormattingToolbarController,
-  blockTypeSelectItems,
-  getFormattingToolbarItems,
-  useDictionary,
-} from '@blocknote/react';
-import React, { useCallback, useMemo, useState } from 'react';
-import { useTranslation } from 'react-i18next';
-
-import { getQuoteFormattingToolbarItems } from '../custom-blocks';
-
-import { AIGroupButton } from './AIButton';
-import { FileDownloadButton } from './FileDownloadButton';
-import { MarkdownButton } from './MarkdownButton';
-import { ModalConfirmDownloadUnsafe } from './ModalConfirmDownloadUnsafe';
-
-export const BlockNoteToolbar = () => {
-  const dict = useDictionary();
-  const [confirmOpen, setIsConfirmOpen] = useState(false);
-  const [onConfirm, setOnConfirm] = useState<() => void | Promise<void>>();
-  const { t } = useTranslation();
-
-  const toolbarItems = useMemo(() => {
-    const toolbarItems = getFormattingToolbarItems([
-      ...blockTypeSelectItems(dict),
-      getQuoteFormattingToolbarItems(t),
-    ]);
-    const fileDownloadButtonIndex = toolbarItems.findIndex(
-      (item) => item.key === 'fileDownloadButton',
-    );
-    if (fileDownloadButtonIndex !== -1) {
-      toolbarItems.splice(
-        fileDownloadButtonIndex,
-        1,
-        <FileDownloadButton
-          key="fileDownloadButton"
-          open={(onConfirm) => {
-            setIsConfirmOpen(true);
-            setOnConfirm(() => onConfirm);
-          }}
-        />,
-      );
-    }
-
-    return toolbarItems;
-  }, [dict, t]);
-
-  const formattingToolbar = useCallback(() => {
-    return (
-      <FormattingToolbar>
-        {toolbarItems}
-
-        {/* Extra button to do some AI powered actions */}
-        <AIGroupButton key="AIButton" />
-
-        {/* Extra button to convert from markdown to json */}
-        <MarkdownButton key="customButton" />
-      </FormattingToolbar>
-    );
-  }, [toolbarItems]);
-
-  return (
-    <>
-      <FormattingToolbarController formattingToolbar={formattingToolbar} />
-      {confirmOpen && (
-        <ModalConfirmDownloadUnsafe
-          onClose={() => setIsConfirmOpen(false)}
-          onConfirm={onConfirm}
-        />
-      )}
-    </>
-  );
-};

src/frontend/apps/impress/src/features/docs/doc-editor/components/BlockNoteToolBar/FileDownloadButton.tsx

@@ -1,111 +0,0 @@
-import {
-  BlockSchema,
-  InlineContentSchema,
-  StyleSchema,
-  checkBlockIsFileBlock,
-  checkBlockIsFileBlockWithPlaceholder,
-} from '@blocknote/core';
-import {
-  useBlockNoteEditor,
-  useComponentsContext,
-  useDictionary,
-  useSelectedBlocks,
-} from '@blocknote/react';
-import { useCallback, useMemo } from 'react';
-import { RiDownload2Fill } from 'react-icons/ri';
-
-import { downloadFile, exportResolveFileUrl } from '@/features/docs/doc-export';
-
-export const FileDownloadButton = ({
-  open,
-}: {
-  open: (onConfirm: () => Promise<void> | void) => void;
-}) => {
-  const dict = useDictionary();
-  const Components = useComponentsContext();
-
-  const editor = useBlockNoteEditor<
-    BlockSchema,
-    InlineContentSchema,
-    StyleSchema
-  >();
-
-  const selectedBlocks = useSelectedBlocks(editor);
-
-  const fileBlock = useMemo(() => {
-    // Checks if only one block is selected.
-    if (selectedBlocks.length !== 1) {
-      return undefined;
-    }
-
-    const block = selectedBlocks[0];
-
-    if (checkBlockIsFileBlock(block, editor)) {
-      return block;
-    }
-
-    return undefined;
-  }, [editor, selectedBlocks]);
-
-  const onClick = useCallback(async () => {
-    if (fileBlock && fileBlock.props.url) {
-      editor.focus();
-
-      const url = fileBlock.props.url as string;
-
-      /**
-       * If not hosted on our domain, means not a file uploaded by the user,
-       * we do what Blocknote was doing initially.
-       */
-      if (!url.includes(window.location.hostname)) {
-        if (!editor.resolveFileUrl) {
-          window.open(url);
-        } else {
-          void editor
-            .resolveFileUrl(url)
-            .then((downloadUrl) => window.open(downloadUrl));
-        }
-
-        return;
-      }
-
-      if (!url.includes('-unsafe')) {
-        const blob = (await exportResolveFileUrl(url, undefined)) as Blob;
-        downloadFile(blob, url.split('/').pop() || 'file');
-      } else {
-        const onConfirm = async () => {
-          const blob = (await exportResolveFileUrl(url, undefined)) as Blob;
-          downloadFile(blob, url.split('/').pop() || 'file (unsafe)');
-        };
-
-        open(onConfirm);
-      }
-    }
-  }, [editor, fileBlock, open]);
-
-  if (
-    !fileBlock ||
-    checkBlockIsFileBlockWithPlaceholder(fileBlock, editor) ||
-    !Components
-  ) {
-    return null;
-  }
-
-  return (
-    <>
-      <Components.FormattingToolbar.Button
-        className="bn-button"
-        label={
-          dict.formatting_toolbar.file_download.tooltip[fileBlock.type] ||
-          dict.formatting_toolbar.file_download.tooltip['file']
-        }
-        mainTooltip={
-          dict.formatting_toolbar.file_download.tooltip[fileBlock.type] ||
-          dict.formatting_toolbar.file_download.tooltip['file']
-        }
-        icon={<RiDownload2Fill />}
-        onClick={() => void onClick()}
-      />
-    </>
-  );
-};

src/frontend/apps/impress/src/features/docs/doc-editor/components/BlockNoteToolBar/ModalConfirmDownloadUnsafe.tsx

@@ -1,74 +0,0 @@
-import { Button, Modal, ModalSize } from '@openfun/cunningham-react';
-import { useTranslation } from 'react-i18next';
-
-import { Box, Text } from '@/components';
-
-interface ModalConfirmDownloadUnsafeProps {
-  onClose: () => void;
-  onConfirm?: () => Promise<void> | void;
-}
-
-export const ModalConfirmDownloadUnsafe = ({
-  onConfirm,
-  onClose,
-}: ModalConfirmDownloadUnsafeProps) => {
-  const { t } = useTranslation();
-
-  return (
-    <Modal
-      isOpen
-      closeOnClickOutside
-      onClose={() => onClose()}
-      rightActions={
-        <>
-          <Button
-            aria-label={t('Close the modal')}
-            color="secondary"
-            onClick={() => onClose()}
-          >
-            {t('Cancel')}
-          </Button>
-          <Button
-            aria-label={t('Download')}
-            color="danger"
-            onClick={() => {
-              console.log('onClick');
-              if (onConfirm) {
-                void onConfirm();
-              }
-              onClose();
-            }}
-          >
-            {t('Download anyway')}
-          </Button>
-        </>
-      }
-      size={ModalSize.SMALL}
-      title={
-        <Text
-          $gap="0.7rem"
-          $size="h6"
-          $align="flex-start"
-          $variation="1000"
-          $direction="row"
-        >
-          <Text $isMaterialIcon $theme="warning">
-            warning
-          </Text>
-          {t('Warning')}
-        </Text>
-      }
-    >
-      <Box aria-label={t('Modal confirmation to download the attachment')}>
-        <Box>
-          <Box $direction="column" $gap="0.35rem" $margin={{ top: 'sm' }}>
-            <Text $variation="700">{t('This file is flagged as unsafe.')}</Text>
-            <Text $variation="600">
-              {t('Please download it only if it comes from a trusted source.')}
-            </Text>
-          </Box>
-        </Box>
-      </Box>
-    </Modal>
-  );
-};

src/frontend/apps/impress/src/features/docs/doc-editor/components/BlockNoteToolbar.tsx

@@ -0,0 +1,30 @@
+import '@blocknote/mantine/style.css';
+import {
+  FormattingToolbar,
+  FormattingToolbarController,
+  FormattingToolbarProps,
+  getFormattingToolbarItems,
+} from '@blocknote/react';
+import React, { useCallback } from 'react';
+
+import { AIGroupButton } from './AIButton';
+import { MarkdownButton } from './MarkdownButton';
+
+export const BlockNoteToolbar = () => {
+  const formattingToolbar = useCallback(
+    ({ blockTypeSelectItems }: FormattingToolbarProps) => (
+      <FormattingToolbar>
+        {getFormattingToolbarItems(blockTypeSelectItems)}
+
+        {/* Extra button to do some AI powered actions */}
+        <AIGroupButton key="AIButton" />
+
+        {/* Extra button to convert from markdown to json */}
+        <MarkdownButton key="customButton" />
+      </FormattingToolbar>
+    ),
+    [],
+  );
+
+  return <FormattingToolbarController formattingToolbar={formattingToolbar} />;
+};

src/frontend/apps/impress/src/features/docs/doc-editor/components/custom-blocks/QuoteBlock.tsx

@@ -1,77 +0,0 @@
-import { defaultProps, insertOrUpdateBlock } from '@blocknote/core';
-import { BlockTypeSelectItem, createReactBlockSpec } from '@blocknote/react';
-import { TFunction } from 'i18next';
-import React from 'react';
-
-import { Text } from '@/components';
-import { useCunninghamTheme } from '@/cunningham';
-
-import { DocsBlockNoteEditor } from '../../types';
-
-export const QuoteBlock = createReactBlockSpec(
-  {
-    type: 'quote',
-    propSchema: {
-      textAlignment: defaultProps.textAlignment,
-      textColor: defaultProps.textColor,
-    },
-    content: 'inline',
-  },
-  {
-    render: (props) => {
-      // eslint-disable-next-line react-hooks/rules-of-hooks
-      const { colorsTokens } = useCunninghamTheme();
-
-      return (
-        <Text
-          className="inline-content"
-          $margin="0 0 1rem 0"
-          $padding="0.5rem 1rem"
-          $variation="600"
-          style={{
-            borderLeft: `4px solid ${colorsTokens()['greyscale-300']}`,
-            fontStyle: 'italic',
-            flexGrow: 1,
-          }}
-          ref={props.contentRef}
-        />
-      );
-    },
-  },
-);
-
-export const getQuoteReactSlashMenuItems = (
-  editor: DocsBlockNoteEditor,
-  t: TFunction<'translation', undefined>,
-  group: string,
-) => [
-  {
-    title: t('Quote'),
-    onItemClick: () => {
-      insertOrUpdateBlock(editor, {
-        type: 'quote',
-      });
-    },
-    aliases: ['quote', 'blockquote', 'citation'],
-    group,
-    icon: (
-      <Text $isMaterialIcon $size="18px">
-        format_quote
-      </Text>
-    ),
-    subtext: t('Add a quote block'),
-  },
-];
-
-export const getQuoteFormattingToolbarItems = (
-  t: TFunction<'translation', undefined>,
-): BlockTypeSelectItem => ({
-  name: t('Quote'),
-  type: 'quote',
-  icon: () => (
-    <Text $isMaterialIcon $size="16px">
-      format_quote
-    </Text>
-  ),
-  isSelected: (block) => block.type === 'quote',
-});

src/frontend/apps/impress/src/features/docs/doc-editor/components/custom-blocks/index.ts

@@ -1 +0,0 @@
-export * from './QuoteBlock';

src/frontend/apps/impress/src/features/docs/doc-editor/components/index.ts

@@ -1,2 +1 @@
 export * from './DocEditor';
-export * from './custom-blocks/';

src/frontend/apps/impress/src/features/docs/doc-editor/styles.tsx

@@ -6,14 +6,6 @@ export const cssEditor = (readonly: boolean) => css`
   & .ProseMirror {
     height: 100%;
 
-    img.bn-visual-media[src*='-unsafe'] {
-      pointer-events: none;
-    }
-
-    .bn-side-menu[data-block-type='quote'] {
-      height: 46px;
-    }
-
     .collaboration-cursor-custom__base {
       position: relative;
     }
@@ -33,7 +25,6 @@ export const cssEditor = (readonly: boolean) => css`
       user-select: none;
       position: absolute;
       top: -17px;
-      left: 0px;
       padding: 0px 6px;
       border-radius: 0px;
       white-space: nowrap;
@@ -84,13 +75,13 @@ export const cssEditor = (readonly: boolean) => css`
 
   .bn-block-outer:not(:first-child) {
     &:has(h1) {
-      margin-top: 32px;
+      padding-top: 32px;
     }
     &:has(h2) {
-      margin-top: 24px;
+      padding-top: 24px;
     }
     &:has(h3) {
-      margin-top: 16px;
+      padding-top: 16px;
     }
   }
 
@@ -100,16 +91,9 @@ export const cssEditor = (readonly: boolean) => css`
     border-radius: 4px;
   }
 
-  @media screen and (width <= 768px) {
-    & .bn-editor {
-      padding-right: 36px;
-    }
-  }
-
   @media screen and (width <= 560px) {
     & .bn-editor {
       ${readonly && `padding-left: 10px;`}
-      padding-right: 10px;
     }
     .bn-side-menu[data-block-type='heading'][data-level='1'] {
       height: 46px;

src/frontend/apps/impress/src/features/docs/doc-editor/types.tsx

@@ -17,12 +17,8 @@ export type HeadingBlock = {
   };
 };
 
-export type DocsBlockSchema = typeof blockNoteSchema.blockSchema;
-export type DocsInlineContentSchema =
-  typeof blockNoteSchema.inlineContentSchema;
-export type DocsStyleSchema = typeof blockNoteSchema.styleSchema;
 export type DocsBlockNoteEditor = BlockNoteEditor<
-  DocsBlockSchema,
-  DocsInlineContentSchema,
-  DocsStyleSchema
+  typeof blockNoteSchema.blockSchema,
+  typeof blockNoteSchema.inlineContentSchema,
+  typeof blockNoteSchema.styleSchema
 >;

src/frontend/apps/impress/src/features/docs/doc-export/blocks-mapping/headingPDF.tsx

@@ -1,25 +0,0 @@
-import { Text } from '@react-pdf/renderer';
-
-import { DocsExporterPDF } from '../types';
-
-export const blockMappingHeadingPDF: DocsExporterPDF['mappings']['blockMapping']['heading'] =
-  (block, exporter) => {
-    const PIXELS_PER_POINT = 0.75;
-    const MERGE_RATIO = 7.5;
-    const FONT_SIZE = 16;
-    const fontSizeEM =
-      block.props.level === 1 ? 2 : block.props.level === 2 ? 1.5 : 1.17;
-    return (
-      <Text
-        key={block.id}
-        style={{
-          fontSize: fontSizeEM * FONT_SIZE * PIXELS_PER_POINT,
-          fontWeight: 700,
-          marginTop: `${fontSizeEM * MERGE_RATIO}px`,
-          marginBottom: `${fontSizeEM * MERGE_RATIO}px`,
-        }}
-      >
-        {exporter.transformInlineContent(block.content)}
-      </Text>
-    );
-  };

src/frontend/apps/impress/src/features/docs/doc-export/blocks-mapping/index.ts

@@ -1,5 +0,0 @@
-export * from './headingPDF';
-export * from './paragraphPDF';
-export * from './quoteDocx';
-export * from './quotePDF';
-export * from './tablePDF';

src/frontend/apps/impress/src/features/docs/doc-export/blocks-mapping/paragraphPDF.tsx

@@ -1,31 +0,0 @@
-import { Text } from '@react-pdf/renderer';
-
-import { DocsExporterPDF } from '../types';
-
-export const blockMappingParagraphPDF: DocsExporterPDF['mappings']['blockMapping']['paragraph'] =
-  (block, exporter) => {
-    /**
-     * Breakline in the editor are not rendered in the PDF
-     * By adding a space if the block is empty we ensure that the block is rendered
-     */
-    if (Array.isArray(block.content)) {
-      block.content.forEach((content) => {
-        if (content.type === 'text' && !content.text) {
-          content.text = ' ';
-        }
-      });
-
-      if (!block.content.length) {
-        block.content.push({
-          styles: {},
-          text: ' ',
-          type: 'text',
-        });
-      }
-    }
-    return (
-      <Text key={block.id}>
-        {exporter.transformInlineContent(block.content)}
-      </Text>
-    );
-  };

src/frontend/apps/impress/src/features/docs/doc-export/blocks-mapping/quoteDocx.tsx

@@ -1,33 +0,0 @@
-import { Paragraph } from 'docx';
-
-import { DocsExporterDocx } from '../types';
-import { docxBlockPropsToStyles } from '../utils';
-
-export const blockMappingQuoteDocx: DocsExporterDocx['mappings']['blockMapping']['quote'] =
-  (block, exporter) => {
-    if (Array.isArray(block.content)) {
-      block.content.forEach((content) => {
-        if (content.type === 'text') {
-          content.styles = {
-            ...content.styles,
-            italic: true,
-            textColor: 'gray',
-          };
-        }
-      });
-    }
-
-    return new Paragraph({
-      ...docxBlockPropsToStyles(block.props, exporter.options.colors),
-      spacing: { before: 10, after: 10 },
-      border: {
-        left: {
-          color: '#cecece',
-          space: 4,
-          style: 'thick',
-        },
-      },
-      style: 'Normal',
-      children: exporter.transformInlineContent(block.content),
-    });
-  };

src/frontend/apps/impress/src/features/docs/doc-export/blocks-mapping/quotePDF.tsx

@@ -1,21 +0,0 @@
-import { Text } from '@react-pdf/renderer';
-
-import { DocsExporterPDF } from '../types';
-
-export const blockMappingQuotePDF: DocsExporterPDF['mappings']['blockMapping']['quote'] =
-  (block, exporter) => {
-    return (
-      <Text
-        style={{
-          fontStyle: 'italic',
-          marginVertical: 10,
-          paddingVertical: 5,
-          paddingLeft: 10,
-          borderLeft: '4px solid #cecece',
-          color: '#666',
-        }}
-      >
-        {exporter.transformInlineContent(block.content)}
-      </Text>
-    );
-  };

src/frontend/apps/impress/src/features/docs/doc-export/blocks-mapping/tablePDF.tsx

@@ -1,52 +0,0 @@
-import { TD, TH, TR, Table } from '@ag-media/react-pdf-table';
-import { View } from '@react-pdf/renderer';
-
-import { DocsExporterPDF } from '../types';
-
-export const blockMappingTablePDF: DocsExporterPDF['mappings']['blockMapping']['table'] =
-  (block, exporter) => {
-    return (
-      <Table>
-        {block.content.rows.map((row, index) => {
-          if (index === 0) {
-            return (
-              <TH key={index}>
-                {row.cells.map((cell, index) => {
-                  // Make empty cells are rendered.
-                  if (cell.length === 0) {
-                    cell.push({
-                      styles: {},
-                      text: ' ',
-                      type: 'text',
-                    });
-                  }
-                  return (
-                    <TD key={index}>{exporter.transformInlineContent(cell)}</TD>
-                  );
-                })}
-              </TH>
-            );
-          }
-          return (
-            <TR key={index}>
-              {row.cells.map((cell, index) => {
-                // Make empty cells are rendered.
-                if (cell.length === 0) {
-                  cell.push({
-                    styles: {},
-                    text: ' ',
-                    type: 'text',
-                  });
-                }
-                return (
-                  <TD key={index}>
-                    <View>{exporter.transformInlineContent(cell)}</View>
-                  </TD>
-                );
-              })}
-            </TR>
-          );
-        })}
-      </Table>
-    );
-  };

src/frontend/apps/impress/src/features/docs/doc-export/components/index.ts

@@ -1 +0,0 @@
-export * from './ModalExport';

src/frontend/apps/impress/src/features/docs/doc-export/index.ts

@@ -1,2 +0,0 @@
-export * from './components';
-export * from './utils';