- π«π·France bakop Lille
Hi, there has been no activity for a while, I take the liberty of modifying the original patch to update it. I didn't have time to set up the tests.
- @bakop opened merge request.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 3:19pm 7 July 2023 - Assigned to bakop
- Status changed to Active
over 1 year ago 3:40pm 7 July 2023 - π«π·France bakop Lille
I push to fast. A little issue with my patch, i will fix it.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 9:02pm 7 July 2023 - First commit to issue fork.
- π©πͺGermany marcoliver Neuss, NRW, Germany
I have updated the issue fork against 3.1.x-dev and fixed the failing tests.
Could someone with a bit more history on this issue kindly check if everything still works as expected?
- Assigned to bakop
- Status changed to Active
over 1 year ago 4:21pm 12 September 2023 - π«π·France bakop Lille
Hi, the patch still works but a duplicate foreach slipped in.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 4:32pm 12 September 2023 - π«π·France bakop Lille
MR !26 updated to remove the duplicated foreach.
-
marcoliver β
committed 93d686fa on 3.1.x-dev authored by
bakop β
Issue #3043243 by jacobbell84, bakop, marcoliver, Peter Majmesku,...
-
marcoliver β
committed 93d686fa on 3.1.x-dev authored by
bakop β
- Status changed to Fixed
over 1 year ago 11:52am 13 September 2023 - π©πͺGermany marcoliver Neuss, NRW, Germany
@bakop Thanks for catching that duplication.
I gave the fork another quick test myself and merged afterwards.
Automatically closed - issue fixed for 2 weeks with no activity.