Skip to content

feat(sdk): add fn to wait for message status#2252

Merged
mslipper merged 1 commit intodevelopfrom
sc/wait-for-status
Mar 4, 2022
Merged

feat(sdk): add fn to wait for message status#2252
mslipper merged 1 commit intodevelopfrom
sc/wait-for-status

Conversation

@smartcontracts
Copy link
Contributor

Description
Adds a function to the SDK (waitForMessageStatus) which will wait until
the status of a particular message matches the target status.
waitForMessageStatus also handles certain special cases where different
status messages are incompatible (e.g., the RELAYED and
FAILED_L1_TO_L2_MESSAGE status for L1 to L2 messages). Useful to avoid
including looping logic in your own application.

@changeset-bot
Copy link

changeset-bot bot commented Mar 3, 2022

🦋 Changeset detected

Latest commit: b66e313

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/sdk Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the A-pkg-sdk Area: packages/sdk label Mar 3, 2022
@smartcontracts smartcontracts marked this pull request as ready for review March 3, 2022 20:29
@github-actions github-actions bot added the A-integration Area: integration tests label Mar 3, 2022
@tynes
Copy link
Contributor

tynes commented Mar 3, 2022

Generally looks good to me, but I don't know the codebase super well

Adds a function to the SDK (waitForMessageStatus) which will wait until
the status of a particular message matches the target status.
waitForMessageStatus also handles certain special cases where different
status messages are incompatible (e.g., the RELAYED and
FAILED_L1_TO_L2_MESSAGE status for L1 to L2 messages). Useful to avoid
including looping logic in your own application.
@mslipper mslipper merged commit 14b7c16 into develop Mar 4, 2022
@mslipper mslipper deleted the sc/wait-for-status branch March 4, 2022 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-integration Area: integration tests A-pkg-sdk Area: packages/sdk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants