h-REA / hREA

A ValueFlows / REA economic network coordination system implemented on Holochain and with supplied Javascript GraphQL libraries
https://docs.hrea.io
Other
142 stars 15 forks source link

HC bump 156 #365

Closed weswalla closed 1 year ago

weswalla commented 1 year ago

There are a whole bunch of trait bound error:

error[E0271]: type mismatch resolving `<EN as std::convert::TryFrom<AvailableCapability>>::Error == std::convert::Infallible`
   --> lib/hdk_semantic_indexes/client/src/lib.rs:476:8
    |
444 | fn request_sync_remote_index<EN, A, B, I, LT, E, E2>(
    |                                               - this type parameter
...
476 |     Ok(call_zome_method::<EN, _, _, _, _, _, _, _>(
    |        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected enum `std::convert::Infallible`, found type parameter `E`
    |
    = note:        expected enum `std::convert::Infallible`
            found type parameter `E`
note: required by a bound in `call_zome_method`
   --> /Users/weswalla/sprillow/hREA/lib/hdk_records/src/rpc_helpers.rs:37:42
    |
37  |         EN: TryFrom<AvailableCapability, Error = E>,
    |                                          ^^^^^^^^^ required by this bound in `call_zome_method`
weswalla commented 1 year ago

@Connoropolous

Was it leaking in from dna-auth-resolver because of the HDK dependency being set to "0.0" there? I expect so.

Yes, exactly.

Connoropolous commented 1 year ago

I will propose to Pospi that we specify the dep version over in dna-auth-resolver. This issue occurs regularly

weswalla commented 1 year ago

@Connoropolous @pospi I forked dna-auth-resolver (https://github.com/weswalla/dna-auth-resolver) as a temp fix to see if it will compile, which is does.

Connoropolous commented 1 year ago

for details on they 'why chrono' thing, see this commit: https://github.com/holochain/holochain/pull/1527/files

weirdly, you have to "disable" the wasm-bindgen feature by disabling all default features and then only enabling the ones that aren't bindgen. bindgen messes with the wasm in ways that breaks the holochain host/guest WASM interface

Connoropolous commented 1 year ago

we can now run the integration tests