Closed turadg closed 3 months ago
I expect others with more react experience could review this more quickly. Without tests, I'd have to pore over each line.
taking myself off the reviewer list.
@turadg LGTM.
UI launches correctly, but shows a dep issue in the logs, after yarn install
, mainly with something in @noble/hashes
:
Trace:
Dependency issues with
@noble/hashes
Thanks for finding. Probably something to do with the patch. If it affects behavior I may have to fork @noble/hashes
to a repo with the built artifacts. Fortunately the package author said there will be a new release around the end of the month.
Extracted from https://github.com/Agoric/dapp-orchestration-basics/pull/10/ and cleaned up to pass CI