Closed jancionear closed 2 months ago
Let's also add some comments to this page? https://docs.near.org/api/rpc/transactions
Also, we need to give some guidelines what should the user do in this case (Contact the contract owner?)
I'm not sure if it deserves to be on the transactions page. It's a low level error, I don't see anything about other ReceiptValidationErrors
in there. Even basic errors like GasLimitExceeded
aren't there.
@jancionear Sorry for jumping in, but I'd like to invite you to participate in the Race of Sloths - a fun and gamified open-source contributions program. Consider mentioning @race-of-sloths user in your github comment or PR description to join!
See how the flow works here: https://github.com/near/nearcore/pull/11778
My comment is inspired by https://github.com/near/docs/commit/fef1566eec69499eaa23faa993345b13ae771680 I think we need to add at least any comment how should it be handled correctly
My comment is inspired by fef1566
I think there's a big difference between ReceiptSizeExceeded
and ShardCongested
. ReceiptSizeExceeded
belongs to a common category of errors that occur when receipt execution fails because of something. Rpc users already know how to handle errors like this.
ShardCongested
is a whole new category on its own, it doesn't belong to any of the existing categories, so it makes sense that it has a special place in the transactions doc. It's a special case that rpc users now need to take into account.
I agree that it'd be nice to have a detailed description of the error somewhere, but I'm hesitant to put it on the main transactions page.
but I still think it's better to leave additional comment here, right where you already added some lines, with the comment how should the user handle this
I added the longer comment under the existing short one.
Add information about the new
ReceiptValidationError::ReceiptSizeExceeded
error added in https://github.com/near/nearcore/pull/11492.I'm new to this repo, I looked at
ReceiptValidationError::InvalidDataReceiverId
and put information about my error in the same places.I also have a longer description of the error that users could find more useful, but I'm not sure where to put it. Can I replace the comment saying "/// Receipt is bigger than the limit."? Or should these docs stay in sync with the source files?
Longer description:
Refs: zulip conversation