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

Polling Transaction Status #315

Merged
merged 56 commits into from
Sep 1, 2023
Merged

Polling Transaction Status #315

merged 56 commits into from
Sep 1, 2023

Conversation

viv-defy
Copy link
Contributor

No description provided.

@viv-defy
added skeletal structure of the code
64ea6b6
@viv-defy
added skeletal structure of the code
9c34761
@viv-defy
implement poll tx status from frontend
e2acab2
@viv-defy
moved the get_status function to transfer service
5595c7f
@viv-defy
changed the status response
cb4c4e3
@viv-defy
renamed struct; used Status enum; changed return types
b9ae1d3
@viv-defy
renamed _txn_id to txn_id
6a820f6
@viv-defy
added update_user_transactions function in TransactionDao
ff86aca
@viv-defy
added chain query for txn status poll
5f78b02
@viv-defy
passed transaction_hash as a String to the DB layer
17dc13b
@viv-defy
modified comment in the update function
8bafb1d
@viv-defy
split a series of function calls into readable chunks
6dfafd4
@viv-defy
return transaction as json reponse
1b7b99b
@viv-defy
Merge branch 'feature/polling-transaction-status' into 272-frontend-flow
79ae61a
@viv-defy
added changes to supress warnings
e2b9081
@viv-defy
added git status
fd086aa
@viv-defy
added passing Pool<> instead of Data<Pool<>> to service layer
c914c8e
@viv-defy
added db functions; response message
5a9cec9
@viv-defy
moved constants to a new file; used entrypoint_address from config, r…
770456f 
…emoved the await for the listener
@viv-defy
Merge branch 'feature/polling-transaction-status' into 275-backend-flow
5dface7
@viv-defy
added fixes to the merge
c6d03f8
@viv-defy
added DB Query to update user_transactions
97b8fac
@viv-defy
added changes to the DB code; refactored code by adding mappers
0764d4d
@viv-defy
refactored get_transactions code
c3a45a2
@viv-defy
removed unused imports
faff52e
@viv-defy
added mappers; removed unnecessary code
b54b241
@viv-defy
changed signature of poll_transaction's return type
680896b
@viv-defy
passed references of db_pool instead of passing the clones
a2ec42c
@viv-defy
removed unwanted dereferencing
45a77e9
@viv-defy
added USEr_OPERATION_EVENT to constants
68f83c1
viv-defy and others added 13 commits August 30, 2023 18:27
@viv-defy
added auth in poll_transaction
30ccfcf
@viv-defy
replaced Bad Request with Unauthorised
ff048c5
@viv-defy
added user_wallet_address check in the query
51e8d48
@viv-defy
spawned a task/thread to execute user_op_event_listener; moved the li…
263b0fd 
…stener call after the db update
@viv-defy
removed wallet_dao from app_data
2b1a8e9
@vishnukvnvv
Merge pull request #273 from roverxio/272-frontend-flow
f8106a8 
poll txn status - dummy apis
@viv-defy
Merge branch 'feature/polling-transaction-status' into 275-backend-flow
138e4dc
@viv-defy
removed cloning for status
ad92f22
@viv-defy
used Status enum for success and failed
9f532e4
@viv-defy
removed txn_hash(which is a String).to_string()
77bcdf1
@viv-defy
reordered the Status enum; replaced unwrap with expect in the dao lay…
5420c67 
…er; removed unwanted code; rename update_user_transactions to update_user_transaction
@vishnukvnvv
Merge pull request #276 from roverxio/275-backend-flow
b269952 
poll txn status - backend flow implementation
@viv-defy
Merge branch 'release/choco-chips' into feature/polling-transaction-s…
22287fc 
…tatus
@viv-defy viv-defy marked this pull request as draft August 31, 2023 11:20
@viv-defy viv-defy linked an issue Aug 31, 2023 that may be closed by this pull request
Polling transaction status #258
Closed
3 tasks
@viv-defy viv-defy marked this pull request as ready for review September 1, 2023 09:31
vishnukvnvv
vishnukvnvv requested changes Sep 1, 2023
View reviewed changes
bundler/src/db/dao/transaction_dao.rs Outdated Show resolved Hide resolved
vishnukvnvv
vishnukvnvv requested changes Sep 1, 2023
View reviewed changes
bundler/src/db/dao/transaction_dao.rs Outdated Show resolved Hide resolved
bundler/src/db/dao/transaction_dao.rs Outdated Show resolved Hide resolved
bundler/src/handlers/wallet.rs Show resolved Hide resolved
ska1296
ska1296 suggested changes Sep 1, 2023
View reviewed changes
bundler/src/db/dao/transaction_dao.rs Outdated Show resolved Hide resolved
@vishnukvnvv vishnukvnvv merged commit 022c9b0 into release/choco-chips Sep 1, 2023
1 check passed
@vishnukvnvv vishnukvnvv deleted the feature/polling-transaction-status branch September 1, 2023 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ska1296 ska1296 ska1296 requested changes

@vishnukvnvv vishnukvnvv vishnukvnvv approved these changes

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

Successfully merging this pull request may close these issues.

Polling transaction status
3 participants
@viv-defy @ska1296 @vishnukvnvv