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.

Sign up for GitHub

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

Jump to bottom

[15.0][MIG] account_invoice_mode_monthly (renamed) #1245

Closed
wants to merge 14 commits into from
Closed

[15.0][MIG] account_invoice_mode_monthly (renamed) #1245

wants to merge 14 commits into from

Conversation

TDu
Copy link
Member

@TDu TDu commented Sep 26, 2022 •
edited
Loading

This module migration depends on #1243

@TDu TDu force-pushed the 15-mig-invoice-mode-monthly branch 4 times, most recently from 8d99abf to f4498e4 Compare September 26, 2022 11:55
TDu and others added 11 commits September 27, 2022 13:51
@TDu
[13.0]]ADD] Add acconut_invoicing_mode
c0b0178 
Add three modules helping on automatically invoicing customers.

The base module `account_invoice_base_invoicing_mode` does not actually
do anything but adds a selection field on partner to allow assigning an
invoicing mode to a customer. And a checkbox to choose regrouping
invoices.

The two other modules add specific invoicing mode.

The module `account_invoice_mode_monthly` creates monthly invoices for
customer on a specific day (configuration is in Accounting Settings)

The module `account_invoice_mode_at_shipping` creates invoices on the
shipping of the goods.

Those modules use queue_job to generate and validate invoices.
@TDu
[13.0][IMP] acconut_invoicing_mode group by terms
291a81d 
Sales order with different payment term are not grouped into the
same invoice anymore.
This grouping can be easily changed by overriding the coresponding
method.

During this change the parameter of the job generating the invoices
has changes and is now the sale_order_ids to process.
@TDu
[13.0][FIX] Add acconut_invoicing_mode
b49c1f0 
Add few changes from reviews.
Use sudo to post the invoice.
@TDu
[13.0] account-invoicing-mode add roadmap
d97264c
@phuctranfxvn @TDu
[IMP] account_invoice_mode: black, isort, prettier
4d92b3f
@phuctranfxvn @TDu
[MIG] account_invoice_mode: Migration to 14.0
fe2904c
@oca-travis @TDu
[UPD] Update account_invoice_mode_monthly.pot
d2a10b0
@OCA-git-bot @TDu
[UPD] README.rst
2ac2107
@SilvioC2C @TDu
[IMP] account_invoice_mode_*: Add ondelete attribute to 'invoicing_mo…
5cd1c67 
…de' fields
@OCA-git-bot @TDu
account_invoice_mode_monthly 14.0.1.1.0
d4d3361
@BertVGroenendael @TDu
[IMP] account_invoice_mode_monthly: black, isort, prettier
0ad0f54
@TDu TDu force-pushed the 15-mig-invoice-mode-monthly branch from f4498e4 to 1a9d568 Compare September 27, 2022 11:51
@StefanRijnhart StefanRijnhart mentioned this pull request Nov 30, 2022
Merged
@rafaelbn
Copy link
Member

rafaelbn commented Dec 9, 2022

/ocabot migration account_invoice_mode_monthly

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Dec 9, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 9, 2022
Open
56 tasks
@OCA-git-bot
Copy link
Contributor

The migration issue (#1022) has been updated to reference the current pull request.
however, a previous pull request was referenced : #1062.
Perhaps you should check that there is no duplicate work.
CC : @rven

@StefanRijnhart
Copy link
Member

Please remove the migration here as well.

@TDu TDu force-pushed the 15-mig-invoice-mode-monthly branch from e45f3f6 to 74157e6 Compare December 13, 2022 07:11
@TDu
Copy link
Member Author

TDu commented Dec 13, 2022

@StefanRijnhart Done

StefanRijnhart
StefanRijnhart approved these changes Dec 13, 2022
View reviewed changes
Copy link
Member

@StefanRijnhart StefanRijnhart left a comment

Choose a reason for hiding this comment

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

Thanks! Any chance you could review the 16.0 version of these modules? #1295

odooNextev pushed a commit to odooNextev/account-invoicing that referenced this pull request Sep 29, 2023
@OCA-git-bot
Merge PR OCA#1245 into 14.0
6132112 
Signed-off-by kittiu
@github-actions GitHub Actions
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 May 12, 2024
@dreispt
Copy link
Member

dreispt commented May 18, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 15.0-ocabot-merge-pr-1245-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request May 18, 2024
@OCA-git-bot
Merge PR #1245 into 15.0
ba91c67 
Signed-off-by dreispt
@OCA-git-bot
Copy link
Contributor

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-1245-by-dreispt-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.

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 19, 2024
@github-actions GitHub Actions
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 Sep 22, 2024
@github-actions github-actions bot closed this Oct 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@i-vyshnevska i-vyshnevska i-vyshnevska left review comments

@StefanRijnhart StefanRijnhart StefanRijnhart approved these changes

Assignees
No one assigned
Labels
needs review stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Milestone
15.0
Development

Successfully merging this pull request may close these issues.