Possible refactor of UpdatePagesService class
The following discussion from !86072 (merged) should be addressed:
-
@mikolaj_wawrzyniak started a discussion: suggestion (non-blocking): Errors in ruby are performance heavy and therefore are discouraged as a tool to control standard (aka expected) execution flow https://mensfeld.pl/2015/03/exceptions-should-not-be-expected-stop-using-them-for-control-flow-or-any-other-logic-handling-in-ruby/
Normally I would kindly ask for change here, but I see that this error based approach was already established in this service prior to this MR, so I only want to kindly ask you to consider opening follow up issue to discuss possible refactor of this class