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

bump client to v0.34.1 and spec version to 2601 #811

Merged
merged 1 commit into from
Dec 11, 2023
Merged

Conversation

eshaben
Copy link
Contributor

@eshaben eshaben commented Dec 8, 2023

Description

This PR bumps the client version to v0.34.1 and the runtime version to 2601 (not 2602 yet because there is no GitHub release tag for 2602. Checking with Andrea about this)

Goes with CN PR: moonbeam-foundation/moonbeam-docs-cn#372

@eshaben eshaben requested a review from themacexpert December 8, 2023 07:23
@eshaben eshaben added the A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) label Dec 8, 2023
Copy link
Contributor

@themacexpert themacexpert left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@@ -276,7 +276,7 @@ Since you'll be interacting with the `transferMultiasset` function of the X-Toke
!!! note
For an `AccountId32`, `AccountIndex64`, or `AccountKey20`, you have the option of specify a `network` parameter. If you don't specify one, it will default to `None`.

3. Set the destination weight limit to `Unlimited`. In JavaScript, you'll need to set `Unlimited` to `null` (as outlined in the [TypeScript interface for `XcmV3WeightLimit`](https://github.com/moonbeam-foundation/moonbeam/blob/{{ networks.moonbase.spec_version }}/typescript-api/src/moonbase/interfaces/augment-api-tx.ts#L5734){target=_blank}):
3. Set the destination weight limit to `Unlimited`. In JavaScript, you'll need to set `Unlimited` to `null` (as outlined in the [TypeScript interface for `XcmV3WeightLimit`](https://github.com/moonbeam-foundation/moonbeam/blob/{{ networks.moonbase.spec_version }}/typescript-api/src/moonbase/interfaces/augment-api-tx.ts#L3045){target=_blank}):
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a nitpick but in some places we say Set the destWeightLimit to Unlimitedand in other place we spell it out as Set the destination weight limit to Unlimited. It might be good to pick one way and use consistently

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll double-check this in the XCM overhaul and apply any changes there

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Looks like this is already taken care of in the XCM overhaul!

@eshaben eshaben merged commit 26d1945 into master Dec 11, 2023
1 check passed
@eshaben eshaben deleted the eshaben/v0.34.1 branch December 11, 2023 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants