Skip to content

Merge a prefix of #960#974

Merged
str4d merged 15 commits into
mainfrom
zsa1-pre-burn-move
Feb 4, 2025
Merged

Merge a prefix of #960#974
str4d merged 15 commits into
mainfrom
zsa1-pre-burn-move

Conversation

@nuttycom
Copy link
Copy Markdown
Contributor

@nuttycom nuttycom commented Feb 4, 2025

No description provided.

vivek-arte and others added 15 commits November 11, 2024 09:46
This merges in the changes from upstream post the merging of our PR#854
to upstream.
This makes a few changes to the usage of terminology such as the use
of the capital MUST terms in the specification.
…ock boundary (#81)

This completes the changes needed to make sure the finalization comes
into effect immediately. A part of this was done in
QED-it#75, the remainder is covered here.

This solves #958.
This removes the tables with the encoding of the issuance action
description and issuance bundle from ZIP 227. This same material has
been included in ZIP 230 as part of the V6 transaction format, and thus
it is simpler to have it defined in a single place.

ZIP 227 has been updated with references to the relevant sections of ZIP
230 where necessary.
This changes the encoding of the transaction format to add Action
Groups to TxV6.
The motivation is to avoid the need of a further transaction format
upgrade while introducing Asset Swaps for ZSAs in a subsequent upgrade.
It also makes some consistency improvements to the ZIPs, and updates
outdated references.
This PR adds an explicit upper bound of $$2^{64} - 1$$ to the total
supply of issued assets in the specification of ZIP 227.

It also does a major rewrite of the consensus rule changes, bringing
them more in line with the protocol specification, and using MUST
terminology where necessary.

Furthermore, these changes make explicit the order of processing fields
within a transaction, and the order of updates to the issuance state.
This PR moves the description of the fees required for OrchardZSA
issuance and transfer to ZIP 227. This is a more contextual place to
have the material, compared to ZIP 230, which specifies the V6
transaction format.
This updates the `zsa1` branch with the changes from upstream, and
resolves conflicts.
This adds the explicit specification (with types) for the Issue Note.
This defines a new function and uses it to derive the value of the
$\rho$ field in Issue Notes. It also updates the consensus rules
accordingly and makes some typographical improvements.
This simplifies the encoding of the $\mathsf{nf}$ argument in the
computation of $\rho$, so that it avoids the double use of
$\mathsf{LEBS2OSP(I2LEOSP(...))}$.
This adjusts the type and specification of the notes to mirror the
implementation, specifically that the Asset Base occurs in the middle of
the tuple, and not at the end.
This adds the specification of reference notes, which are to be
created at the time of first issuance of a Custom Asset. Alongside, we
also update the issuance state and consensus rules to account for these
changes.
This makes various changes based on the comments on #960 made
during ZIP Editor syncs.
Copy link
Copy Markdown
Collaborator

@daira daira left a comment

Choose a reason for hiding this comment

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

ACK

@daira daira changed the title Merge a prefix of #950 Merge a prefix of #960 Feb 4, 2025
Copy link
Copy Markdown
Collaborator

@str4d str4d left a comment

Choose a reason for hiding this comment

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

ACK b5b434c, which is the correct commit from #960 that we wish to merge.

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