John Simons

Results 56 comments of John Simons

Hi @dnv-kimbell Thanks for providing more context to this feature request. We acknowledge that, currently, the API is not ideal for your scenario. We will take this into consideration in...

I tried the master version and still failing. I also tried changing the import to `import { graphlib, layout } from "@dagrejs/dagre";` and still no good. The error: ``` src/components/messages/FlowDiagram/useLayout.ts:1:34...

@rustedgrail I tried the latest master, and it seems `index.d.ts` is not defining all the types. So we are close. Here are the latest errors from the build. ``` vite...

Just tried master again, and the only errors now are: ``` vite v7.2.4 building client environment for production... ✓ 1278 modules transformed. src/components/messages/FlowDiagram/useLayout.ts:8:40 - error TS2339: Property 'Graph' does not...

Actually, just read the error better, it seems the latest [change](https://github.com/dagrejs/dagre/commit/ec51f60d5d224fbbbc95689e4388f0620f7b94e8) broke it more than before, sorry.

You may need to add more [JSDoc tags to the code](https://www.typescriptlang.org/docs/handbook/declaration-files/dts-from-js.html) so that the generation of the `d.ts` file includes all the public APIs.