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

Quebec #3068

Open
wants to merge 44 commits into
base: master
Choose a base branch
from
Open

Quebec #3068

wants to merge 44 commits into from

Conversation

hui-an-yang
Copy link
Collaborator

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.

…vider for user query old block

BREAKING CHANGE: PollingSubscribeProvider doesn't filter endorsement anymore and removed rpc
entrypoints using param version  0

re #3036
…oved type PendingOperationsV1

BREAKING CHANGE: PendingOperationsQueryArguments.version used to accept 1 is removed for protocol q

re #3034
Copy link

github-actions bot commented Oct 18, 2024

A new deploy preview is available on Cloudflare Pages at https://11bb04aa.taquito-test-dapp.pages.dev

Copy link

github-actions bot commented Oct 18, 2024

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/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/

Copy link

cloudflare-workers-and-pages bot commented Oct 18, 2024

Deploying taquito with  Cloudflare Pages  Cloudflare Pages

Latest commit: 797bfc9
Status: ✅  Deploy successful!
Preview URL: https://f66b3789.taquito.pages.dev
Branch Preview URL: https://quebec.taquito.pages.dev

View logs

@hui-an-yang hui-an-yang changed the base branch from betanet0829 to master November 14, 2024 18:16
@hui-an-yang hui-an-yang changed the base branch from master to betanet0829 November 14, 2024 18:16
@github-actions github-actions bot requested a deployment to taquito-test-dapp (Preview) December 3, 2024 22:34 Abandoned
@hui-an-yang hui-an-yang marked this pull request as ready for review December 7, 2024 01:24
@hui-an-yang hui-an-yang changed the base branch from betanet0829 to master December 7, 2024 01:24
env:
# Ternary operator workaround
TEZOS_RPC_${{ matrix.testnet_uppercase }}: ${{ github.event.pull_request.head.repo.fork && format('https://{0}.ecadinfra.com', matrix.testnet) || null }}
TEZOS_RPC_${{ matrix.testnet_uppercase }}: ${{ github.event.pull_request.head.repo.fork && matrix.rpc_url || null }}
# integration-tests-quebec-secret-key:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is this commented out?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I only use secret key script to develop when there is no keygen support yet in a new protocol, find it handy to have the script around but not necessary. I don't mind removing it if it's cleaner :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

I feel it cleaner to remove it, and we can always find it in the git history, but it is up to you :)

apps/taquito-test-dapp/src/config.ts Outdated Show resolved Hide resolved
cspell.json Show resolved Hide resolved
@@ -8,120 +8,122 @@ author: Roxane Letourneau
1. The RPC nodes' integration tests are disabled by default.
Remove `./rpc-nodes.spec.ts` from `"testPathIgnorePatterns"` in the package.json.

**parisnet**: `npm run test:parisnet rpc-nodes.spec.ts`
**parisnet**: `npm run test:quebecnet-secret-key integration-tests/__tests__/rpc/nodes.spec.ts`
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should parisnet be replaced with quebecnet?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes will update this, thank you!

packages/taquito-rpc/src/rpc-client-interface.ts Outdated Show resolved Hide resolved
@@ -2385,14 +2388,15 @@ export type LastRemovedCommitmentHashes = {
};

export interface PendingOperationsQueryArguments {
version?: '1' | '2';
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we keep '1' to stay compatible with precedent protocol?

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.

2 participants