Skip to content
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

amelioration(dossier.pdf): pour les champs de type Champs::AddressChamp, ajoute le code insee de la commune sous l'adresse complete #9624

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

mfo
Copy link
Contributor

@mfo mfo commented Oct 20, 2023

@mfo mfo force-pushed the US/dossier.export.pdf branch from 7d169a8 to e2beb93 Compare October 20, 2023 09:58
when 'Champs::AddressChamp'
value = champ.blank? ? 'Non communiqué' : champ.to_s
format_in_2_lines(pdf, tdc.libelle, value)
format_in_2_lines(pdf, "Code INSEE :", champ.commune&.fetch(:code)) if champ.full_address?
Copy link
Member

Choose a reason for hiding this comment

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

Tu pourrais aussi en profiter pour ajouter le nom et le code du département ici, ainsi qu'au champ commune ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fait

@mfo mfo force-pushed the US/dossier.export.pdf branch from e2beb93 to 0520fb4 Compare October 24, 2023 07:36
@mfo mfo requested a review from tchak October 24, 2023 09:09
…mp, ajoute le code insee de la commune sous l'adresse complete, ajoute le code postal, ajoute le departemenent et son code. Pour les champs de type Champs::CommuneChamp ajoute le code postal de la commune
@mfo mfo force-pushed the US/dossier.export.pdf branch from 0520fb4 to 2295d64 Compare October 25, 2023 07:22
@mfo mfo added this pull request to the merge queue Oct 26, 2023
Merged via the queue into demarches-simplifiees:main with commit 9866acc Oct 26, 2023
15 checks passed
@mfo mfo deleted the US/dossier.export.pdf branch October 26, 2023 06:46
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