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

[ADD] marked account_balance_line as merged in Odoo core. (account) #3760

Merged

Conversation

legalsylvain
Copy link
Contributor

Ref : OCA/account-financial-tools#1472

[X] account_balance_line Included in core

@MiquelRForgeFlow MiquelRForgeFlow added this to the 16.0 milestone Mar 14, 2023
@pedrobaeza pedrobaeza merged commit de6543d into 16.0 Mar 14, 2023
@OCA-git-bot OCA-git-bot deleted the 16.0-legalsylvain-account_balance_line--merged-account branch March 14, 2023 20:03
@pedrobaeza
Copy link
Member

No need to wait in this one.

@hbrunn
Copy link
Member

hbrunn commented Mar 15, 2023

@legalsylvain @pedrobaeza but this should then also be added to the merged_modules dict, this is the renamed_modules dict which breaks the migration.

good reason to merge #3680? then we'll have a working CI...

@pedrobaeza
Copy link
Member

Ouch, I trusted too much in Sylvain, hehe

I have fixed it in a02bcdd

I'm reviewing the base PR now for merging it.

@legalsylvain
Copy link
Contributor Author

Hi @hbrunn I didn't pay attention. Sorry for the inconvenience and the broken tests.
have a nice day.

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

Successfully merging this pull request may close these issues.

5 participants