-
Notifications
You must be signed in to change notification settings - Fork 198
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
fix: table block bugs #431
Merged
Merged
Commits on Aug 30, 2023
-
Configuration menu - View commit details
-
Copy full SHA for e585bff - Browse repository at this point
Copy the full SHA e585bffView commit details -
Configuration menu - View commit details
-
Copy full SHA for 978d881 - Browse repository at this point
Copy the full SHA 978d881View commit details -
Configuration menu - View commit details
-
Copy full SHA for 830bb02 - Browse repository at this point
Copy the full SHA 830bb02View commit details -
refactor: don't pass editorState to TableNode
pass only transaction to methods directly.
Configuration menu - View commit details
-
Copy full SHA for f260258 - Browse repository at this point
Copy the full SHA f260258View commit details -
Configuration menu - View commit details
-
Copy full SHA for bd111b4 - Browse repository at this point
Copy the full SHA bd111b4View commit details -
fix: mitigate Appflowy transaction adapter problem
With current transaction adapter in Appflowy which creates `BlockActionPB` for `documentService` (backend). Its not possible to have multiple `InsertOperation`s separately as a single transaction (multiple `transaction.insertNode()`). The `BlockActionPB` creator (`toBlockAction`) can't figure out the `prevId` (previous node of inserting node id), The first insert operations shifts all next nodes path. Because of this situation we decided to apply the insert operations one by one.
Configuration menu - View commit details
-
Copy full SHA for 834079f - Browse repository at this point
Copy the full SHA 834079fView commit details -
Configuration menu - View commit details
-
Copy full SHA for ae60deb - Browse repository at this point
Copy the full SHA ae60debView commit details -
Configuration menu - View commit details
-
Copy full SHA for 4be847e - Browse repository at this point
Copy the full SHA 4be847eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 493ec1f - Browse repository at this point
Copy the full SHA 493ec1fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7cbf5ff - Browse repository at this point
Copy the full SHA 7cbf5ffView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.