-
Notifications
You must be signed in to change notification settings - Fork 172
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
feat (Typescript): Add typescript support , work done by @Metavirulen with PR fixes. #70
Conversation
Hey @jafin, is there any work being done on this PR? |
Is there anything else for this PR to be added? It would be great to have typings. |
@huyvohcmc and @TienNguyenEH and @tvkhoa (adding you all as you have recently approved a PR) Can we merge this? We presently have to pin to @jafin 's branch to have TS support |
Are there any plans to merge this? Would be great... 🙏 |
package.json
Outdated
} | ||
} | ||
"popper.js": "^1.11.1", | ||
"@types/react": "*" |
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.
Is this should be in devDependencies
?
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.
@moshfeu yep, fixed. pushed to PR.
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.
I hope that @tvkhoa will ever merge it 🙏
Tested. It works. Thank you @jafin |
This is #24 redone with PR comments.