Merge remote-tracking branch 'origin/ecospheres-demo' into ecospheres-prod-major-merge

Author: abulte

.github/workflows/create-deploy-release.yml

@@ -103,7 +103,7 @@ jobs:
         shell: bash
         run: |
           git clone --quiet --depth 1 -b ${{ env.REPO_CURRENT_BRANCH }} ${{ env.REPO_SSH_URL }}
-          git clone --quiet --depth 1 -b pnpm-ufk $SCAFFOLD_REPO_SSH_URL ${{ env.SCAFFOLD_DIR }}
+          git clone --quiet --depth 1 $SCAFFOLD_REPO_SSH_URL ${{ env.SCAFFOLD_DIR }}
 
       - name: Parse commit message and set deployment variables
         if: (github.event_name == 'push' && startsWith(github.ref, 'refs/heads/') && startsWith(github.event.head_commit.message, '['))
@@ -145,7 +145,9 @@ jobs:
           pattern="${CONFIG_NAME}-${ENV}-[0-9]+\.[0-9]+\.[0-9]+$"
           tags=$(git ls-remote --tags origin | grep -Eo "refs/tags/${pattern}" | sed 's#refs/tags/##' || true)
           if [ -n "$tags" ]; then
-              for tag in $tags; do git fetch origin tag $tag; done
+              # --no-tags prevents git from auto-following related tags,
+              # which would cause "tag already exists" errors on subsequent fetches
+              for tag in $tags; do git fetch --no-tags origin tag $tag; done
           fi
           RELEASE_VERSION=$(../${{ env.SCAFFOLD_DIR }}/scripts/bump_version_non_semver.sh $CONFIG_NAME $ENV $VERSION_TYPE)
 

.github/workflows/review-app.yml

@@ -32,6 +32,7 @@ jobs:
       group: ${{ github.workflow }}-${{ github.ref }}-${{ matrix.site }}
       cancel-in-progress: false
     strategy:
+      max-parallel: 2
       matrix:
         site:
           - ecospheres

.npmrc

@@ -11,6 +11,7 @@ node-linker=hoisted
 # Security: Only install packages released at least 4 days ago
 # Protects against supply-chain attacks by giving community time to detect malicious releases
 minimum-release-age=5760
