Skip to content

Conversation

@andrei-marinica
Copy link
Contributor

@andrei-marinica andrei-marinica commented Jan 14, 2026

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

Updated the call value payments API docs with the new functions and objects.

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

@andrei-marinica andrei-marinica changed the base branch from main to development January 14, 2026 11:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the smart contract payments API documentation to reflect the new unified payment handling model introduced in recent framework versions. The update replaces legacy EGLD/ESDT separation patterns with a unified Payment type approach.

Changes:

  • Updated to reflect that EGLD and ESDT can now be sent together (since Spica release)
  • Added comprehensive documentation for new payment types (Payment<A>, PaymentVec<A>)
  • Documented new call value methods (all(), single(), single_optional(), array())
  • Moved old payment methods to a "Legacy" section with deprecation notes
  • Fixed spelling error: "extrernally" → "externally"

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@andrei-marinica andrei-marinica marked this pull request as ready for review January 15, 2026 18:25
@andrei-marinica andrei-marinica merged commit 1f940dd into development Jan 16, 2026
3 checks passed
@andrei-marinica andrei-marinica deleted the new-payments branch January 16, 2026 13:33
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.

4 participants