accept retracted XtStatus about substrate-api-client HOT 6 OPEN

example commits (no PR because based on old release):
brenzi@480823e
brenzi@d01bc07

from substrate-api-client.

@brenzi I will have a look at it.
Just to clarify: The two commits from your comment would basically implement this? Or am I missing somthing?

from substrate-api-client.

Yes they would. Happy to submit a PR, but I first wanted to raise the discussion to see if you agree and would merge rhis

from substrate-api-client.

I also gave it some more thought. In general I find the current separation into expected and unexpected status quite arbitrary. Whether they are expected or not depends on the use case and I feel that by making this separation we make a decision that actually the user should take.
In polkadot there is a is_final method: https://github.com/paritytech/polkadot-sdk/blob/74a42cebc1a9fd4e4a7713d5e41caba77a0fa172/substrate/client/transaction-pool/api/src/lib.rs#L161
Maybe we should also use this in our methods. You can wait for some status but once we receive a final status we stop waiting. What do you think?

from substrate-api-client.

That sounds better than my hack commits

from substrate-api-client.

Ok, I will try to come up with a PR. Still need to do some thinking on how to do this. This change could lead to some very subtle breaks of backward compatibility for clients.

from substrate-api-client.