+minimum-release-age-exclude[]=@ecolabdata/*
 
 # Enforce pnpm usage - fail if npm or yarn is used
 engine-strict=true

README.md

@@ -169,11 +169,78 @@ Les **review apps** ne sont **pas créées automatiquement** lors de l'ouverture
 
 ### 🏭 Déploiement en preprod et en production
 
-Le déploiement des verticales thématiques en preprod et en production s'effectue via un workflow GitHub qui peut être déclenché de deux manières différentes :
-
 #### Comment déployer en préproduction et en production
 
-## Solution 1 - par le message de Git commit
+##### Solution 1 : Script de déploiement local (recommandé)
+
+Un script bash `scripts/deploy.sh` simplifie le processus de déploiement en deux étapes.
+
+**Prérequis :** [GitHub CLI (`gh`)](https://cli.github.com/) et `jq` installés, `gh` authentifié.
+
+###### Étape 1 : Préparer le déploiement
+
+Crée une branche de merge temporaire, fusionne les changements (avec résolution de conflits si nécessaire), et crée une PR draft :
+
+```bash
+# Pour demo/preprod (source = main par défaut)
+./scripts/deploy.sh prepare <site> <env> <version>
+
+# Pour prod (--source obligatoire)
+./scripts/deploy.sh prepare <site> prod <version> --source <site>-preprod
+```
+
+**Exemples :**
+
+```bash
+./scripts/deploy.sh prepare ecospheres demo minor
+./scripts/deploy.sh prepare ecospheres prod minor --source ecospheres-demo
+```
+
+Le script :
+
+- Crée une branche `{site}-{env}-merge` depuis `{site}-{env}`
+- Fusionne la branche source dans cette branche
+- En cas de conflits, vous les résolvez localement puis relancez la commande
+- Pousse la branche et crée une PR draft
+- Poste un commentaire `/deploy {site}` pour créer une review app
+
+###### Étape 2 : Déployer
+
+Après validation et approbation de la PR, lancez le déploiement avec l'URL ou le numéro de la PR :
+
+```bash
+./scripts/deploy.sh deploy <pr>
+
+# Exemples
+./scripts/deploy.sh deploy 123
+./scripts/deploy.sh deploy https://github.com/opendatateam/udata-front-kit/pull/123
+```
+
+Le script :
+
+- Vérifie que la PR est ouverte et approuvée
+- Merge la PR avec le message normalisé `[{env}:{site}:{version}] {titre_pr} #{numéro_pr}`
+- Supprime la branche de merge
+- Pour les déploiements **prod** : crée une release GitHub avec changelog auto-généré
+- Déclenche automatiquement le pipeline GitLab
+
+**Arguments :**
+
+| Argument    | Valeurs possibles                                      |
+| ----------- | ------------------------------------------------------ |
+| `<site>`    | Déterminé dynamiquement depuis les dossiers `configs/` |
+| `<env>`     | `demo`, `preprod`, `prod`                              |
+| `<version>` | `major`, `minor`, `patch`                              |
+
+**Options :**
+
+| Option               | Description                                                               |
+| -------------------- | ------------------------------------------------------------------------- |
+| `--source <branch>`  | Branche source (obligatoire pour prod, défaut : `main` pour demo/preprod) |
+| `--ignore-git-clean` | Ignore la vérification de l'état git (utile pour les tests)               |
+| `--skip-release`     | Ne pas créer de release GitHub (prod uniquement)                          |
+
+##### Solution 2 : Message de commit
 
 Le déploiement des verticales thématiques en preprod et en production peut s'effectuer via un workflow GitHub qui se déclenche automatiquement à partir du message de commit. Le format du message de commit doit être :
 
@@ -204,9 +271,9 @@ Le workflow se déclenche sur tous les push vers toutes les branches, mais ne s'
 
 Toutes les variables et secrets nécessaires pour ce workflow sont listés dans la section `env:` du [workflow de déploiement](.github/workflows/create-deploy-release.yml).
 
-## Solution 2 — sur l'interface web de GitHub Actions
+##### Solution 3 : Interface GitHub Actions
 
-Le déploiement peut également être déclenché manuellement via l'interface GitHub Actions :
+Le déploiement peut être déclenché manuellement via l'interface GitHub Actions :
 
 1. **Aller dans l'onglet "Actions"** du dépôt GitHub
 2. **Sélectionner "Deployment on datagouv domains with version bump"** dans la liste des workflows

configs/ecospheres/config.yaml

@@ -379,6 +379,16 @@ pages:
             name: Zones de gestion, de restriction ou de réglementation et unités de déclaration
           - id: zones-a-risque-naturel
             name: Zones à risque naturel
+      - name: Label de données
+        default_option: Tous les labels
+        id: badge
+        type: select
+        use_filter_prefix: false
+        values:
+          - id: hvd
+            name: Données de forte valeur
+          - id: inspire
+            name: INSPIRE
   # DATASERVICES
   dataservices:
     list_all: true
@@ -458,10 +468,6 @@ pages:
         form:
           required: true
         values: *fnv_themes
-      - name: Couverture territoriale
-        default_option:
-        id: geozone
-        type: spatial_zone
   # INDICATORS
   indicators:
     list_all: true

configs/meteo-france/config.yaml

@@ -69,6 +69,10 @@ website:
               description: 'Archive des observations pour différentes sources de données sol, mer et altitude'
               url: 'datasets?category=observations'
               image_url: '/static/meteo-france/assets/observations.png'
+            - name: 'Bulletins météorologiques'
+              description: 'Archive des bulletins météorologiques pour différentes sources de données : forêt, marine, montagne, climatologie ...'
+              url: 'datasets?category=bulletins'
+              image_url: '/static/meteo-france/assets/bulletins.png'
         sub_section_tiles:
           title:
           tiles:
@@ -223,6 +227,8 @@ pages:
             name: Fiches climatologiques
           - id: observations
             name: Données d'observations météorologiques
+          - id: bulletins
+            name: Archive des bulletins météorologiques
       - name: Sous-catégories
         default_option: Toutes les sous-catégories
         id: subcategory

cypress/e2e/custom/ecospheres/a11y/contrast.cy.ts

@@ -1,6 +1,5 @@
-import type { DataserviceWithRel } from '@/model/dataservice'
 import type { Topic } from '@/model/topic'
-import type { DatasetV2 } from '@datagouv/components-next'
+import type { Dataservice, DatasetV2 } from '@datagouv/components-next'
 import { dataserviceFactory } from 'cypress/support/factories/dataservices_factory'
 import { datasetFactory } from 'cypress/support/factories/datasets_factory'
 import { topicFactory } from 'cypress/support/factories/topics_factory'
@@ -9,7 +8,7 @@ import { mockUniverseOrganizations } from '../mocks'
 describe('a11y contrast testing', () => {
   let testTopics: Topic[]
   let testDatasets: DatasetV2[]
-  let testDataservices: DataserviceWithRel[]
+  let testDataservices: Dataservice[]
 
   beforeEach(() => {
     // global mocks

cypress/e2e/custom/ecospheres/dataservices/detail.cy.ts

@@ -1,15 +1,15 @@
-import type { DataserviceWithRel } from '@/model/dataservice'
-import type { DatasetV2 } from '@datagouv/components-next'
+import type { Dataservice, DatasetV2 } from '@datagouv/components-next'
 import { dataserviceFactory } from '../../../../support/factories/dataservices_factory'
 import { datasetFactory } from '../../../../support/factories/datasets_factory'
+import { createIndicator } from '../indicators/support'
 
 /**
  * Helper to create a dataservice with all necessary mocks
  */
 function createMockedDataservice(
-  overrides: Partial<DataserviceWithRel> = {},
+  overrides: Partial<Dataservice> = {},
   datasets: DatasetV2[] = []
-): DataserviceWithRel {
+): Dataservice {
   const dataservice = dataserviceFactory.one({ overrides })
 
   // Mock the dataservice itself
@@ -32,7 +32,7 @@ function createMockedDataservice(
 }
 
 describe('Dataservices (API) - Detail Page', () => {
-  let testDataservice: DataserviceWithRel
+  let testDataservice: Dataservice
   let testDatasets: DatasetV2[]
 
   beforeEach(() => {
@@ -215,7 +215,9 @@ describe('Dataservices (API) - Detail Page', () => {
 
       // Discussions tab content should be visible
       cy.get('#tab-content-1').should('be.visible')
-      cy.contains('Pas de discussion pour cette API').should('be.visible')
+      cy.contains("Il n'y a pas encore de discussion pour cette API").should(
+        'be.visible'
+      )
     })
 
     it('should switch to informations tab when clicked', () => {
@@ -278,4 +280,39 @@ describe('Dataservices (API) - Detail Page', () => {
       cy.contains('0 jeu de données').should('be.visible')
     })
   })
+
+  describe('Indicator datasets', () => {
+    it('should display indicator datasets with the Indicateur badge', () => {
+      // Create a mix of regular datasets and indicator datasets
+      const regularDataset = datasetFactory.one({
+        overrides: { title: 'Regular Dataset Title' }
+      })
+      const indicatorDataset = createIndicator({
+        title: 'Indicator Dataset Title'
+      })
+
+      const ds = createMockedDataservice({}, [regularDataset, indicatorDataset])
+
+      cy.visit(`/dataservices/${ds.id}`)
+      cy.wait(`@get_dataservices_${ds.id}`)
+      cy.wait('@getDataserviceDatasets')
+
+      // Check that both datasets are displayed
+      cy.contains('Regular Dataset Title').should('be.visible')
+      cy.contains('Indicator Dataset Title').should('be.visible')
+
+      // Check that the indicator dataset has the "Indicateur" badge
+      cy.contains('Indicator Dataset Title')
+        .closest('.fr-col-12')
+        .find('.fr-badge')
+        .contains('Indicateur')
+        .should('be.visible')
+
+      // Check that the regular dataset does NOT have the "Indicateur" badge
+      cy.contains('Regular Dataset Title')
+        .closest('.fr-col-12')
+        .find('.fr-badge')
+        .should('not.exist')
+    })
+  })
 })

cypress/e2e/custom/ecospheres/dataservices/list.cy.ts

@@ -1,9 +1,9 @@
-import type { DataserviceWithRel } from '@/model/dataservice'
+import type { Dataservice } from '@datagouv/components-next'
 import { dataserviceFactory } from '../../../../support/factories/dataservices_factory'
 import { mockUniverseOrganizations } from '../mocks'
 
 describe('Dataservices (API) - List Page', () => {
-  let testDataservices: DataserviceWithRel[]
+  let testDataservices: Dataservice[]
 
   beforeEach(() => {
     cy.mockMatomo()

cypress/e2e/custom/ecospheres/indicators/detail.cy.ts

@@ -1,4 +1,7 @@
 import type { Indicator } from '@/custom/ecospheres/model/indicator'
+import type { Dataservice, Reuse } from '@datagouv/components-next'
+import { dataserviceFactory } from 'cypress/support/factories/dataservices_factory'
+import { reuseFactory } from 'cypress/support/factories/reuses_factory'
 import { createIndicator, createIndicatorResource } from './support'
 
 describe('Indicator Detail View', () => {
@@ -56,25 +59,6 @@ describe('Indicator Detail View', () => {
     })
   })
 
-  describe('Fichiers et API Tab', () => {
-    beforeEach(() => {
-      cy.visit(`/indicators/${indicator.id}`)
-      cy.contains('Fichiers et API').click()
-    })
-
-    it('should display axis annee values', () => {
-      cy.contains('2020').should('be.visible')
-      cy.contains('2021').should('be.visible')
-      cy.contains('2022').should('be.visible')
-    })
-
-    it('should display axis secteur values', () => {
-      cy.contains('transport').should('be.visible')
-      cy.contains('energie').should('be.visible')
-      cy.contains('agriculture').should('be.visible')
-    })
-  })
-
   describe('Custom Metadata In Technical Details Tab', () => {
     beforeEach(() => {
       cy.visit(`/indicators/${indicator.id}`)
@@ -116,6 +100,51 @@ describe('Indicator Detail View', () => {
     })
   })
 
+  describe('Réutilisations et API Tab', () => {
+    it('should display dataservices when available', () => {
+      const dataservices = dataserviceFactory.many<Dataservice>(2)
+      const indicatorWithApis = createIndicator(
+        {},
+        { enable_visualization: false }
+      )
+      cy.mockDatasetAndRelatedObjects(indicatorWithApis, [], dataservices)
+      cy.visit(`/indicators/${indicatorWithApis.id}`)
+      cy.contains('Réutilisations et API').click()
+      cy.contains(`${dataservices.length} API`).should('be.visible')
+      cy.contains(dataservices[0].title).should('be.visible')
+      cy.contains(dataservices[1].title).should('be.visible')
+    })
+
+    it('should display empty state when no dataservices', () => {
+      cy.visit(`/indicators/${indicator.id}`)
+      cy.contains('Réutilisations et API').click()
+      cy.contains("Il n'y a pas encore d'API pour cet indicateur.").should(
+        'be.visible'
+      )
+    })
+
+    it('should display reuses when available', () => {
+      const reuses = reuseFactory.many<Reuse>(2)
+      const indicatorWithReuses = createIndicator(
+        {},
+        { enable_visualization: false }
+      )
+      cy.mockDatasetAndRelatedObjects(indicatorWithReuses, [], [], reuses)
+      cy.visit(`/indicators/${indicatorWithReuses.id}`)
+      cy.contains('Réutilisations et API').click()
+      cy.contains(reuses[0].title).should('be.visible')
+      cy.contains(reuses[1].title).should('be.visible')
+    })
+
+    it('should display empty state when no reuses', () => {
+      cy.visit(`/indicators/${indicator.id}`)
+      cy.contains('Réutilisations et API').click()
+      cy.contains(
+        "Il n'y a pas encore de réutilisation pour cet indicateur."
+      ).should('be.visible')
+    })
+  })
+
   describe('No Datavisualisation Configured', () => {
     it('should not display the previsualisation', () => {
       cy.visit(`/indicators/${indicator.id}`)