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

Add getTransaction wasm function, to poll for new txs #1260

Merged
merged 30 commits into from
Feb 14, 2025
Merged

Add getTransaction wasm function, to poll for new txs #1260

merged 30 commits into from
Feb 14, 2025

Conversation

hardingjam
Copy link
Contributor

@hardingjam hardingjam commented Feb 10, 2025 •
edited
Loading

Motivation

  • We want to await new transactions arriving in SG after each transaction

Solution

  • Add a wasm function to find transactions by ID
  • Wait for transactions to complete as part of tx flow

Checks

By submitting this for review, I'm confirming I've done the following:

  • made this PR as small as possible
  • unit-tested any new functionality
  • linked any relevant issues or PRs
  • included screenshots (if this involves a front-end change)

@hardingjam hardingjam requested a review from hardyjosh February 10, 2025 15:01
@hardingjam hardingjam self-assigned this Feb 10, 2025
This was linked to issues Feb 13, 2025
Navigate to orders page on webapp after deploying a strat #1253
Closed
Add a wasm binding for a query to get new AddOrder events by their transaction ID #1293
Closed
Order page isn't refetching #1285
Closed
@hardingjam hardingjam force-pushed the 02/10/25-Add-getTransaction-wasm-fn branch from 439c3c5 to a654c20 Compare February 13, 2025 15:16
@hardingjam hardingjam mentioned this pull request Feb 13, 2025
Merged
4 tasks
@hardingjam hardingjam removed a link to an issue Feb 14, 2025
Add a wasm binding for a query to get new AddOrder events by their transaction ID #1293
Closed
@hardingjam hardingjam enabled auto-merge February 14, 2025 20:39
@hardingjam hardingjam merged commit e776932 into main Feb 14, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hardyjosh hardyjosh hardyjosh approved these changes

Assignees

@hardingjam hardingjam

Labels
webapp
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Order page isn't refetching Navigate to orders page on webapp after deploying a strat
2 participants
@hardingjam @hardyjosh