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

Update cookbook for SDK-JS #619

Merged
merged 3 commits into from
Apr 7, 2023
Merged

Update cookbook for SDK-JS #619

merged 3 commits into from
Apr 7, 2023

Conversation

andreibancioiu
Copy link
Contributor

@andreibancioiu andreibancioiu commented Apr 7, 2023

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

Update SDK-JS cookbook for v12. Based on: multiversx/mx-sdk-js-examples#17.

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

@andreibancioiu andreibancioiu changed the base branch from main to development April 7, 2023 10:48
@andreibancioiu andreibancioiu marked this pull request as ready for review April 7, 2023 10:48
@andreibancioiu andreibancioiu self-assigned this Apr 7, 2023
bogdan-rosianu
bogdan-rosianu previously approved these changes Apr 7, 2023
```

<!-- END_INCLUDE_FILE -->

<!-- BEGIN_INCLUDE_FILE { "url": "https://raw.githubusercontent.com/multiversx/mx-sdk-js-examples/next/cookbook/generated/contracts_01_deployments.md" } -->
Copy link
Contributor

Choose a reason for hiding this comment

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

might reference a commit hash instead of the branch name? otherwise, make sure the next branch isn't removed in that repo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added commit hash.


A `TokenPayment` object for **EGLD transfers** (value movements):
A `TokenTransfer` object for **EGLD transfers** (value movements):
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

```

Note that the transaction needs to be signed before broadcasting it. Signing can be achieved using a signing provider.
Note that the transaction **must to be signed before being broadcasted**. Signing can be achieved using a signing provider.
Copy link
Contributor

Choose a reason for hiding this comment

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

must be signed before being broadcasted

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.


[comment]: # (mx-context-auto)

## Token transfers

First, let's create a `TransferTransactionsFactory`.
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

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.

3 participants