Tom Morris
Tom Morris
This is why we need better isolation for extension dependencies. I'll try to have a look at what's going on sometime over the long weekend.
> I've managed to track down that https://github.com/OpenRefine/OpenRefine/commit/0945595a795363743517c3e362871e832bdb367a is the revision Commons stops showing up in the list of sources. It's a small change that only updates the Velocity library....
I suspect the error in https://github.com/OpenRefine/OpenRefine/issues/6581#issuecomment-2126387761 is caused by stale `.class` files. The change in resource loading behavior actually happened between Velocity 1.5, which is a dependency of OpenRefine 3.7.7,...
The compilation errors in https://github.com/OpenRefine/OpenRefine/issues/6581#issuecomment-2138895162 are worrying because they seem to indicate incompatible API changes. That deserves more investigation. Similarly, I came across #6375 while debugging and we should be...
The deleted constructors have been restored and a new version of the Commons extension has been released, but we should probably release note the behavior change in case any other...
You can address the linter failures with the command `./refine lint`. For the UI, let's keep things simple to start and just do "Add N rows at the beginning" (which...
The dialog button changes look good. I've only done cursory testing so far, but my first attempt generated something weird which I wanted to report. I did: 1) prepend, 2)...
@steve-kasica @wetneb Sorry for the delay! Reviewing now...
@steve-kasica Are you still working on this? Are you able to address the test comments above? I don't think they'll be a ton of work, so I can probably add...
Awesome. Thank you for sticking with it!