Skip to content

add GetTaddressTransactions, label GetTaddressTxids deprecated #522

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LarryRuane
Copy link
Collaborator

@LarryRuane LarryRuane commented May 21, 2025

Closes #426.

Add GetTaddressTransactions as a new gRPC method, with the same functionality as GetTaddressTxids. The motivation is that GetTaddressTransactions is a more accurate name, since it returns transactions, not transaction IDs (txids). Label the old method as deprecated.

@LarryRuane LarryRuane self-assigned this May 21, 2025
@LarryRuane LarryRuane added the enhancement New feature or request label May 21, 2025
These two gRPC methods are functionally identical, just a name change
(but leaving the old method deprecated during transition).
@LarryRuane LarryRuane force-pushed the 2025-05-add-GetTaddressTransactions branch from 7b0c1b4 to cf0b73a Compare May 21, 2025 04:34
@LarryRuane LarryRuane requested review from str4d and daira May 21, 2025 04:42
Copy link
Contributor

@y4ssi y4ssi left a comment

Choose a reason for hiding this comment

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

utACK

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GetTaddressTxids is misnamed
2 participants