Skip to content

Integrate EditTsv into EditGrid#376

Merged
jgunstone merged 47 commits intomainfrom
366-add-io-to-edit-grid
Nov 18, 2025
Merged

Integrate EditTsv into EditGrid#376
jgunstone merged 47 commits intomainfrom
366-add-io-to-edit-grid

Conversation

@Arshadwaqas115
Copy link
Contributor

@Arshadwaqas115 Arshadwaqas115 commented Sep 24, 2025

Feature(s) added to edittsv:

  • When we click the upload button, a new view of the changes appears, prompting the user to either accept or decline the changes.

  • When the user clicks on the accept button, the changes are applied.

  • When the user clicks on the decline button, the changes are reverted.

1 Screenshot 2025-09-30 134605

@Arshadwaqas115 Arshadwaqas115 linked an issue Sep 24, 2025 that may be closed by this pull request
4 tasks
@jgunstone
Copy link
Collaborator

looking good... but just tried the demo and the final button to accept changes isn't there...?

image

also. would be good to update the text where it currently says "change diff" to "addition in green, deletions in red" (with the word green colored green and red red

@Arshadwaqas115
Copy link
Contributor Author

The issue has now been fixed. It was a rendering issue in voila which caused the deepdiff custom widget to take up the full width, pushing the buttons off the screen. Setting a fixed with for the deepdiff widget solved the issue.

@jgunstone
Copy link
Collaborator

great - could you add a couple of tests... don't need to be comprehensive but just to make sure it instantiates properly and basis functionality

@Arshadwaqas115
Copy link
Contributor Author

The tests have been added for functionality of the deepdiff and accept / reject buttons

@Arshadwaqas115 Arshadwaqas115 changed the title Updates to edittsv - Added functionality to approve/revert changes displayed with deepdiff before committing changes to data Integrate EditTsv into EditGrid Nov 6, 2025
@jgunstone jgunstone merged commit 3244a2a into main Nov 18, 2025
2 checks passed
@jgunstone jgunstone deleted the 366-add-io-to-edit-grid branch November 18, 2025 10:49
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.

✨ add io to edit grid

2 participants