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

[16.0][MIG] l10n_fr account invoice import modules #430

Open
wants to merge 41 commits into
base: 16.0
Choose a base branch
from

Conversation

alexis-via
Copy link
Contributor

No description provided.

alexis-via and others added 30 commits March 25, 2023 00:01
Adapt the code of partner matching for SIREN to the recent changes in the modules of the account-invoicing project
Fix module l10n_fr_business_document_import
Improve module l10n_fr_business_document_import
PEP8 fix in l10n_fr_chorus_account
…integer

When partner_dict comes from invoice2data, siren may be an integer: we convert it to a string.
Improve module l10n_fr_business_document_import
PEP8 fix in l10n_fr_chorus_account
The module has been renamed from l10n_fr_account_invoice_import_factur-x
to l10n_fr_account_invoice_import_facturx
Add readme subdirectory
Unfuck the "Donald Trump" pull request: this is the real/serious work made
by the anti-Donald Trump squad.
Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

Diff review / no test : lgtm.

Please take a look on red ci + red runboat.

Thanks !

Copy link

@njeudy njeudy left a comment

Choose a reason for hiding this comment

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

Ok I test a try to review code on V16 community and V16 enterprise :) Ok for me

@legalsylvain
Copy link
Contributor

/ocabot migration l10n_fr_account_invoice_import_facturx
/ocabot migration l10n_fr_account_invoice_import_simple_pdf
/ocabot migration l10n_fr_business_document_import

works for many modules ?

@OCA-git-bot OCA-git-bot modified the milestone: 16.0 Jun 9, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Jun 9, 2023
29 tasks
@legalsylvain
Copy link
Contributor

/ocabot migration l10n_fr_account_invoice_import_facturx

@github-actions
Copy link

github-actions bot commented Oct 8, 2023

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 8, 2023
@legalsylvain
Copy link
Contributor

@alexis-via CI is red. could you take a look ? thanks.

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 15, 2023
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 18, 2024
@github-actions github-actions bot closed this Mar 24, 2024
@alexis-via alexis-via reopened this Aug 23, 2024
@alexis-via alexis-via added no stale Use this label to prevent the automated stale action from closing this PR/Issue. and removed stale PR/Issue without recent activity, it'll be soon closed automatically. labels Aug 23, 2024
Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

CI still red. Could you take a look ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no stale Use this label to prevent the automated stale action from closing this PR/Issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants