Skip to content
This repository was archived by the owner on Nov 5, 2024. It is now read-only.

Add log capture functionality to all interactions #170

Merged
merged 2 commits into from
Aug 26, 2022

Conversation

jribbink
Copy link
Contributor

@jribbink jribbink commented Aug 15, 2022

Closes #68

Description

Captures logs for all interactions and appends to result tuple.

For contributor use:

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

@changeset-bot
Copy link

changeset-bot bot commented Aug 15, 2022

🦋 Changeset detected

Latest commit: 062921c

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

This PR includes changesets to release 1 package
Name Type
@onflow/flow-js-testing Minor

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

@jribbink jribbink force-pushed the jribbink/add-capture-logs branch from 1f672a6 to eb8ed91 Compare August 16, 2022 16:17
@jribbink jribbink force-pushed the jribbink/add-capture-logs branch 2 times, most recently from 0048f1a to 9448b49 Compare August 26, 2022 22:51
@jribbink jribbink force-pushed the jribbink/add-capture-logs branch from 9448b49 to db69955 Compare August 26, 2022 23:00
@jribbink jribbink marked this pull request as ready for review August 26, 2022 23:04
@jribbink jribbink requested a review from a team as a code owner August 26, 2022 23:04
Copy link

@gregsantos gregsantos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LFG2M

🍻 was a pleasure @jribbink!!

@jribbink jribbink merged commit da5e666 into master Aug 26, 2022
@jribbink jribbink deleted the jribbink/add-capture-logs branch August 26, 2022 23:29
This was referenced Mar 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add the ability to access logs in sendTransaction
2 participants