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.

Sign up for GitHub

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

Jump to bottom

feat: Expose transact async for asynchronous transactions #222

Merged
merged 37 commits into from
Oct 31, 2022
Merged

feat: Expose transact async for asynchronous transactions #222

merged 37 commits into from
Oct 31, 2022

Conversation

ChaoticTempest
Copy link
Member

@ChaoticTempest ChaoticTempest commented Oct 19, 2022 •
edited
Loading

This adds transact_async to CallTransaction and Transaction objects so users can call into committing transactions without having to wait for them to complete if their transactions are independent of each other. TransactionStatus type also added to check the status of the transaction afterwards and finally grab its result.

addresses #216

@ChaoticTempest
Initial into_future working
ad7def3
@ChaoticTempest
Moved over to generic impl
96a7d5a
@ChaoticTempest
Cleanup and fixing ViewFunction
84d9105
@ChaoticTempest
Added ViewFunction impls and FunctionOwned/Args type
a91c877
@ChaoticTempest
Cleanup usage of futures::TryFutureExt
4cb54d1
@ChaoticTempest
Some more cleanup
1ce4b43
@ChaoticTempest
Updated view_code and view_state to use new async builder
38ef6d0
@ChaoticTempest
Added view async builder to API surface
f39a58b
@ChaoticTempest
Removed view_latest_block for view_block instead
1902dd4
@ChaoticTempest
view_account async builder now apart of the API surface
ef4fe5a
@ChaoticTempest
Got rid of unused client.view_{code, state}
3b5b414
@ChaoticTempest
Sort imports
50bcf1f
@ChaoticTempest
Added Finality type
5f2a6d2
@ChaoticTempest
Added doc
6e431c8
@ChaoticTempest
Expose access keys
eb99079
@ChaoticTempest
Expose access keys list
99b12a4
@ChaoticTempest
Rename QueryMethod => Method
807e937
@ChaoticTempest
Added gas price
b22ac35
@ChaoticTempest
Rename Queryable methods
ae020ef
@ChaoticTempest
Rename Queryable to ProcessQuery
27b6899
@ChaoticTempest
Merge branch 'main' of https://github.com/near/runner-rs into feat/vi…
897ab47 
…ew-builders
austinabell
austinabell reviewed Oct 25, 2022
View reviewed changes
workspaces/src/operations.rs Outdated Show resolved Hide resolved
workspaces/src/operations.rs Show resolved Hide resolved
workspaces/tests/parallel_transactions.rs Outdated Show resolved Hide resolved
workspaces/tests/parallel_transactions.rs Show resolved Hide resolved
workspaces/src/operations.rs Outdated Show resolved Hide resolved
workspaces/tests/parallel_transactions.rs Outdated Show resolved Hide resolved
Base automatically changed from feat/view-builders to main October 31, 2022 23:11
@ChaoticTempest ChaoticTempest merged commit 0b80d06 into main Oct 31, 2022
@ChaoticTempest ChaoticTempest deleted the feat/transact_async branch October 31, 2022 23:34
@frol frol mentioned this pull request Oct 4, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@austinabell austinabell austinabell approved these changes

@itegulov itegulov Awaiting requested review from itegulov

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ChaoticTempest @austinabell