Is ownership(deps: Deps) really needed?

[taitruong]

From what I see, ownership(deps: Deps) is only used in one place (query):

cw-nfts/contracts/cw721-base/src/query.rs

Lines 321 to 336 in 177a993

	QueryMsg::Ownership {} => to_binary(&Self::ownership(deps)?),
	QueryMsg::Extension { msg: _ } => Ok(Binary::default()),
	}
	}
	pub fn minter(&self, deps: Deps) -> StdResult<MinterResponse> {
	let minter = cw_ownable::get_ownership(deps.storage)?
	.owner
	.map(|a| a.into_string());
	Ok(MinterResponse { minter })
	}
	pub fn ownership(deps: Deps) -> StdResult<cw_ownable::Ownership<Addr>> {
	cw_ownable::get_ownership(deps.storage)
	}

So do we need to make this public? If so, maybe it is better re-exporting from ownable, like this:

// re-export to make it easier for anyone using this, without using ownable directly
pub use cw_ownable::get_ownership;

[taitruong]

After checking it again, this is a bit confusing:

cw-nfts/contracts/cw721-base/src/msg.rs

Line 155 in 177a993

Minter {},

Since v0.17.0 cw-ownable replaces minter. So QueryMsg for Minter and Ownership is the same thing. So either minter should be noted as deprecated or one of these 2 should be removed.