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

fix: transaction ws notifications #1525

Merged

mathnogueira merged 2 commits into main from fix/transaction-ws-notifications

Nov 17, 2022

Member

mathnogueira commented Nov 17, 2022

This PR makes websocket notifications send trigger info and test result.

Checklist

tested locally
added new dependencies
updated the docs
added a test

mathnogueira added 2 commits

November 17, 2022 15:27


          include trigger and assertion information on notification

e74e772


          fix result notification

747d46f

xoscar approved these changes

View reviewed changes

mathnogueira merged commit 744eda9 into main

mathnogueira deleted the fix/transaction-ws-notifications branch

November 17, 2022 19:07

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment