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

2599 oxford2 supports attestation #2775

Merged
merged 8 commits into from
Jan 5, 2024

Conversation

hui-an-yang
Copy link
Collaborator

@hui-an-yang hui-an-yang commented Dec 21, 2023

supported new name attestation along with endorsement(default but deprecated in oxford2)
[breaking change] local-forging now can only forge and parse attestation operation (no endorsement)

closes #2599

Thank you for your contribution to Taquito.

Before submitting this PR, please make sure:

  • Your code builds cleanly without any errors or warnings
  • You have run the linter against the changes
  • You have added unit tests (if relevant/appropriate)
  • You have added integration tests (if relevant/appropriate)
  • All public methods or types have TypeDoc coverage with a complete description, and ideally an @example
  • You have added or updated corresponding documentation
  • If relevant, you have written a first draft summary describing the change for inclusion in Release Notes.

In this PR, please also make sure:

  • You have linked this PR to the issue by putting closes #TICKETNUMBER in the description box (when applicable)
  • You have added a concise description on your changes

Release Note Draft Snippet

If relevant, please write a summary of your change that will be suitable for
inclusion in the Release Notes for the next Taquito release.

Copy link

github-actions bot commented Dec 21, 2023

New packages have been deployed to the preview repository at https://npm.preview.tezostaquito.io/.

Published packages:

npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/

@hui-an-yang hui-an-yang requested a review from dsawali December 22, 2023 18:55
@hui-an-yang hui-an-yang marked this pull request as ready for review December 22, 2023 18:55
@@ -21,7 +21,7 @@ CONFIGS().forEach(
}) => {
const Tezos = lib;
const unrestrictedRPCNode = rpc.endsWith("ecadinfra.com") ? test.skip : test;
const oxfordAndAlpha = ProtoGreaterOrEqual(protocol, Protocols.ProxfordY) ? test : test.skip
const unrestrictedOxfordAndAlpha = ProtoGreaterOrEqual(protocol, Protocols.ProxfordY) ? test : test.skip;
Copy link
Contributor

Choose a reason for hiding this comment

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

could you remind me what unrestricted is for again?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

line24 should rename to oxfordAndAlpha, will update in next commit
answering unrestricted is for bakingRights, endorsingRights, attestationRights that usually are not accessible through public rpc node, but we have made it able to test with rpc url rpc.endsWith("ecadinfra.com")

@hui-an-yang hui-an-yang requested a review from dsawali January 4, 2024 19:22
@hui-an-yang hui-an-yang linked an issue Jan 5, 2024 that may be closed by this pull request
Copy link
Contributor

@dsawali dsawali left a comment

Choose a reason for hiding this comment

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

lgtm!

@hui-an-yang hui-an-yang merged commit 4a0a6e8 into oxford2 Jan 5, 2024
3 of 7 checks passed
@hui-an-yang hui-an-yang deleted the 2599-oxford2-supports-attestation branch January 5, 2024 21:39
dsawali added a commit that referenced this pull request Jan 29, 2024
* 2762 oxford2 branch configuration (#2764)

* ci: initial configuration of oxford2 branch

re #2762

* docs: replaced mumbai ref with oxford2 of rpc_nodes and wallet_api on webstie

* docs: add oxfordnet to wallet_api in next version of website

* ci: update ci script origiante-known-contracts command

* remove tx rollups references in taquito (#2770)

* remove tx rollups references in taquito

* updated unit test

* removed remaining tx rollup references

* 2596 oxford2 update rpc types (#2772)

* feat: updated rpc endpoint changes for oxford2

re #2596

* fix: fix auto-merge mistake by removing tx ref

* test: fix integration test of pendingOperations

* test: removed unnecessory import

* refactor: imporved ux of rpc endpoint of blockOption version

* 2597 remove proof (#2774)

* removed references to originationProof

* trigger checks

* trigger checks

* test commit

* adjusted more tests

* removed remaining origination proof artifacts

* updated smart rollup docs

* adjusted estimates

* 2776 smart rollup origination with whitelist (#2781)

* removed references to originationProof

* trigger checks

* trigger checks

* test commit

* adjusted more tests

* removed remaining origination proof artifacts

* updated smart rollup docs

* adjusted estimates

* feat: supported smartRollupOriginate with new pvmKind and property whitelist in localForger

re #2776

* refactor: addressed comments of naming for clarity

* test: updated contract-estimation-tests-spec.ts assertions

---------

Co-authored-by: Davis Sawali <[email protected]>

* 2599 oxford2 supports attestation (#2775)

* feat: supports attesation along side endorsement

re #2599

* feat: the rest of files of supports attesation along side endorsement

* test: fix protocol logic in integration-tests/rpc-nodes.spec.ts

* test: addressed pr comments of naming

* test: updated contract-estimation-tests.spec.ts assertions

* 2653 oxford2 reintroduce timelock (#2780)

* test: added local-forging timelock test case with chest, chest_key and OPEN_CHEST

re #2653

* test: added integration test originating with chest&chest_key in storage (chest_key unfinished)

* test: updated chestKeyValue in integration-tests

re #2653

* test: removed unneeded comment

* test: address comment to reduce timeout risk

* Oxford2 cleanup (#2791)

* docs: updated to oxford references whereever fits

* feat: updated rpc types and doc changes

* ci: removed nairobi and replace mondaynet with weeklynet

* fix: replaced modanynet as weeklynet

* ci: comment out oxfordbox tests until oxford2 flextesa image releases

* docs: address typedoc comments

* removed newline

* chore(releng) bump version to 19.0.0-beta-RC.0

* chore: satisfy lerna

* chore(releng) bump version to 19.0.0

---------

Co-authored-by: Davis Sawali <[email protected]>
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.

Change endorsement into attestation
2 participants