As I was saying in this PR it's usually better to handle modals imperatively through Mantine's Modal Manager.
This allows to:
keep the flow of the events, making it easier to debug for everyone by avoiding using effects
improve the callback handlers
modals are opened in the button's onClick/handler, and params can be passed directly from the list instead of having to save and rerender the whole component
avoid reinventing the wheel for every modal
e.g. having to add a piece of state just to know if the modal is opened or closed
IMO, this behavior should be used for ALL the modals, whether they are confirm modals or not
As I was saying in this PR it's usually better to handle modals imperatively through Mantine's Modal Manager.
This allows to:
IMO, this behavior should be used for ALL the modals, whether they are
confirm
modals or not