Skip to content

Blocs natifs#3977

Draft
arnaudlevy wants to merge 10 commits into
mainfrom
native-blocks
Draft

Blocs natifs#3977
arnaudlevy wants to merge 10 commits into
mainfrom
native-blocks

Conversation

@arnaudlevy
Copy link
Copy Markdown
Member

@arnaudlevy arnaudlevy commented May 9, 2026

Type

  • Nouvelle fonctionnalité
  • Bug
  • Ajustement
  • Rangement

Description

Sur l'idée de Yuna, on abandonne #2445 et on crée des blocs natifs qui listent le contenu des pages d'index.

Ces blocs sont donc :

  1. dépubliables
  2. réordonnables
  3. paramétrables en layout et options

Ces 3 propriétés donnent tout le contrôle éditorial attendu.

Prototype (avec les projets)

  • Propriété native
  • Création des blocs natifs à la volée
  • Interdiction de suppression, duplication, copie
  • Gestion spéciale de l'édition des blocs (suppression des choix d'objets)
  • Gestion de la pagination pour les gros volumes
  • Gestion des liens cliquables vers les objets
  • Gestion du statique (avec ou sans objets ?)

Pour le statique, est-ce qu'on préfère l'option avec toute la liste ou sans liste ?
A. (explicite, long)

  - template: projects
    native: true
    ranks:
      base: 2
      children: 2
    top:
      active: false
    data:
      layout: grid
      options:
        categories: true
        image: true
        subtitle: true
        summary: false
        year: true
      projects:
        - permalink: "/projets/2023-bonnes-notes/"
          path: "/projects/2023/bonnes-notes"
          slug: "bonnes-notes"
          file: "content/fr/projects/2023/bonnes-notes.html"
          name: >-
            Bonnes notes
        - permalink: "/projets/2023-nig/"
          path: "/projects/2023/nig"
          slug: "nig"
          file: "content/fr/projects/2023/nig.html"
          name: >-
            NIG
        - permalink: "/projets/2023-ran-coper/"
          path: "/projects/2023/ran-coper"
          slug: "ran-coper"
          file: "content/fr/projects/2023/ran-coper.html"
          name: >-
            Ran Coper
      quantity: 3

B. (implicite, court)

  - template: projects
    native: true
    ranks:
      base: 2
      children: 2
    top:
      active: false
    data:
      layout: grid
      options:
        categories: true
        image: true
        subtitle: true
        summary: false
        year: true

Types de pages

  • Home (avec les pages non listées, type mentions légales)
  • Pages normales (avec des enfants)
  • Actualités
  • Projets
  • Agenda
  • Expositions
  • Archives d'expositions
  • Offres d'emploi
  • Organisations
  • Personnes

Migration

  • Création des blocs natifs manquants (legacy)
  • Récupérer les options de config yaml des sites

Niveau d'incidence

  • Incidence faible 😌
  • Incidence moyenne 😲
  • Incidence forte 😱

@qltysh
Copy link
Copy Markdown

qltysh Bot commented May 9, 2026

Qlty


Coverage Impact

⬆️ Merging this pull request will increase total coverage on main by 0.10%.

Modified Files with Diff Coverage (6)

RatingFile% DiffUncovered Line #s
Coverage rating: F Coverage rating: F
app/models/communication/block/template/project.rb25.0%33, 76-80
Coverage rating: A Coverage rating: A
app/models/communication/website/page/localization.rb100.0%
Coverage rating: C Coverage rating: C
app/models/communication/block.rb100.0%
Coverage rating: A Coverage rating: A
app/models/communication/website/page.rb100.0%
Coverage rating: F Coverage rating: F
app/models/communication/website/page/communication_portfolio.rb50.0%8
New Coverage rating: A
...s/communication/website/page/localization/with_native_block.rb92.9%13
Total80.0%
🤖 Increase coverage with AI coding...
In the `native-blocks` branch, add test coverage for this new code:

- `app/models/communication/block/template/project.rb` -- Lines 33 and 76-80
- `app/models/communication/website/page/communication_portfolio.rb` -- Line 8
- `app/models/communication/website/page/localization/with_native_block.rb` -- Line 13

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@alexisben
Copy link
Copy Markdown
Contributor

Réponse B

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants