Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: support adding custom konva nodes #242

Merged
merged 8 commits into from
Aug 11, 2024

Conversation

ComfyFluffy
Copy link
Contributor

@ComfyFluffy ComfyFluffy commented Aug 6, 2024

Supporting using custom nodes by pass a customNodes property into options.

A few changes are made:

  • Run Prettier format for the whole project.
  • Get name directly from node constructors.
  • Add an option that takes a new (...args: any) => Node<any> array.
  • Use Node<any> instead of union types.

Related issue: #200

@ComfyFluffy
Copy link
Contributor Author

The changes are yet tested. We will test them soon on on our project.

src/index.ts Outdated Show resolved Hide resolved
@nighca
Copy link

nighca commented Aug 6, 2024

@ComfyFluffy I think it's better to link the related issues (such as #200) in your PR description.

@ComfyFluffy
Copy link
Contributor Author

ComfyFluffy commented Aug 6, 2024

We have tested the changes in our project and updated the docs.

@nighca
Copy link

nighca commented Aug 9, 2024

@lavrton Plz have a look on this PR.

@lavrton lavrton merged commit c71fc3d into konvajs:master Aug 11, 2024
1 check passed
@lavrton
Copy link
Member

lavrton commented Aug 11, 2024

Released as 3.1.0. Thanks for the good PR. Let me know how it works.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants