Enable multi-cell selection for deleting content of cells

[guergana]

• fixes Cannot delete content of multiple cells #350

[cloudflare-pages]

Deploying opendataeditor with    Cloudflare Pages

Latest commit:	21e2579
Status:	✅  Deploy successful!
Preview URL:	https://a4205b81.opendataeditor.pages.dev
Branch Preview URL:	https://350-select-multiple-cells.opendataeditor.pages.dev

View logs

[roll]

@guergana
I think I found the problem. Take a look at saveEditing action. Note, that we never change the grid values directly. What we do is:

• add a change to the history
• initiate grid reloading
  • grid calls loader
  • loader gets data from the server
  • loader applies changes from the history
  • grid gets a new data that is patched

The same flow needs to be used in deleteMultipleCells

[guergana]

@guergana I think I found the problem. Take a look at saveEditing action. Note, that we never change the grid values directly. What we do is:

* add a change to the history

* initiate grid reloading
  
  * grid calls loader
  * loader gets data from the server
  * loader applies changes from the history
  * grid gets a new data that is **patched**

The same flow needs to be used in deleteMultipleCells

Done, thanks!

[roll]

Great! It works for me 👍