-
Notifications
You must be signed in to change notification settings - Fork 14
Feat: Mint operation saga #113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
507b571
test(core): add mintOperationService tests and update quotes api test
igbopharaoh f4f5f9a
feat(core): add mintOperationService saga and method handler abstract…
igbopharaoh 6797fb6
fix: add mint handler methods
igbopharaoh c8ad95f
fix: use pending lifecycle for mint operations
Egge21M 10e22d7
fix: persist failed mint operation state
Egge21M 1feaf48
refactor: expand mint operation metadata and remote-state handling
Egge21M b375413
feat(core): move mint quote flows into mint ops
Egge21M 8272225
refactor(core): complete mint operation migration
Egge21M 3f7cd67
docs: update mint operation migration docs
Egge21M c800ae9
feat(core): persist mint operation quote observations
Egge21M f2a1365
test(adapter-tests): migrate mint integration flows to ops api
Egge21M 259d7b0
fix(core): reject unsupported mint prepare units
Egge21M a40834f
fix(core): reject unsupported mint import units
Egge21M File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| --- | ||
| 'coco-cashu-expo-sqlite': patch | ||
| 'coco-cashu-sqlite-bun': patch | ||
| 'coco-cashu-indexeddb': patch | ||
| 'coco-cashu-sqlite3': patch | ||
| 'coco-cashu-core': patch | ||
| 'coco-cashu-adapter-tests': patch | ||
| --- | ||
|
|
||
| Finish the mint quote removal migration and make mint operations the runtime source of truth. | ||
|
|
||
| - Replace legacy `mint-quote:*` runtime events with operation-based mint events. | ||
| - Rename watcher and processor config and manager methods to the operation-based surface: | ||
| `mintOperationWatcher`, `mintOperationProcessor`, `enableMintOperationWatcher()`, | ||
| `enableMintOperationProcessor()`, and related disable/wait helpers. | ||
| - Remove the legacy `MintQuoteService` runtime path and keep `MintQuoteRepository` only for | ||
| cold-start reconciliation of old persisted quote rows. | ||
| - Move mint watcher, processor, history, and recovery flows onto `manager.ops.mint`. | ||
|
|
||
| This is a breaking change for consumers using the old mint watcher/processor config keys, | ||
| manager methods, or `mint-quote:*` events. | ||
Large diffs are not rendered by default.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This changeset still publishes
coco-cashu-coreas a patch even though the diff removes publicmanager.quotes.*mint methods, renames the watcher/processor config keys and manager helpers, and drops themint-quote:*event surface. Consumers on a normal^range will pick up this patch automatically and hit compile/runtime breaks without any semver warning, so this needs a breaking-version release or compatibility shims.Useful? React with 👍 / 👎.