Only LSB `BitVec` is currently supported, found `bitvec::order::Msb0`

polkadot-js / api

Promise and RxJS APIs around Polkadot and Substrate based chains via RPC calls. It is dynamically generated based on what the Substrate runtime provides in terms of metadata.

Apache License 2.0

1.07k stars 353 forks source link

Only LSB `BitVec` is currently supported, found `bitvec::order::Msb0` #5588

Open AurevoirXavier opened 1 year ago

AurevoirXavier commented 1 year ago

I'm submitting a ...

[ ] Bug report
[ ] Feature request
[X] Support request
[ ] Other

What is the current behavior and expected behavior?

We cannot connect our chain anymore.

Due to https://github.com/polkadot-js/api/pull/5585.

AFAIK, this change will affect any chain which uses parity-bridges-common.

Please tell us about your environment:

https://polkadot.js.org/apps/?rpc=wss%3A%2F%2Fdarwiniacrab-rpc.dwellir.com#/explorer

jacogr commented 1 year ago

Umpf. Will revert that specific throw and turn it into a warning as a start.

jacogr commented 1 year ago

We are back to where we were, aka not-failing. However keeping this open so we can properly address this TODO so the toHuman & toBoolArray formatting can work as expected -

https://github.com/polkadot-js/api/blob/8fe02a14345b57e6abb8f7f2c2b624cf70c51b23/packages/types/src/metadata/PortableRegistry/PortableRegistry.ts#L733-L746