Skip to content

[16.0][IMP]mis_builder_analytic:add journal Item to mis.account.analytic.…#64

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
acsone:16.0-63-add_journal_item-bjo
Apr 15, 2026
Merged

[16.0][IMP]mis_builder_analytic:add journal Item to mis.account.analytic.…#64
OCA-git-bot merged 1 commit into
OCA:16.0from
acsone:16.0-63-add_journal_item-bjo

Conversation

@bjouini-acsone
Copy link
Copy Markdown

@bjouini-acsone bjouini-acsone commented Apr 8, 2026

…line

this commit adds the link from mis.account.analytic.line to account.move.line by adding the field move_line_id that it's value will be retrieved by the init method

closes #63

@bjouini-acsone bjouini-acsone force-pushed the 16.0-63-add_journal_item-bjo branch from 54ee842 to 45536a9 Compare April 8, 2026 11:07
@bjouini-acsone bjouini-acsone changed the title [IMP]mis_builder_analytic:add journale Ittem to mis.account.analytic.… [16.0][IMP]mis_builder_analytic:add journale Ittem to mis.account.analytic.… Apr 8, 2026
@bjouini-acsone bjouini-acsone changed the title [16.0][IMP]mis_builder_analytic:add journale Ittem to mis.account.analytic.… [16.0][IMP]mis_builder_analytic:add journal Item to mis.account.analytic.… Apr 8, 2026
Copy link
Copy Markdown
Member

@sbidoul sbidoul left a comment

Choose a reason for hiding this comment

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

Thanks. A couple of ergonomy improvements, otherwise it looks good and works correctly (tested on runboat).

Can you also rebase to get the pre-commit fixes from #65?

<group>
<group>
<field name="analytic_line_id" />
<field name="move_line_id" />
Copy link
Copy Markdown
Member

@sbidoul sbidoul Apr 8, 2026

Choose a reason for hiding this comment

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

Suggested change
<field name="move_line_id" />
<field name="move_line_id" widget="line_open_move_widget" />

So the move can be opened easily like in the Analytic Items list view.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also, can you put it on the right of analytic_account_id so it is more similar to the Analytic Items view?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@sbidoul you can check

@bjouini-acsone bjouini-acsone force-pushed the 16.0-63-add_journal_item-bjo branch 5 times, most recently from 38f4839 to c75e915 Compare April 8, 2026 14:39
…line

this commit adds the link from mis.account.analytic.line  to account.move.line
by adding the field move_line_id that it's value will be retrieved by the init method
@sbidoul sbidoul force-pushed the 16.0-63-add_journal_item-bjo branch from c75e915 to 07b683a Compare April 8, 2026 15:18
Copy link
Copy Markdown
Member

@sbidoul sbidoul left a comment

Choose a reason for hiding this comment

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

Thanks!

The drilldown now looks like this and clicking on the Journal Item brings you directly to the invoice or journal entry.

Image

@vdewulf
Copy link
Copy Markdown

vdewulf commented Apr 8, 2026

@sbidoul @bjouini-acsone
Hello,
Thanks for this development.
I tested successfully with the runboat.
In the Test Mis Report which is configured to take the data from the "MIS Account Analytic Line", I can click on the amount and see the details of the analytic accounts, with a new column showing the accounting item, and which is clicable (we can see the invoice behind the line).
image
That's great and what was expected.
Thanks

@sbidoul
Copy link
Copy Markdown
Member

sbidoul commented Apr 13, 2026

Thanks for the test Virginie. If there are no other comments I'll merge this later this week.

@sbidoul
Copy link
Copy Markdown
Member

sbidoul commented Apr 15, 2026

/ocabot merge minor

@OCA-git-bot
Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-64-by-sbidoul-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 3645e6b into OCA:16.0 Apr 15, 2026
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

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.

4 participants