-
Notifications
You must be signed in to change notification settings - Fork 11.1k
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
Add dev-inspect-txn to RPC and SDK #7008
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Accepting to unblock, thanks Chris!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leaving a more detailed review -- most of the comments are not urgent so don't block the PR for them if this needs to go out urgently, but note the one about batch transactions always being dev inspect now. Maybe I've misunderstood what that code does, but it seemed not right to me.
@amnn Thanks for the detailed review and great feedback! I've addressed them in the last pushed commit |
Same as #6998 but - fixed typescript types - Relax RPC restriction for calling non-entry functions Co-authored-by: Ge Gao <106119108+gegaowp@users.noreply.github.com>
Same as #6998 but