Skip to content

Conversation

TheMaskedTurtle
Copy link
Contributor

@TheMaskedTurtle TheMaskedTurtle commented Sep 17, 2025

  • Reuse code to update all nodes in the same notification callback
  • Extract notification handling at spreadsheet-view level to avoid duplicated calls
  • Delete by-pass for local spreadsheet modification
  • Delete local blonk handling on notification
  • Fix in reducer to map update data only if necessary
  • Fix in remove node data to do it only once in active tab

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

license-eye has totally checked 863 files.

Valid Invalid Ignored Fixed
814 1 48 0
Click to see the invalid file list
  • src/components/spreadsheet-view/hooks/use-spreadsheet-nodes.ts

@TheMaskedTurtle TheMaskedTurtle force-pushed the jorism/fix-spreadsheet-update-on-notifications branch from 49d61c0 to fbd0365 Compare September 17, 2025 15:19
@TheMaskedTurtle TheMaskedTurtle force-pushed the jorism/fix-spreadsheet-update-on-notifications branch from fbd0365 to b555cad Compare September 17, 2025 15:19
Copy link

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.

2 participants