Skip to content

Fixed type definition for the toPDF method#127

Open
franciscofsales wants to merge 1 commit intoivmarcos:mainfrom
franciscofsales:fix/topdf-type-definition
Open

Fixed type definition for the toPDF method#127
franciscofsales wants to merge 1 commit intoivmarcos:mainfrom
franciscofsales:fix/topdf-type-definition

Conversation

@franciscofsales
Copy link

The toPDF method in reality exports a promise from the jsPDF lib.
The typing was misleading for whom-ever wants to set up a loader or some UI feedback while the process is going.

Fixed the type definition for toPDF method.

@eirikur-nc
Copy link

I've been burned by this in the past, as VS code will currently flag the await as redundant in await toPDF(...). I would be nice to get this in to prevent others from hurting themselves on this sharp corner.

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