node-zendesk icon indicating copy to clipboard operation
node-zendesk copied to clipboard

The TypeScript version seems to be broken, even the examples don't compile

Open ipeychev opened this issue 1 year ago • 4 comments
trafficstars

Describe the Bug The types are broken, the TypeScript code doesn't compile. I took an example from node_modules/node-zendesk/dist/types/clients/core/tickets.d.ts. Note the @example:

* Create a new ticket.
* @example
     * const newTicket = await client.tickets.create({ subject: 'New ticket', description: 'Ticket description' });
     */
    create(ticket: CreateOrUpdateTicket): Promise<{
        result: Ticket;
    }>;

I took this code and created an app:

Example Code

import { createClient } from "node-zendesk";

async function test123() {
  const client = createClient({});
  const newTicket = await client.tickets.create({ subject: 'New ticket', description: 'Ticket description' });
}

Expected Behavior The code to compile successfully.

Actual Behavior There is a compilation error:

Object literal may only specify known properties, and 'subject' does not exist in type 'CreateOrUpdateTicket'.ts(2353)
(property) subject: string

Screenshot 2024-09-10 at 18 32 08

Environment Information

  • node-zendesk version: 5.0.13
  • Node.js version: v20.9.0
  • Operating System: MacOS
  • Any other relevant software versions?

Additional Context

"dependencies": {
    "node-zendesk": "^5.0.13",
    "typescript": "^5.6.2"
  }

Maybe the problem is in the TypeScript version?

ipeychev avatar Sep 10 '24 16:09 ipeychev

I have it too

m-rolana avatar Sep 13 '24 12:09 m-rolana

Try importing this:

import type { CreateOrUpdateTicket } from "node-zendesk/dist/types/clients/core/tickets"

and then adding the newly imported CreateOrUpdateTicket type to the object you're passing to the create function:

await client.tickets.create({ subject: 'New ticket', description: 'Ticket description' } as CreateOrUpdateTicket);

kustomrtr avatar Oct 03 '24 19:10 kustomrtr

Same here

louis49 avatar Oct 04 '24 06:10 louis49

same here, types and signatures do not match for some reason

7c avatar Oct 11 '24 12:10 7c

Me too

michal-boruczkowski avatar Oct 18 '24 12:10 michal-boruczkowski

@blakmatrix Made a PR with a fix https://github.com/blakmatrix/node-zendesk/pull/434 Would appreciate some attention if you have time :pray:

m-rolana avatar Nov 06 '24 07:11 m-rolana