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 package rewrite #363

Merged
merged 11 commits into from
Oct 1, 2022
Merged

React package rewrite #363

merged 11 commits into from
Oct 1, 2022

Conversation

BitPhinix
Copy link
Owner

Rewrite to make @slate-yjs/react ready for async react, add cursors decorations and other utility hooks for commonly requested features

@changeset-bot
Copy link

changeset-bot bot commented Aug 22, 2022

🦋 Changeset detected

Latest commit: 210fd21

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@slate-yjs/core Major
@slate-yjs/react Major
@slate-yjs/example-backend Patch
@slate-yjs/example Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@BitPhinix BitPhinix changed the title Initial useDecorateRemoteCursors implementation Reach package rewrite Aug 22, 2022
@BitPhinix BitPhinix changed the title Reach package rewrite React package rewrite Aug 22, 2022
@codecov
Copy link

codecov bot commented Aug 22, 2022

Codecov Report

Merging #363 (210fd21) into main (95d5203) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #363   +/-   ##
=======================================
  Coverage   71.31%   71.31%           
=======================================
  Files          30       30           
  Lines        2301     2301           
  Branches      297      297           
=======================================
  Hits         1641     1641           
  Misses        660      660           
Impacted Files Coverage Δ
packages/core/src/plugins/withCursors.ts 26.39% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@BitPhinix BitPhinix marked this pull request as ready for review September 30, 2022 19:25
@BitPhinix BitPhinix merged commit 3710c48 into main Oct 1, 2022
@BitPhinix BitPhinix deleted the react-rewrite branch October 1, 2022 14:39
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

1 participant