doc: Document additional parametrization of serializers `#[near(serializers=[borsh(…), json(…)])]`

[frol]

#1158 Implementation is not documented and as such is not discoverable

Let’s extend the reference documentation https://docs.rs/near-sdk/latest/near_sdk/attr.near.html#nearserializers-annotates-structsenums

[Manuelshub]

Could I be assigned to this?

[frol]

@Manuelshub sure, you are welcome. Just note that extra parameters are only supported for borsh, but not json, so you may either document this behavior and suggest to use #[serde(…)] explicitly, or add the support for serde params passed via json element of the list and document both borsh and serde the same way

[Manuelshub]

@frol Hey thanks, would it be possible to get more context regarding this issue.

[Manuelshub]

@frol Most of the changes from the #1158 is not reflecting in the Master branch, please I'll need you to look into it. Thank you.

[Manuelshub]

@dj8yfo @frol I would love to continue working on the issue, is it still viable for me to work on??

[frol]

@Manuelshub Here is the current documentation: https://docs.rs/near-sdk/latest/near_sdk/attr.near.html#nearserializers-annotates-structsenums and it does not cover the fact that you can pass parameters to borsh(...) serializer, like #[near(serializers=[borsh(use_discriminant=false)]).

https://github.com/PolyProgrammist/near-sdk-rs/blob/da51b6556f134f72b4dbdc8a43ee5bf015b5e53d/near-sdk/src/types/public_key.rs#L7

Borsh macro does not support multiple instances like:

#[derive(BorshSerialize)]
#[borsh(crate="borsh")]
#[borsh(use_discriminant=true)]

It requires all the params to be specified at once:

#[derive(BorshSerialize)]
#[borsh(crate="borsh", use_discriminant=true)]

Since #[near(serializers=[borsh])] adds #[borsh(crate="near_sdk::borsh")], we had to extend the support for #[near(serializers=[borsh(...)])] to support passing through the attributes (there must be just a link to the list of all supported attributes https://docs.rs/borsh/latest/borsh/derive.BorshSerialize.html#attributes).

Serde macro supports multiple #[serde(...)] attributes, so we didn't need the dedicated support, but it makes sense to hint that it is there and devs can just use #[serde()] with attributes without a need to #[derive(serde::Serialize)], instead they just need to have #[near(serializers=[json])]. Example in the real life: https://github.com/NEAR-DevHub/neardevhub-contract/blob/9bb94872a32e6f7a0769e031726cbf8e6cc4801a/src/rfp/timeline.rs#L5