Closed PsychoLlama closed 5 months ago
Latest commit: c770ff22a40d7b2de4be9594877b4a295faea9cd
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
Name | Link |
---|---|
Latest commit | c770ff22a40d7b2de4be9594877b4a295faea9cd |
Latest deploy log | https://app.netlify.com/sites/preact-signals-demo/deploys/657622214238a10008f430fa |
Deploy Preview | https://deploy-preview-461--preact-signals-demo.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Closing - see related issue for discussion.
This is an implementation for my feature request in issue 460. It adds a new function called
transaction(...)
which behaves similar tobatch(...)
, except it rolls back all the state changes if an error is thrown.Intermediate states aren't observable until the transaction is complete.
I know this change is public facing so I'm not expecting it to be straightforward. Think of the PR as a proof of concept and feel free to say no.