Skip to content

implement typescript#41

Open
anselmpaul wants to merge 1 commit into
derrickpelletier:masterfrom
anselmpaul:master
Open

implement typescript#41
anselmpaul wants to merge 1 commit into
derrickpelletier:masterfrom
anselmpaul:master

Conversation

@anselmpaul

Copy link
Copy Markdown

tackling #39, here's my attempt to use this library with typescript.
I'm sure it's not perfectly typed since I'm not too familiar with the codebase but it works :)
Would love to hear your comments.

@Haseeb710 Haseeb710 mentioned this pull request Apr 17, 2020
@gorjan-mishevski

gorjan-mishevski commented Jun 9, 2020

Copy link
Copy Markdown

Why is this still not merged? He did a great job.

Comment thread src/LoadingOverlay.tsx Outdated
@tschoeki

tschoeki commented Feb 5, 2021

Copy link
Copy Markdown

Wow....it's been over a year since this PR was opened....please merge it :)

@ghost

ghost commented May 6, 2021

Copy link
Copy Markdown

So this will not be merged?

@tjosepo

tjosepo commented Jul 2, 2021

Copy link
Copy Markdown

I made a PR on the DefinitelyTyped repository for this library.

The types and they are now available on npm or by doing:

npm i --save-dev @types/react-loading-overlay

Feel free to submit your own PR on DefinitelyTyped and modify the type definitions if I made a mistake.

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.

5 participants