search

Xahau / xahaud

Codebase for Xahaud - The consensus, RPC & blockchain app for the Xahau network.
https://xahau.network
ISC License
22 stars 11 forks source link

Change "enable this flag" for "disable this flag" asfAuthorizedNFTokenMinter documentation #305

Closed JosepMariaVila closed 3 months ago

JosepMariaVila commented 3 months ago

https://docs.xahau.network/technical/protocol-reference/transactions/transaction-types/accountset Context: "AccountSet flags" section, regarding asfAuthorizedNFTokenMinter:

This text in the documentation:

"asfAuthorizedNFTokenMinter 10 (None) Enable to allow another account to mint non-fungible tokens (NFTokens) on this account's behalf. Specify the authorized account in the NFTokenMinter field of the AccountRoot object. To remove an authorized minter, enable this flag and omit the NFTokenMinter field. (Added by the [NonFungibleTokensV1_1 amendment][].)"

maybe it's better if it says: "...DISABLE this flag and omit the NFTokenMinter field..." because it seems that you send a ClearFlag transaction omiting the NFTokenMinter field, not a SetFlag transaction omiting the NFTokenMinter field.

JosepMariaVila commented 3 months ago

It seems that this flag is not even a thing on Xahau. So I close that.