-
Notifications
You must be signed in to change notification settings - Fork 391
chore: [UIE-9926] - Bump jspdf to 4.0.0
#13248
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
base: develop
Are you sure you want to change the base?
Conversation
jspdf to 4.0.0jspdf to 4.0.0
| "@types/eslint-plugin-jsx-a11y": "^6.10.0", | ||
| "@types/he": "^1.1.0", | ||
| "@types/history": "4", | ||
| "@types/jspdf": "^1.3.3", |
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.
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.
Great! I hate it when packages don't provide their own type-definitions!
| reader.onloadend = () => resolve(reader.result as string); | ||
| reader.onerror = reject; | ||
| reader.readAsDataURL(response.data); | ||
| }); |
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.
tests were failing - this method is valid for both node (tests) and the browser
bnussman-akamai
left a comment
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.
Thanks for taking care of this! PDF generation looks good from my testing. Left one comment about the window.location patch
| "@types/eslint-plugin-jsx-a11y": "^6.10.0", | ||
| "@types/he": "^1.1.0", | ||
| "@types/history": "4", | ||
| "@types/jspdf": "^1.3.3", |
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.
Great! I hate it when packages don't provide their own type-definitions!
Cloud Manager UI test results🔺 1 failing test on test run #6 ↗︎
Details
TroubleshootingUse this command to re-run the failing tests: pnpm cy:run -s "cypress/e2e/core/linodes/clone-linode.spec.ts" |
|||||||||||||||||
Opened my own PR in favor of #13241
This update resulted in some type pollution which were tricky to resolve (mostly in test files).
This should be tested properly!
Changes
Bumps jspdf from 3.0.2 to 4.0.0.
Release notes
Sourced from jspdf's releases.
Commits
e6cf03d4.0.0a688c8frestrict file system access in node build (#3931)a504e973.0.4de802abFix Incorrect Typing for Margins in the TableConfig Interface Definition (#3816)87162d1chore: bump checkout, setup-node, and stale actions (#3907)e7dc622Fix: Context2d font regex too restrictive ( #3904 ) (#3906)e080935Do not add pages dynamically unless autoPaging is enabled (#3915)c768910add package.json exports field (#3903)c10d90cFix API.internal.pages not being updated when restoring a RenderTarget ( #389...2db3d9dfix font list cache invalidation issue in context2d module (#3891)What to test