[18.0][FIX] agreement_legal: remove redundant t-name from report_agreement_document

[BhaveshHeliconia]

Issue:
The report_agreement_document template contained a redundant <t t-name="agreement.report_agreement_document"> wrapper inside the <template> tag. In modern Odoo versions, this triggers the following warning in the logs:
Unknown directives or unused attributes: {'t-name'} in agreement_legal.report_agreement_document

Reference :- OCA/sign#118

[OCA-git-bot]

Hi @max3903, @ygol,
some modules you are maintaining are being modified, check this out!

[BhaveshHeliconia]

ping @lbarry-apsl @peluko00

[lbarry-apsl]

Code and runboat reviewed. LGTM

[peluko00]

LGTM!

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[BhaveshHeliconia]

@pedrobaeza can we merge this?

[pedrobaeza]

Why the diff is so big?

[BhaveshHeliconia]

Why the diff is so big?

The diff is large because of the pre-commit formatting changes. Some lines were reformatted. which increased the overall diff.

[pedrobaeza]

That doesn't seem the problem. You have reduced an indentation level removing <t t-name="agreement.report_agreement_document">. Is that provoking any problem?

[BhaveshHeliconia]

That doesn't seem the problem. You have reduced an indentation level removing <t t-name="agreement.report_agreement_document">. Is that provoking any problem?

The indentation change is due to removing . After removing that tag, I ran the pre-commit hooks, which only reformatted the file accordingly. No other changes were introduced apart from that.

[BhaveshHeliconia]

ping @pedrobaeza

[pedrobaeza]

/ocabot merge patch

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-109-by-pedrobaeza-bump-patch, awaiting test results.

[OCA-git-bot]

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