vmpyre/helius_sdk

"txn_status" missing from WebhooksAPI create_webhook and edit_webhook

Closed this issue · 1 comments

I've noticed that the current implementation of the WebhooksAPI class in the Helius SDK does not support the txn_status parameter, which is available in the Helius API as per the official documentation. This parameter allows users to filter webhook triggers based on transaction status, such as all, success, or failed.

This functionality is crucial for applications that need to differentiate between successful and unsuccessful transactions for better handling and processing.

I propose updating the create_webhook and edit_webhook methods to include an optional txn_status parameter to accommodate this functionality. I have already implemented this change locally and am interested in contributing it back to the repository.

Please let me know if this change aligns with your roadmap, and I would be happy to submit a pull request.

Thank you for considering this enhancement.

Definitely. Going to give the changes a quick test and merge later 🤝