Skip to content

Add Email Reports - #135

Open
Lightning11wins wants to merge 137 commits into
masterfrom
html-reports2
Open

Add Email Reports#135
Lightning11wins wants to merge 137 commits into
masterfrom
html-reports2

Conversation

@Lightning11wins

@Lightning11wins Lightning11wins commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

This PR makes HTML reports email-ready and also greatly reduces the size of HTML reports in general. Reports have two new flags that independently toggle email and pagination (you can even have both if you want a paginated email for some reason).

GitHub Relationships

These will need to be reviewed and merged into master before this PR is reviewed.

Greptile Summary

Dev - Kardia and others added 30 commits June 23, 2026 15:58
- Removed soft line wraps
- Removed replacement of all spaces in html text with &nbsp
- Removed excess font/style markings
- Modified to preserve only leading spaces as &nbsp
- Fixed justification of text
- Removed a <tr> element that seemed unnecessary
Finished support for table borders
Made sure that XSET elements were in new <td>s as appropriate
- fixed image align
- fixed unspecified size assignment
- added specified padding between elements
Not sure if this is the permanent solution
SVG now embedded in base64 for web
fixed &nbsp; missing semicolons
multipart/mixed untested
whoops (1 case of using wrong flag, a few cases of
using wrong operator for bitwise NOT)
Add typedefs to make flag ints explicit.
Add explicit NO_FLAGS values.
Replace hard-coded max values with compiler-computed values.
Fix prt_htmlfm_SetKeepSpaces() taking a parameter with no specified type.
Fix prt_htmlfm_WriteStyle() not returning 0 on success.
Clean up unused variables.
Move _PSFI to .h file.
Move PrtHTMLfmSubtype to .h file.
Clean up comment formatting.
Comment thread centrallix/report/prtmgmt_v3_fm_html.c
Comment thread centrallix/report/prtmgmt_v3_fm_html.c Outdated
Comment thread centrallix/report/prtmgmt_v3_fm_html.c Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request AI review for PRs. size: medium Might be hard to review, usually less than ~5000 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant