Skip to content

feat(mint): add issued_time to mint_quotes table#985

Merged
callebtc merged 1 commit into
mainfrom
feature/add-issued-time-to-mint-quotes
Apr 25, 2026
Merged

feat(mint): add issued_time to mint_quotes table#985
callebtc merged 1 commit into
mainfrom
feature/add-issued-time-to-mint-quotes

Conversation

@a1denvalu3
Copy link
Copy Markdown
Collaborator

Closes #980

Adds issued_time to mint_quotes in the mint's database to provide a precise timestamp of the most important event for the minting process, which is necessary for failed minting investigations.

Modifies:

  • DB migrations: added m033_add_issued_time_to_mint_quote
  • Mint operations and db writes
  • RPC: Added issued_time to Nut04Quote in management.proto

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 23, 2026

Codecov Report

❌ Patch coverage is 23.68421% with 29 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.24%. Comparing base (f32a6ce) to head (99a34e7).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
cashu/mint/management_rpc/protos/management_pb2.py 3.33% 29 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #985      +/-   ##
==========================================
+ Coverage   75.14%   75.24%   +0.09%     
==========================================
  Files          99       99              
  Lines       11714    11722       +8     
==========================================
+ Hits         8803     8820      +17     
+ Misses       2911     2902       -9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Collaborator

@callebtc callebtc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good one

@callebtc callebtc merged commit 75926d7 into main Apr 25, 2026
84 of 88 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in nutshell Apr 25, 2026
jooray added a commit to jooray/nutshell that referenced this pull request May 10, 2026
* upstream/main:
  fix(models): increase filter max_length to accommodate quote IDs
  refactor: move hardcoded string max lengths to constants
  feat(mint): add issued_time to mint_quotes table (cashubtc#985)
  fix(models): correctly apply Pydantic max_length to strings in lists
  ci: allow manual docker build trigger (cashubtc#973)
  fix(docker): replace deprecated --no-dev with --without dev for Poetry 2.x (cashubtc#972)
  fix(mint): enforce unique x-coordinates in p2pk pubkey deduplication (cashubtc#969)
jooray added a commit to jooray/nutshell that referenced this pull request May 10, 2026
* units:
  fix(models): increase filter max_length to accommodate quote IDs
  refactor: move hardcoded string max lengths to constants
  feat(mint): add issued_time to mint_quotes table (cashubtc#985)
  fix(models): correctly apply Pydantic max_length to strings in lists
  ci: allow manual docker build trigger (cashubtc#973)
  fix(docker): replace deprecated --no-dev with --without dev for Poetry 2.x (cashubtc#972)
  fix(mint): enforce unique x-coordinates in p2pk pubkey deduplication (cashubtc#969)
  fix(mint): preserve unitsd keyset units and decimals
  Fix unit inference: use coin_type (index 2) not counter (last segment) from derivation path
  Update fiat-backend.md for unitsd architecture
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[feature] Add issued_time to mint_quotes table

2 participants