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

Merge feat/next into main (v13.1.0) #445

Merged
merged 27 commits into from
Apr 18, 2024
Merged

Merge feat/next into main (v13.1.0) #445

merged 27 commits into from
Apr 18, 2024

Conversation

andreibancioiu
Copy link
Contributor

@andreibancioiu andreibancioiu commented Apr 18, 2024

🛠️ ❗ In TokenManagementTransactionsFactory, added missing role ESDTTransferRole on fungible token (by @popenta in #437). Made in mandatory (by @popenta in #440). This shouldn't be considered a breaking change. It's a minor (trivial) fixing change.

🔵 Implemented AccountTransactionsFactory (by @popenta in #439).

🔵 Improved SmartContractTransactionsFactory: in createTransactionForUpgrade(), use the upgrade definition, if available (by @andreibancioiu in #438).

❗ Analyze serialization of variadic arguments (v12 vs v13) (by @andreibancioiu in #441). Documented a previously unforeseen minor breaking change (when migrating from v12 to v13).

✍️ Formatting part I (by @popenta in #442).

✍️ Formatting part II (by @popenta in #443).

✍️ Formatting part III (by @popenta in #444).

popenta and others added 27 commits April 15, 2024 16:03
Added missing role(ESDTTransferRole) on fungible token
In "createTransactionForUpgrade()", use the "upgrade" definition (if available)
Formatting for release v13.1.0 part II
Analyze serialization of variadic arguments (v12 vs v13)
@andreibancioiu andreibancioiu marked this pull request as ready for review April 18, 2024 11:56
@andreibancioiu andreibancioiu merged commit b877bbe into main Apr 18, 2024
1 check passed
@andreibancioiu andreibancioiu added the ignore-for-release-notes Ignore for release notes label Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-for-release-notes Ignore for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants