[16.0][FIX] remove nested loop in tax reports#135
Conversation
remove the nested loop in the donation tax report template to ensure that each record is present once in the generated report instead of the number of records squared when generating a report for multiple records.
|
Hi @alexis-via, |
|
@OCA/donation-maintainers would someone please merge this? |
|
@OCA/donation-maintainers 👋 This PR looks ready (2 approvals + approved label), but it’s blocked because the required runboat/build check never appeared under “Checks” on GitHub. The PR shows “Waiting for status to be reported” indefinitely. It seems Runboat didn’t create a build for this PR (no runboat/build line under checks, only the branch build is visible). Could you please retrigger the Runboat build or adjust the required check so this can be merged? Thanks 🙏 |
|
@mkanderson afaik, the runboat build it not required for the pr to be mergeable. all that is needed is that a maintainer or psc member triggers the merge. |
|
/ocabot merge minor |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 91a7515. Thanks a lot for contributing to OCA. ❤️ |
|
Thanks @alexis-via |
6 participants
remove the nested loop in the donation tax report template to ensure that each record is present once in the generated report instead of the number of records squared when generating a report for multiple records.