Skip to content

Commit

Permalink
Add a note under TODO comment
Browse files Browse the repository at this point in the history
To explain the reason for this to stay.

Also improved a NetworkId arg haddock.
  • Loading branch information
ffakenz committed Jan 3, 2024
1 parent fbc9110 commit b3fd478
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
3 changes: 1 addition & 2 deletions hydra-node/src/Hydra/Chain/CardanoClient.hs
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,7 @@ data CardanoClient = CardanoClient
mkCardanoClient :: NetworkId -> SocketPath -> CardanoClient
mkCardanoClient networkId nodeSocket =
CardanoClient
{ queryUTxOByAddress = \addresses -> do
queryUTxO networkId nodeSocket QueryTip addresses
{ queryUTxOByAddress = queryUTxO networkId nodeSocket QueryTip
, networkId
}

Expand Down
2 changes: 2 additions & 0 deletions hydra-node/src/Hydra/Chain/Direct.hs
Original file line number Diff line number Diff line change
Expand Up @@ -316,6 +316,8 @@ chainSyncClient handler wallet startingPoint =
pure clientStIdle
BlockInMode _ block ConwayEraInCardanoMode -> do
-- TODO: uses cardano-api:internal
-- NOTE: we should remove this dependency once we have ShelleyBlock available
-- on the normal cardano-api library.
let (ShelleyBlock ShelleyBasedEraConway conwayBlock) = block
-- XXX: We should not be needing to wrap / unwrap in addition. We
-- just found those functions to satisfy the types.
Expand Down
3 changes: 2 additions & 1 deletion hydra-node/src/Hydra/Chain/Direct/ScriptRegistry.hs
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,8 @@ registryUTxO scriptRegistry =
--
-- Can throw at least 'NewScriptRegistryException' on failure.
queryScriptRegistry ::
-- | Current network discriminant
-- | cardano-node's network identifier.
-- A combination of network discriminant + magic number.
NetworkId ->
-- | Filepath to the cardano-node's domain socket
SocketPath ->
Expand Down

0 comments on commit b3fd478

Please sign in to comment.