Skip to content

Limit the upcoming audit email to 30 records, added optional --with-output#18109

Merged
snipe merged 2 commits intodevelopfrom
audit-improvements
Oct 28, 2025
Merged

Limit the upcoming audit email to 30 records, added optional --with-output#18109
snipe merged 2 commits intodevelopfrom
audit-improvements

Conversation

@snipe
Copy link
Copy Markdown
Member

@snipe snipe commented Oct 28, 2025

This modifies the audit notification to limit it to 50 assets in the email, since organizations with a lot of items to audit could easily exceed the email size limit on some servers. This also adds an --with-output option, which will give you the full table output.

Screenshot 2025-10-28 at 2 21 12 PM Screenshot 2025-10-28 at 2 21 25 PM Screenshot 2025-10-28 at 2 28 54 PM

@snipe snipe merged commit 60a7b7f into develop Oct 28, 2025
7 of 8 checks passed
@snipe snipe deleted the audit-improvements branch October 28, 2025 14:32
@yusuf-basith
Copy link
Copy Markdown

Hello @snipe,
As you switched back to Markdown, the issue reported in #17273 is repeating again and the email body is messed up, when we have a line break in the Asset Notes field.
image

Can you take this at the next possible sprint?

Thanks & Regards,
Yusuf

@snipe
Copy link
Copy Markdown
Member Author

snipe commented Nov 14, 2025

Hm, I'm not able to reproduce this:

Screenshot 2025-11-14 at 1 56 20 PM

@yusuf-basith
Copy link
Copy Markdown

Hello Snipe,

Could you please try with asset notes as given below:

Lorem ipsum dolor sit amet, consectetur adipiscing elit, [Line Break]
sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. [Line Break]
[Empty Line]
Ut enim ad minim veniam, [Line Break]
quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat.  [Line Break]

The [Line Break]s are handled perfectly.
However the [Empty Line] is breaking the formatting.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants