Skip to content

Conversation

WBoudabous
Copy link
Contributor

@WBoudabous WBoudabous commented Jun 17, 2025

Ajout du vocabulaire "directive communal" et d’un nouveau champ associé

Cette pull request introduit les modifications suivantes :

  • Ajout d’un nouveau vocabulaire nommé Directive communal.
    
  • Création d’un nouveau champ  utilisant ce vocabulaire.
    

@WBoudabous WBoudabous requested a review from mpeeters June 17, 2025 11:52
Copy link
Contributor

@duchenean duchenean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @WBoudabous

Ça m'a l'air OK. Peut-être demander à quelqu'un d'autre un second avis.

profile="Products.urban:default"
/>
<gs:upgradeStep
title="Add new vocabulary Municaipal Directive"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo ici

Copy link
Member

@mpeeters mpeeters left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase this branch and resolve conflicts

msgid "zip_folder_title"
msgstr "Zone d'initiative privilégiée"

msgid "chapter_1"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please run the script to generate po files, which avoids duplicates and indicates the file and line number.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please run black, multiple issues in this

from zope.component import getUtility

import logging
from zope.i18n import translate
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please run isort to ensure that imports are ordered correctly


municipal_directive_vocabulary_config = [

{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrong indent, please run black.

@mpeeters
Copy link
Member

In addition to my previous comments, please update your branch to resolve conflicts.

@WBoudabous WBoudabous requested a review from mpeeters August 22, 2025 12:18
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.

4 participants