Skip to content

Commit

Permalink
Fix apps-routing for staking (Westend) (#10394)
Browse files Browse the repository at this point in the history
* Fix apps-routing for staking (Westend)

* comment
  • Loading branch information
TarikGul authored Mar 25, 2024
1 parent b79463b commit 0cf860f
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions packages/apps-routing/src/staking.ts
Original file line number Diff line number Diff line change
Expand Up @@ -45,15 +45,25 @@ function needsApiCheck (api: ApiPromise): boolean {
return false;
}

// For compatibility - `api.query.staking.ledger` returns `legacyClaimedRewards` instead of `claimedRewards` as of v1.4
try {
const v = api.registry.createType<PalletStakingStakingLedger>(
unwrapStorageType(api.registry, api.query.staking.ledger.creator.meta.type),
{ claimedRewards: [1, 2, 3] }
);

assert((v as unknown as { claimedRewards: Vec<u32> }).claimedRewards.eq([1, 2, 3]), 'Needs a claimedRewards array');
if ((v as unknown as { claimedRewards: Vec<u32> }).claimedRewards) {
assert((v as unknown as { claimedRewards: Vec<u32> }).claimedRewards.eq([1, 2, 3]), 'Needs a claimedRewards array');
} else {
const v = api.registry.createType<PalletStakingStakingLedger>(
unwrapStorageType(api.registry, api.query.staking.ledger.creator.meta.type),
{ legacyClaimedRewards: [1, 2, 3] }
);

assert(v.legacyClaimedRewards.eq([1, 2, 3]), 'Needs a legacyClaimedRewards array');
}
} catch {
console.warn('No known claimedRewards inside staking ledger, disabling staking route');
console.warn('No known legacyClaimedRewards or claimedRewards inside staking ledger, disabling staking route');

return false;
}
Expand Down

0 comments on commit 0cf860f

Please sign in to comment.