-
Notifications
You must be signed in to change notification settings - Fork 0
Urb-3342 #380
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: urban2.7.x
Are you sure you want to change the base?
Urb-3342 #380
Conversation
This change is still in progress This reverts commit 49a305e.
…for SPW cyberattack
There was a problem hiding this 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" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo ici
There was a problem hiding this 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" |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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 = [ | ||
|
||
{ |
There was a problem hiding this comment.
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.
In addition to my previous comments, please update your branch to resolve conflicts. |
Ajout du vocabulaire "directive communal" et d’un nouveau champ associé
Cette pull request introduit les modifications suivantes :