OpenRefine / OpenRefine / 12500407313

50%

push

github

Issue 3218 - Add "remove duplicate rows" menu item (#7040)

* Issue-3218 Feature - Remove duplicate rows

Issue-3218 Feature - Remove duplicate rows

* Added unit test

Added unit test
Updated/ removed copyright text
bug fixes

* Feedback addressed

UI - removed select all checkbox; added buttons for select all & deselect all. labels updated, extra div removed

* Feedback addressed

- Hashkey generation
- Removed json handling
- Used RowVisitor
- Tested with Facets

* unit test fix

* Unit test failure fix

Unit test failure fix

* Update main/webapp/modules/core/langs/translation-en.json

Co-authored-by: Antonin Delpeuch <antonin@delpeuch.eu>

* Feedback addressed

Feedback addressed

* Update main/webapp/modules/core/styles/dialogs/remove-duplicate-rows-dialog.css

Co-authored-by: Antonin Delpeuch <antonin@delpeuch.eu>

* Addressed feedback

- fixed UI styling issues
- updated unit test

* Update main/webapp/modules/core/langs/translation-en.json

Co-authored-by: Thad Guidry <thadmguidry@outlook.com>

---------

Co-authored-by: Antonin Delpeuch <antonin@delpeuch.eu>
Co-authored-by: Thad Guidry <thadmguidry@outlook.com>

5416 of 12400 branches covered (43.68%)

Branch coverage included in aggregate %.

13582 of 25858 relevant lines covered (52.53%)

2.41 hits per line