Skip to content

Conversation

@radumojic
Copy link
Contributor

Description of the pull request (what is new / what has changed)

Did you test the changes locally ?

  • yes
  • no

Which category (categories) does this pull request belong to?

  • document new feature
  • update documentation that is not relevant anymore
  • add examples or more information about a component
  • fix grammar issues
  • other

vinibarbosabr and others added 5 commits February 15, 2025 11:55
…et-owner.pem"`

When following the tutorial, I wasn't able to deploy the sc because the program was panicking when not finding the "/wallet-owner.pem".

The fix was adding the absolute path, completing with the missing directories above "/ping-pong/wallet/wallet-owner.pem".

I added a comment at line 178, suggesting this fix for devs following the tutorial.
Added a paragraph after «(...) by automated processes.» and before «The **nodes** (...)».

Better to distinguish the entities and easier to read.
Add paragraph before «The **nodes** (...)»
Add comment related to `Wallet::from_pem_file("/ping-pong/wallet/wall…
CiprianDraghici
CiprianDraghici previously approved these changes Mar 6, 2025
andreibancioiu
andreibancioiu previously approved these changes Mar 6, 2025
Update docs for relayed transactions
@danielailie danielailie merged commit 0edeadb into main Mar 18, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants