Removed incorrectly implemented and non-functional print asset functionality.

ydahal1 commented 2 weeks ago

Description

Removed incorrectly implemented and non-functional print asset functionality. The original goal was to print asset-related data. This incorreclty implemented functionality attempts to generate the file directly from the DOM, which is not the correct method. The proper implementation would involve generating a PDF with asset details on the server and sending it to the client via res.sendFile

Fixes # (issue)

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Vulnerability fix (package bumps or CodeQL adjustments to ensure code security)
[ ] This change requires a documentation update

Developer Checklist:

[x] I have commented my code, particularly in hard-to-understand areas
[x] I have resolved any conflicts with the branch I am attempting to merge to.
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings
[x] I have ensured that my code does not unecessarily duplicate existing cod
[x] I have ensured that all security checks have been passed
[x] All input boxes have sensible character limits applied
[x] Refreshing related pages puts page in a workable and sensible state

Reviewer Checklist

[ ] I have pulled the branch into my local environemtn and started the project succesfully
[ ] I have reviewed the code for proper comments and mispellings
[ ] All input boxes have sensible character limits applied
[ ] Refreshing related pages puts page in a workable and sensible state
[ ] Submitting any relevant Forms relays proper messaging to user
[ ] I have checked that all security checks have been passed
[ ] I have checked that all backend routes have proper validation

ydahal1 commented 2 weeks ago

Removed incorrectly implemented and non-functional print asset functionality. The original goal was to print asset-related data. This incorreclty implemented functionality attempts to generate the file directly from the DOM, which is not the correct method. The proper implementation would involve generating a PDF with asset details on the server and sending it to the client via res.sendFile

FancMa01 commented 2 weeks ago

Merge Conflict now, please fix

hpcc-systems / Tombolo