Skip to content

Comments

Parse fees as numbers instead of strings#141

Open
tomhatzer wants to merge 2 commits intoOpen-Trader:devfrom
tomhatzer:feature/parse_external_data_to_numbers
Open

Parse fees as numbers instead of strings#141
tomhatzer wants to merge 2 commits intoOpen-Trader:devfrom
tomhatzer:feature/parse_external_data_to_numbers

Conversation

@tomhatzer
Copy link

As reported in Discord by user SammyDk:

I have the following problem.
Here is the log:

PrismaClientValidationError:
Invalid prisma.order.update() invocation:

{
  where: {
    id: 73
  },
  data: {
    status: "Filled",
    filledPrice: 4.668,
    filledAt: new Date("2025-05-20T21:02:01.706Z"),
    fee: "0.0005"
         ~~~~
  },
  include: {
    smartTrade: true
  }
}

Argument fee: Invalid value provided. Expected Float, NullableFloatFieldUpdateOperationsInput or Null, provided String.
    at En (/home/ubuntu/.local/share/fnm/node-versions/v22.15.1/installation/lib/node_modules/opentrader/node_modules/@prisma/client/runtime/library.js:29:1363)
    at Mn.handleRequestError (/home/ubuntu/.local/share/fnm/node-versions/v22.15.1/installation/lib/node_modules/opentrader/node_modules/@prisma/client/runtime/library.js:121:6981)
    at Mn.handleAndLogRequestError (/home/ubuntu/.local/share/fnm/node-versions/v22.15.1/installation/lib/node_modules/opentrader/node_modules/@prisma/client/runtime/library.js:121:6663)
    at Mn.request (/home/ubuntu/.local/share/fnm/node-versions/v22.15.1/installation/lib/node_modules/opentrader/node_modules/@prisma/client/runtime/library.js:121:6370)
    at async l (/home/ubuntu/.local/share/fnm/node-versions/v22.15.1/installation/lib/node_modules/opentrader/node_modules/@prisma/client/runtime/library.js:130:9633)
    at async OrdersStream.onOrderFilled (file:///home/ubuntu/.local/share/fnm/node-versions/v22.15.1/installation/lib/node_modules/opentrader/dist/chunk-PNY4QK66.mjs:59906:26) {
  clientVersion: '6.0.1'
}

This should fix this issue. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant