Skip to content

Retire les champs adresse du formulaire Client#435

Merged
florimondmanca merged 1 commit into
masterfrom
fix/drop-customer-address-fields
Jun 14, 2024
Merged

Retire les champs adresse du formulaire Client#435
florimondmanca merged 1 commit into
masterfrom
fix/drop-customer-address-fields

Conversation

@florimondmanca
Copy link
Copy Markdown
Contributor

Je ne sais pas pourquoi ils étaient encore là, l'adresse des clients a été retirée dans #416, il n'y a plus rien concernant l'adresse en base ou dans Customer.entity

Sûrement un oubli de #415

@codecov
Copy link
Copy Markdown

codecov Bot commented May 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.00%. Comparing base (71cc6cb) to head (ec7198a).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #435   +/-   ##
=======================================
  Coverage   91.00%   91.00%           
=======================================
  Files         184      184           
  Lines        2123     2123           
  Branches      154      154           
=======================================
  Hits         1932     1932           
  Misses        185      185           
  Partials        6        6           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@florimondmanca florimondmanca requested a review from ip512 May 31, 2024 14:35
@florimondmanca florimondmanca requested a review from fuuuzz June 14, 2024 08:47
Copy link
Copy Markdown

@fuuuzz fuuuzz left a comment

Choose a reason for hiding this comment

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

👌

@florimondmanca florimondmanca force-pushed the fix/drop-customer-address-fields branch from 7e787a3 to ec7198a Compare June 14, 2024 09:18
@florimondmanca florimondmanca merged commit 1b1d3ee into master Jun 14, 2024
@florimondmanca florimondmanca deleted the fix/drop-customer-address-fields branch June 14, 2024 09:22
@florimondmanca
Copy link
Copy Markdown
Contributor Author

Thx d'avoir checké @benpaquier 🙏

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants