-
Notifications
You must be signed in to change notification settings - Fork 234
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#1588 - Download Report Functionality using existing components #1743
Closed
Closed
Changes from 12 commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
a51c873
added component Drop Down Button
DMJain 063c446
report download function
DMJain 19b36d6
new hook for getting report data
DMJain d05a9ca
using function from timeUtlis and new time util fucntion
DMJain e187fb1
moved report Util function to new file
DMJain 107cc5c
added proptypes for new files
DMJain a27b64e
minor UI change for download button
DMJain 68e6914
Merge branch '1588/report_download' into develop
DMJain bc65054
Merge branch 'bluewave-labs:develop' into develop
DMJain a321afe
used existing component
DMJain 59c0500
Merge branch '1588/report_download' into develop
DMJain babb1c1
1st PR suggestion
DMJain 9a19d4c
Merge branch 'bluewave-labs:develop' into develop
DMJain d695e17
all changes done as per suggested in PR
DMJain 6d89ac9
Merge branch '1588/report_download' into develop
DMJain File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
Yo, let's verify this package version, eh?
The package version uses a caret (^), which could lead to compatibility issues down the road. Let's make sure we're using the most stable version and check for any security concerns.
🌐 Web query:
💡 Result:
The latest stable version of html2pdf.js (the JavaScript library) is v0.9.3, as newer versions (v0.10.x) have reported unresolved issues[3][4]. However, npm lists v0.10.2 as technically the most recent release[3][10].
Key Security Notes:
JS Library Vulnerabilities:
html2pdf.js
package in the provided sources[6].PHP Library Confusion:
spipu/html2pdf
) had critical vulnerabilities:Recommendations:
spipu/html2pdf
is updated to ≥v5.2.8[5][7][8].Citations:
Yo, heads up on the html2pdf.js dependency!