Closed brendon closed 1 month ago
I just wanted to float this concept before cleaning it up (documentation and tests). What do you think? I know it breaks the existing API but it's better to rename the methods to *All
rather than call the new methods and callbacks onRestoreTarget
IMO, but we could look at that as an alternative.
Ended up rolling my own solution in the end.
onBeforeRestore
andonRestore
are now granular and provide the target form. They also don't trigger if there is no data to restore for that particular form.onBeforeRestoreAll
andonRestoreAll
operate likeonBeforeRestore
andonRestore
used to.No tests so far.
Potential to close #138 with more work.