Skip to content

Conversation

@InformatiqueProg
Copy link

Add department on leads,

This PR is the continuation of our discussion on #699

Copy link
Contributor

Choose a reason for hiding this comment

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

Done with LLM ? ;-)

Copy link
Author

Choose a reason for hiding this comment

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

@legalsylvain
Copy link
Contributor

legalsylvain commented Nov 27, 2025

Thanks for sharing. I put the PR In draft, waiting for the merge of #701

So, we can then remove all the duplicated code.

In the meantime, could you take a look on the pre-commit error ?

thanks !

@legalsylvain legalsylvain marked this pull request as draft November 27, 2025 09:04
@InformatiqueProg InformatiqueProg force-pushed the 18-add-l10n_fr_department_crm branch from 5360d5b to 3afccf7 Compare November 27, 2025 09:55
@InformatiqueProg
Copy link
Author

InformatiqueProg commented Nov 27, 2025

Sorry @legalsylvain , I must have misunderstood.
The Readme.rst if not create automatically with the files in readme folder?

@legalsylvain
Copy link
Contributor

The Readme.rst if not create automatically with the files in readme folder?

run

pre-commit install
pre-commit run -a

it will fix all the errors, and show the remaining errors.

also when commiting, there is a check.

@InformatiqueProg InformatiqueProg deleted the 18-add-l10n_fr_department_crm branch January 7, 2026 11:12
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