search

lowcoder-org / lowcoder

🔥 🔥 🔥 The Open Source Retool, Tooljet and Appsmith Alternative
https://www.lowcoder.cloud/
GNU Affero General Public License v3.0
812 stars 175 forks source link

[Feat]: Table - Mark column as excluded from toUpdateRows #973

Open beatles1 opened 3 months ago

beatles1 commented 3 months ago

Is there an existing issue for this?

Current Behavior

When an editable table has a column manually added (in my case for a button to delete the row) that value is included in the toUpdateRows property. This means that if you use the GUI Mode Bulk Update option you get an error about this column. Screenshot 2024-06-20 170722

Expected Behavior

There should be a setting for the column to not include it in the toUpdateRows property.

Steps to reproduce

Create a table from some data, add a custom column not from the data, make some of the columns editable, add a query using the GUI Mode and the Bulk Update option to update the values in the DB and pass in the table toUpdateRows property.

Environment

Lowcoder 2.4.1 - Docker Compose All in One

Additional Information

Initial thread on Discord: https://discord.com/channels/1096896040159957084/1253378632555167886

FalkWolsky commented 1 month ago

We agree to change this behaviour - but we see it as enhancement. How so? Generally Lowcoder has all tools on board to deal with such cases too. For example, you can use a data-transformer to "clean" your data fro. toUpdateRows before the GUI Mode Query. But indeed - we understand the need by itself and will work on it.