Skip to content

Refactor to use rescue_from

Thong Kuah requested to merge ar_rescuable into master

What does this MR do and why?

Refactor reset on error to use AR rescue_from

We use the recently added ActiveRecord rescue_from (from !124131 (merged)) to move the reset on union error from monkey-patch to a model concern.

Related issue: #414226 (closed), !121957 (merged)

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Thong Kuah

Merge request reports