Skip to content

Refonte - suppression classe comptabilite#2200

Open
vgreb wants to merge 2 commits intoafup:masterfrom
vgreb:refacto/class-comptabilite
Open

Refonte - suppression classe comptabilite#2200
vgreb wants to merge 2 commits intoafup:masterfrom
vgreb:refacto/class-comptabilite

Conversation

@vgreb
Copy link
Copy Markdown
Collaborator

@vgreb vgreb commented Apr 23, 2026

A merger après #2210

@vgreb vgreb self-assigned this Apr 23, 2026
@vgreb vgreb force-pushed the refacto/class-comptabilite branch 2 times, most recently from 45af07b to be5ba53 Compare April 23, 2026 21:21
@vgreb vgreb force-pushed the refacto/class-comptabilite branch 5 times, most recently from dda8e85 to 8247776 Compare May 10, 2026 14:36
@vgreb vgreb marked this pull request as ready for review May 10, 2026 14:40
@vgreb
Copy link
Copy Markdown
Collaborator Author

vgreb commented May 10, 2026

resolves #2228

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Il y a encore des références à cette classe dans ces fichiers :

  • htdocs/pages/administration/compta_facture.php
  • app/config/services.yml

C'est normale ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Oui, j'ai pris de l'avance sur la suppression de la compta_facture.php. J'ai rajouté une mention pour ne pas merge avec la MR #2210

Comment thread sources/AppBundle/Compta/CsvExtractor.php Outdated
Comment thread sources/AppBundle/Compta/CsvExtractor.php Outdated
@vgreb vgreb force-pushed the refacto/class-comptabilite branch from 8247776 to fa949b6 Compare May 10, 2026 17:11
@vgreb vgreb force-pushed the refacto/class-comptabilite branch from fa949b6 to 900ac94 Compare May 10, 2026 20:44
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