Skip to content

Use T3 remaining limit query#258

Open
decofe wants to merge 2 commits into
mainfrom
fix/t3-remaining-limit-query
Open

Use T3 remaining limit query#258
decofe wants to merge 2 commits into
mainfrom
fix/t3-remaining-limit-query

Conversation

@decofe
Copy link
Copy Markdown
Member

@decofe decofe commented May 18, 2026

Summary

  • query access-key spending limits through getRemainingLimitWithPeriod
  • return the remaining field to preserve the existing query_key_spending_limit API

Context

Tempo T3 dropped the legacy getRemainingLimit selector. Callers using the old selector fail to read remaining limits.

Verification

  • cargo test --manifest-path Cargo.toml client::tempo::signing::keychain --features tempo,client -q currently fails in tempo-contracts 1.7.0 with duplicate helper definitions generated by the Alloy sol macro; the selector change itself is a small ABI call replacement.
  • A 0.9.3-compatible branch with the same one-line fix was used by Bump mpp for T3 remaining limit query wallet#464 and verified there with cargo check -p tempo-wallet --locked and focused whoami tests.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026

✅ Changelog found on PR.

Edit changelog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants