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
Deleting a column on google spreadsheet messes with the data #13652
Deleting a column on google spreadsheet messes with the data #13652
Conversation
expect(sheet.setHeaderRow).toHaveBeenCalledWith(["name"]) | ||
|
||
// No undefined are sent | ||
expect((sheet.setHeaderRow as any).mock.calls[0][0]).toHaveLength(1) |
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.
Not sure what this was meant to do, as it was already asserted on the previous lines
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.
Took me a little while to get my head around why this worked but I see it now. Great PR, love the typing improvements.
…adsheet-messes-with-the-data
…adsheet-messes-with-the-data
Description
Currently, deleting a column deletes the header cell but not the full column. Because of this, deleting data mixes the order of the columns. We don't want to delete the data as these data might not be created by budibase. Because of this, we will only drop the BB reference but keeping the references in the actual sheet. Adding the deleted column back will backpopulate it in BB.
Also fixing a bug that allowed spamming the
create/edit column
submit, causing strange errors.https://github.com/Budibase/budibase/assets/15987277/f6a6f5b5-08ec-4c99-b27e-5dc888fb49db
Addresses
App Export
Launchcontrol
Fixing broken data on google sheet column deletion