Skip to content

Commit

Permalink
merge main branch
Browse files Browse the repository at this point in the history
  • Loading branch information
paulo-ocean committed Nov 21, 2024
2 parents 94a6619 + fa242a5 commit 89e0573
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 6 deletions.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,14 @@ All notable changes to this project will be documented in this file. Dates are d

Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).

#### [v3.4.4](https://github.com/oceanprotocol/ocean.js/compare/v4.0.0-next.1...v3.4.4)

- Remove checks from publishing. ADded chain ID check for ordering. [`#1875`](https://github.com/oceanprotocol/ocean.js/pull/1875)

#### [v4.0.0-next.1](https://github.com/oceanprotocol/ocean.js/compare/v4.0.0-next.0...v4.0.0-next.1)

> 12 November 2024
- add datasets on ComputeAsset, new start compute fn [`8563429`](https://github.com/oceanprotocol/ocean.js/commit/85634293306fca9aaa3ab5ae06a114149ccc8911)
- support for consumer signature on initialize compute [`71c5923`](https://github.com/oceanprotocol/ocean.js/commit/71c59230db99a08a6c28df66c26e32ca61c71089)
- add file object types [`cd936c2`](https://github.com/oceanprotocol/ocean.js/commit/cd936c24a989633d3d8e71b908a375802d2e2970)
Expand Down
1 change: 0 additions & 1 deletion src/@types/DDO/DDO.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import { Service, Metadata, Credentials, Event } from '..'

/**
* DID Descriptor Object.
* Contains metadata about the asset, and define access in at least one service.
Expand Down
9 changes: 4 additions & 5 deletions src/utils/Assets.ts
Original file line number Diff line number Diff line change
Expand Up @@ -154,11 +154,10 @@ export async function createAsset(
}
const chainID = (await owner.provider.getNetwork()).chainId

if (ddo.chainId !== chainID) {
throw new Error('Chain ID from DDO is different than the configured network.')
}
if (ddo.id) {
throw new Error('DID already filled in.')
if (ddo.chainId) {
if (ddo.chainId !== chainID) {
throw new Error('Chain ID from DDO is different than the configured network.')
}
}

const config = new ConfigHelper().getConfig(parseInt(String(chainID)))
Expand Down
4 changes: 4 additions & 0 deletions src/utils/OrderUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,10 @@ export async function orderAsset(
consumeMarketFeeToken:
asset.stats.price.tokenAddress || '0x0000000000000000000000000000000000000000'
}
const chainID = (await consumerAccount.provider.getNetwork()).chainId
if (asset.chainId !== chainID) {
throw new Error('Chain ID from DDO is different than the configured network.')
}

if (!asset.datatokens[datatokenIndex].address)
throw new Error(
Expand Down
4 changes: 4 additions & 0 deletions test/integration/helpers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -133,6 +133,10 @@ export async function handleComputeOrder(
- have validOrder and providerFees -> then order is valid but providerFees are not valid, we need to call reuseOrder and pay only providerFees
- no validOrder -> we need to call startOrder, to pay 1 DT & providerFees
*/
const chainID = (await payerAccount.provider.getNetwork()).chainId
if (config.chainId !== chainID) {
throw new Error('Chain ID from DDO is different than the configured network.')
}
const hasProviderFees = order.providerFee && order.providerFee.providerFeeAmount
if (hasProviderFees && Number(order.providerFee.providerFeeAmount) > 0) {
await approveWei(
Expand Down

0 comments on commit 89e0573

Please sign in to comment.