search

OCA / l10n-france

France Localization for Odoo
GNU Affero General Public License v3.0
42 stars 114 forks source link

[17.0][MIG] Chorus modules #527

Closed alexis-via closed 5 months ago

alexis-via commented 6 months ago

Migrate l10n_fr_chorus_account, l10n_fr_chorus_sale and l10n_fr_chorus_facturx from v16 to v17

Several methods are now private methods Move code to send invoices to chorus from wizard to account.move name_get converted to _compute_display_name() read_group() converted to _read_group() name_search() converted to _name_search() protect login and password of res.company with groups="base.group_system"

alexis-via commented 5 months ago

/ocabot merge nobump

OCA-git-bot commented 5 months ago

This PR looks fantastic, let's merge it! Prepared branch 17.0-ocabot-merge-pr-527-by-alexis-via-bump-nobump, awaiting test results.

OCA-git-bot commented 5 months ago

@alexis-via your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-527-by-alexis-via-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

OCA-git-bot commented 5 months ago

@alexis-via your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-527-by-alexis-via-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

alexis-via commented 5 months ago

/ocabot merge nobump

OCA-git-bot commented 5 months ago

On my way to merge this fine PR! Prepared branch 17.0-ocabot-merge-pr-527-by-alexis-via-bump-nobump, awaiting test results.

OCA-git-bot commented 5 months ago

Congratulations, your PR was merged at 39b607c2b230f7259dbb2ebb89430471d3df5da0. Thanks a lot for contributing to OCA. ❤️