I’m wondering if it’s right that the group “Accounting Party” have the “Accounting” parent.
For me, the parent, it’s giving Accounting rights without any need.
I propose to remove the parent.
Initially the idea was to use the “Accounting” group but during the review process it was proposed to use a dedicated group for the payment terms defined by account_invoice.
Then on a second review, the payment terms on party were not considered as sensible enough to be restricted.
So the dedicated group has no longer any reason to exist but it was kept until the commit.
For me we should remove the group and use just “Accounting” group.