Skip to content

Add TypeScript support with ts folder#94

Open
Tomkoooo wants to merge 1 commit intoewngs:masterfrom
Tomkoooo:add-ts-support
Open

Add TypeScript support with ts folder#94
Tomkoooo wants to merge 1 commit intoewngs:masterfrom
Tomkoooo:add-ts-support

Conversation

@Tomkoooo
Copy link
Copy Markdown

Had some issue using the package on ts based projekt so I added typesafety with creating all classes in ts.
Also published szamlazz.ts with the compiled version on npm so it can be used as now.

@ert78gb
Copy link
Copy Markdown
Collaborator

ert78gb commented Apr 22, 2025

Thank you for your effort but I don't plan to convert the project to typescript. The type definitions are more than welcome.

@Tomkoooo
Copy link
Copy Markdown
Author

Thanks for the reply.
I hope its not bothering that I re-released the package with typescript base under the name of szamlazz.ts.
But if you need I can fork the type definitions so whose in need can use the stable package with correct typescript support.

Should I do an index.d.ts file with ever type declaration in one place or one file for every class e.g. buyer.d.ts, invoice.d.ts etc.

@ert78gb
Copy link
Copy Markdown
Collaborator

ert78gb commented Apr 23, 2025

I hope its not bothering that I re-released the package with typescript base under the name of szamlazz.ts.

Absolutely not. Feel free to fork and reorganise.

Should I do an index.d.ts file with ever type declaration in one place or one file for every class e.g. buyer.d.ts, invoice.d.ts etc.

I will add the type definitions today or tomorrow because it is the most wanted feature request. Less effort from me if I do it.

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.

2 participants