Skip to content
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

Rewrite in typescript #63

Merged
merged 12 commits into from
Feb 26, 2022
Merged

Rewrite in typescript #63

merged 12 commits into from
Feb 26, 2022

Conversation

imranbarbhuiya
Copy link
Collaborator

@imranbarbhuiya imranbarbhuiya commented Nov 7, 2021

I'm rewriting this package in typescript.

You may find error while trying to test using npm link
check https://reactjs.org/warnings/invalid-hook-call-warning.html#duplicate-react to fix it, but the code is correct.

@imranbarbhuiya imranbarbhuiya requested a review from apal21 November 7, 2021 15:35
@imranbarbhuiya
Copy link
Collaborator Author

I've tested the changes and everything seems working fine again.

I've updated version to 0.0.14 also updated changelog.

package.json Show resolved Hide resolved
@imranbarbhuiya imranbarbhuiya requested a review from apal21 November 7, 2021 16:46
@imranbarbhuiya imranbarbhuiya self-assigned this Nov 20, 2021
@imranbarbhuiya imranbarbhuiya added the enhancement New feature or request label Nov 20, 2021
@imranbarbhuiya imranbarbhuiya changed the title Added nonce support Feat: added new props which add nonce to the style component. Nov 20, 2021
@imranbarbhuiya imranbarbhuiya changed the title Feat: added new props which add nonce to the style component. WIP: Rewrite in typescript Dec 20, 2021
@imranbarbhuiya imranbarbhuiya marked this pull request as draft December 20, 2021 14:58
- rewritten in typescript
- used react functional component
added github tests for lint and build
updated npmignore file
added esm export
@imranbarbhuiya imranbarbhuiya marked this pull request as ready for review December 21, 2021 15:51
@imranbarbhuiya
Copy link
Collaborator Author

@apal21 review the pr when you get time.

@imranbarbhuiya imranbarbhuiya changed the title WIP: Rewrite in typescript Rewrite in typescript Dec 21, 2021
@Shahzad6077
Copy link

Thanks @imranbarbhuiya for this PR.

Hey @apal21, any updates regarding this PR?

@apal21
Copy link
Owner

apal21 commented Jan 17, 2022

@Shahzad6077 will approve and merge it once I test it on the previous next.js versions.

@imranbarbhuiya thanks for the changes.

@imranbarbhuiya imranbarbhuiya force-pushed the master branch 2 times, most recently from 5b17332 to cdb9542 Compare February 26, 2022 02:02
@apal21 apal21 merged commit a122fd2 into apal21:master Feb 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants