cybercongress / cw-cyber

core semantic libs
11 stars 6 forks source link

Update to v1.0.0-beta #1

Closed cyborgshead closed 2 years ago

cyborgshead commented 2 years ago

Issue with custom querier during update to latest cosmwasm v1.0.0

https://github.com/cybercongress/cw-cyber/blob/main/packages/cyber-std/src/query.rs#L10

 pub struct CyberQueryWrapper {
    pub route: CyberRoute,
    pub query_data: CyberQuery,
}
impl CustomQuery for CyberQueryWrapper{}

And queries (changed from custom_query to query during upgrade) https://github.com/cybercongress/cw-cyber/blob/main/packages/cyber-std/src/querier.rs#L32

Now, it's built with error:

   --> packages/cyber-std/src/querier.rs:191:72
    |
191 |         let res: NeuronBandwidthResponse = self.querier.query(&request.into())?;
    |                                                                        ^^^^ the trait `From<CyberQueryWrapper>` is not implemented for `QueryRequest<cosmwasm_std::Empty>`
    = help: the following implementations were found:
              <QueryRequest<C> as From<BankQuery>>
              <QueryRequest<C> as From<C>>
              <QueryRequest<C> as From<WasmQuery>>
    = note: required because of the requirements on the impl of `Into<QueryRequest<cosmwasm_std::Empty>>` for `CyberQueryWrapper`

cosmwasm-std' CustomQuery: https://github.com/CosmWasm/cosmwasm/blob/main/packages/std/src/query/mod.rs#L81

impl<C: CustomQuery> From<C> for QueryRequest<C> {
    fn from(msg: C) -> Self {
        QueryRequest::Custom(msg)
    }
}