Refactor the "NoFraudListBuilder" into the default core "EntityListBuilder"

Comments & Activities

Issue created by @andrei.vesterli
@andreivesterli opened merge request.
Assigned to thenchev
Status changed to Needs review 8 months ago5:04pm 27 October 2023
Comment 8 months ago →
🇲🇩Moldova andrei.vesterli Chisinau
Hi @thenchev

Can you perform a quick review for this one?

1. We need the NoFraudListBuilder class as there are 2 new operations special for view listing
2. I just had to remove some non-relevant code from that listing class. The rest is ok.
Assigned to dylf
Comment 8 months ago →
🇲🇩Moldova andrei.vesterli Chisinau
Hey Dil

Take your time to review this one
Assigned to andrei.vesterli
Status changed to RTBC 8 months ago5:28pm 27 October 2023
Comment 8 months ago →
🇨🇦Canada dylf
@andrei.vesterli Tested this patch on my local. Everything appears to be working correctly RTBC +1!
Comment 8 months ago →
🇨🇦Canada dylf
Comment 8 months ago →
System Message

dylf → committed ffa4ac6d on 1.x authored by andrei.vesterli →
Issue #3396984 by andrei.vesterli, dylf: Refactor the "...
Comment 8 months ago →
🇨🇦Canada dylf
Status changed to Fixed 8 months ago5:32pm 27 October 2023
Comment 8 months ago →
🇲🇩Moldova andrei.vesterli Chisinau
Status changed to Fixed 8 months ago5:33pm 27 October 2023
Comment 8 months ago →
🇲🇩Moldova andrei.vesterli Chisinau

Comments & Activities