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

Release v1.7.14 #55

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Release v1.7.14 #55

wants to merge 5 commits into from

Conversation

sstanculeanu
Copy link
Contributor

No description provided.

@sstanculeanu sstanculeanu self-assigned this Jul 30, 2024
schimih
schimih previously approved these changes Jul 31, 2024

# Contents

This document explains the contents of the rc/v1.7.next1 release codenamed Interim. It is split in 2 sections:

Choose a reason for hiding this comment

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

Codename should be a star name.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Decided with @schimih that this one would be called Interim for now.

Choose a reason for hiding this comment

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

@@ -0,0 +1,124 @@
[comment]: <> (tags/v1.7.16)

Choose a reason for hiding this comment

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

Maybe we should pick a proper semver tag, 1.8 (new features), even if we've mistakenly used 1.7next1 until now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Indeed 1.8 would be the correct choice, considering that features were also added. But we somehow reserved 1.8.0 for Andromeda release, thus only increased the last number.

Relayed v3 feature brings a cheaper, improved version of relayed transactions, that can hold multiple inner transactions.
The inner transactions will be full transactions.

With this feature, a fix on the base cost of relayed transactions was implemented, to follow the minimum gas rule.

Choose a reason for hiding this comment

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

Additionally.

Also, maybe it should be specified that this additional fix affects all types of relayed transactions?


With this feature, a fix on the base cost of relayed transactions was implemented, to follow the minimum gas rule.

### Impact:

Choose a reason for hiding this comment

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

Maybe we should split the impact section into more sections - impact on validator, on developers, on integrators etc.?

Choose a reason for hiding this comment

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

Additional note for integrators that read blockchain data - they need to handle relayed V3 transactions.

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 a note that v1 and v2 will be deprecated, should be pretty clear that this is a new type of tx that must be handled separately


# Features

## 1. Relayed v3 [#5741](https://github.com/multiversx/mx-chain-go/pull/5741)

Choose a reason for hiding this comment

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

Maybe link towards docs, as well?

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. it was not pushed yet when I initially opened the PR.


## 3. Crypto API, new Opcodes and EGLD in MultiESDTTransfer [#6139](https://github.com/multiversx/mx-chain-go/pull/6139)

With this feature, new opcodes were enabled for developers, among with new crypto endpoints(`VerifySecp256r1`, `VerifyBLSSignatureShare`, `VerifyBLSMultiSig`).

Choose a reason for hiding this comment

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

We can specify that these endpoints are VM endpoints / VM EEI additions - since the word "endpoint" is a bit ambiguous.

## 3. Crypto API, new Opcodes and EGLD in MultiESDTTransfer [#6139](https://github.com/multiversx/mx-chain-go/pull/6139)

With this feature, new opcodes were enabled for developers, among with new crypto endpoints(`VerifySecp256r1`, `VerifyBLSSignatureShare`, `VerifyBLSMultiSig`).
Another improvement was EGLD as part of MultiESDTTransfer (EGLD-000000).

Choose a reason for hiding this comment

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

It's not obvious that since this upgrade, one can send native tokens along with custom ones by means of a MultiESDTTransfer.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated + added note

With this feature, new opcodes were enabled for developers, among with new crypto endpoints(`VerifySecp256r1`, `VerifyBLSSignatureShare`, `VerifyBLSMultiSig`).
Another improvement was EGLD as part of MultiESDTTransfer (EGLD-000000).

### Impact:

Choose a reason for hiding this comment

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

Additional note for integrators that read blockchain data - they need to handle EGLD transfers within MultiESDTTransfer.

Comment on lines 86 to 89
* No activation epoch
* No configuration files changes
* No binary flags changes
* No API endpoints changes

Choose a reason for hiding this comment

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

Any changes within the database structure that is worth mentioning?

Alternatively, replace section with a simple statement e.g. no impact on validators / developers etc.?

* There are new enable epochs definitions for this feature, called `EGLDInMultiTransferEnableEpoch` and `CryptoOpcodesV2EnableEpoch`
* There are changes to the configuration options in the gas schedule files, new values were added for `VerifySecp256r1`, `VerifyBLSSignatureShare`,`VerifyBLSMultiSig`
* No binary flags changes
* No API endpoints changes

Choose a reason for hiding this comment

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

In this context, API is ambiguous: VM API vs. HTTP API. Perhaps say Node HTTP API (here and above)?

@@ -65,14 +70,18 @@ The improvements implemented on ESDTs enable the dynamic NFT functionality.

## 3. Crypto API, new Opcodes and EGLD in MultiESDTTransfer [#6139](https://github.com/multiversx/mx-chain-go/pull/6139)

With this feature, new opcodes were enabled for developers, among with new crypto endpoints(`VerifySecp256r1`, `VerifyBLSSignatureShare`, `VerifyBLSMultiSig`).
Another improvement was EGLD as part of MultiESDTTransfer (EGLD-000000).
With this feature, new opcodes were enabled for developers, among with new crypto VM endpoints(`VerifySecp256r1`, `VerifyBLSSignatureShare`, `VerifyBLSMultiSig`).

Choose a reason for hiding this comment

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

Formatting / missing space.

Another improvement was EGLD as part of MultiESDTTransfer (EGLD-000000).
With this feature, new opcodes were enabled for developers, among with new crypto VM endpoints(`VerifySecp256r1`, `VerifyBLSSignatureShare`, `VerifyBLSMultiSig`).

Additionally, EGLD(native tokens) can be now sent within MultiESDTTransfer (token identifier: EGLD-000000) along with custom tokens.

Choose a reason for hiding this comment

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

Formatting / missing space before parenthesis.

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