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

refactor(sdk): Refactor transferRelayToPara function 🔧 #536

Merged
merged 2 commits into from
Nov 25, 2024

Conversation

michaeldev5
Copy link
Collaborator

@michaeldev5 michaeldev5 commented Nov 25, 2024

Closes #462
Closes #459

@michaeldev5 michaeldev5 self-assigned this Nov 25, 2024
Copy link

codecov bot commented Nov 25, 2024

Bundle Report

Changes will decrease total bundle size by 1.44kB (-0.24%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
sdk-cjs-esm 446.62kB 1.44kB (-0.32%) ⬇️

Copy link

codecov bot commented Nov 25, 2024

Codecov Report

Attention: Patch coverage is 99.13420% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
packages/sdk/src/papi/PapiApi.ts 60.00% 2 Missing ⚠️
packages/sdk/src/pjs/PolkadotJsApi.ts 50.00% 2 Missing ⚠️
Flag Coverage Δ
visualizator-be 100.00% <ø> (ø)
xcm-analyser 98.60% <ø> (ø)
xcm-api 96.60% <ø> (ø)
xcm-router 92.37% <ø> (ø)
xcm-sdk 95.86% <99.13%> (+0.67%) ⬆️
Files with missing lines Coverage Δ
packages/sdk/src/api/IPolkadotApi.ts 100.00% <ø> (ø)
packages/sdk/src/nodes/ParachainNode.ts 98.39% <100.00%> (+0.03%) ⬆️
...k/src/nodes/polkadotXcm/PolkadotXCMTransferImpl.ts 100.00% <100.00%> (ø)
packages/sdk/src/nodes/supported/Acala.ts 100.00% <100.00%> (ø)
packages/sdk/src/nodes/supported/Altair.ts 95.83% <100.00%> (ø)
packages/sdk/src/nodes/supported/Amplitude.ts 90.00% <100.00%> (ø)
packages/sdk/src/nodes/supported/AssetHubKusama.ts 68.51% <100.00%> (+8.51%) ⬆️
...ckages/sdk/src/nodes/supported/AssetHubPolkadot.ts 98.86% <100.00%> (-0.02%) ⬇️
packages/sdk/src/nodes/supported/Astar.ts 86.20% <100.00%> (ø)
packages/sdk/src/nodes/supported/Bajun.ts 100.00% <100.00%> (ø)
... and 73 more

... and 2 files with indirect coverage changes

@michaeldev5 michaeldev5 marked this pull request as ready for review November 25, 2024 19:34
@dudo50 dudo50 merged commit 4d49456 into main Nov 25, 2024
14 checks passed
@dudo50 dudo50 deleted the refactor/relay-to-para branch November 25, 2024 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants