NFT supply

sg721 has two addresses that control it. The `minter` which can mint
NFTs, and the `creator` which can update collection metadata and
royalties.

We now set both of those to be the DAO.

…eries

Many of these methods are not needed thanks to Authz.

For example, a DAO may wish to white list a Token Staking contract (to
allow users to stake their tokens in the DAO) or a Merkle Drop contract
(to allow users to claim their tokens).

Osmosis test tube doesn't support metadata queries anymore, we assume
metadata is set properly by the Cosmos SDK and this is well tested
upstream.

…inting or burning

This is intended functionality. If a token is frozen, a DAO needs to
both grant a minter allowance as well as adding the minter to the
allowlist to allow for token transfers when the token is frozen.

The new error message should make next steps clear if this edge case is
encountered.

DAOs may wish to disable the BeforeSendHook at some point, or
potentially set it to a custom cosmwasm contract if they wish to
customize functionality a bit more.

Modifies the SetBeforeSendHook method to allow for this, and adds tests
that it's indeed possible to set it to nil.

These are powerful features, but many may not want them as they can be
abused. DAOs that want these features will have to explicitly enable
them via a governance prop.

The concept of ownership doesn't really make sense for voting contracts.
IMO the owner should always be the DAO.