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

React 18, Upgrade Dependencies, Fix Copy/Paste #22

Merged
merged 6 commits into from
Mar 3, 2023

Conversation

GeorgS
Copy link
Contributor

@GeorgS GeorgS commented Mar 3, 2023

Hi!

I've upgraded the dependencies and made sure it works with React 18. Also fixed some type errors I got during upgrading.

When trying the copy/paste the example I noticed that "[Object object]" was being pasted when copying cells. I tried to fix it, but I am not sure that the way I changed it is the way it was intended. Let me know if this should be handled differently.

I have also bumped the version to 9.0.0.

@rmdort rmdort merged commit 01965ff into rowsncolumns:master Mar 3, 2023
@rmdort
Copy link
Member

rmdort commented Mar 3, 2023

Thank you. I have made my edits and merged your PR.

@GeorgS
Copy link
Contributor Author

GeorgS commented Mar 3, 2023

Thanks for the quick merge, appreciate it!
I've noticed that the Readme states the port 6006 from my edit (due to the storybook upgrade) but should also be reverted to 9002 with the revert of storybook.

@rmdort
Copy link
Member

rmdort commented Mar 3, 2023

Sure. I have updated readme. I had to downgrade Storybook to version 6, due to some incompatibility issues. Will upgrade once its stable.

@GeorgS
Copy link
Contributor Author

GeorgS commented Mar 3, 2023

When installing the package in the new 9+ version I cannot import, likely due to "main": "dist/index.tsx", - index.tsx does not existing in the dist folder.

@rmdort
Copy link
Member

rmdort commented Mar 3, 2023

oops. Could you try with @rowsncolumns/grid@9.0.3

@GeorgS
Copy link
Contributor Author

GeorgS commented Mar 3, 2023

works now, thanks 👍

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.

None yet

2 participants