-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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: add useBlocksuiteEditor hooks #5366
Conversation
Your org has enabled the Graphite merge queue for merging into canaryYou must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. You can enable merging using labels in your Graphite merge queue settings. |
Current dependencies on/for this PR:
This stack of pull requests is managed by Graphite. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## canary #5366 +/- ##
=======================================
Coverage 67.21% 67.21%
=======================================
Files 190 190
Lines 15384 15384
Branches 1059 1059
=======================================
Hits 10341 10341
Misses 4900 4900
Partials 143 143
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
packages/frontend/component/src/components/block-suite-editor/index.tsx
Outdated
Show resolved
Hide resolved
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.
Please resolve my comments
194291a
to
5741bea
Compare
So the new impl does not support multiple editors? |
Apologies, it seems I’ve misconstrued your meaning. I interpreted the comment as suggesting that the editor atom didn’t require correspondence with the pageid. @pengx17
|
5741bea
to
41d907b
Compare
Merge activity
|
41d907b
to
e11e827
Compare
No description provided.