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.

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

fix: comment on make transactions and add transaction mined period #201

Merged
merged 3 commits into from
Mar 15, 2024

Conversation

0xneves
Copy link
Contributor

@0xneves 0xneves commented Mar 15, 2024

Doc Refactor

  • Changing the comment description of make transaction script on Makefile.
  • Adding wait tx.wait() to wait transactions to be mined to not mess up the script sequence when saving on .env and risking script continuation without the right parameters.

@0xneves 0xneves added the bug Something isn't working label Mar 15, 2024
@0xneves 0xneves self-assigned this Mar 15, 2024
@0xneves 0xneves added documentation Improvements or additions to documentation and removed bug Something isn't working labels Mar 15, 2024
@0xneves 0xneves changed the title fix: wrong comment on make transactions fix: comment on make transactions and add transaction mined period Mar 15, 2024
@0xneves 0xneves merged commit 257d907 into blockful-io:develop Mar 15, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

1 participant