Skip to content
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

Refactor SubscribeTransactionStatus #2424

Open
rodrigo-pino opened this issue Feb 5, 2025 · 1 comment
Open

Refactor SubscribeTransactionStatus #2424

rodrigo-pino opened this issue Feb 5, 2025 · 1 comment

Comments

@rodrigo-pino
Copy link
Contributor

SubscribeTransactionStatus function, in the rpc package is a huge function (not adding any links because the package is going through heavy refactoring).

It is a complex function which holds in itself the definition of another complex function. The inner function should be re-factored apart and be defined externally.

@TheRanomial
Copy link

I’d like to resolve this.

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

No branches or pull requests

2 participants