BUG: `query nft denoms` about gon-evidence HOT 4 OPEN

result output has pagination.next_key. The --page flag is not used here and should be removed.

from gon-evidence.

This is not a bug. Iris has forbidden the use of --page, and you can only use --page-key.

from gon-evidence.

This is not a bug. Iris has forbidden the use of --page, and you can only use --page-key.

Agree, but it then should show an error, like this:

$ iris query block --page 132
Error: unknown flag: --page

--page-key is at a couple of places: like iris query nft denoms and iris query nft collection. If you enter iris query nft denoms --help it shows --page uint pagination page of all denoms to query for. This sets offset to a multiple of limit (default 1).

For devs it might be not a bug, since it just ignores - for users it is, since --page is passed and provides some output and user wonders why it always outputs the same results for different pages :).

from gon-evidence.

As discussed with @taramakage, technically it seems to be irrelevant to ICS721. But here also from a business perspective: if a misleading command like iris query nft collection --page XYZ is not providing all tokens, then as a result, not all tokens can be transferred using ICS721.

We are all techies, but we sometimes need to see what business requirements there are in 1st place 😎!

from gon-evidence.