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(bindings/nodejs): Add more APIs and examples #1799

Merged
merged 5 commits into from
Mar 30, 2023

Conversation

suyanhanx
Copy link
Member

@suyanhanx suyanhanx commented Mar 30, 2023

Add APIs

  • check
  • isExist
  • remove
  • removeAll

Signed-off-by: suyanhanx <suyanhanx@gmail.com>
Signed-off-by: suyanhanx <suyanhanx@gmail.com>
@Xuanwo
Copy link
Member

Xuanwo commented Mar 30, 2023

Maybe we can remove the example file in this PR?

@suyanhanx
Copy link
Member Author

Maybe we can remove the example file in this PR?

Right.

Signed-off-by: suyanhanx <suyanhanx@gmail.com>
Signed-off-by: suyanhanx <suyanhanx@gmail.com>
Signed-off-by: suyanhanx <suyanhanx@gmail.com>
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So nice! Thanks a lot!

@Xuanwo Xuanwo merged commit f199ca0 into apache:main Mar 30, 2023
@suyanhanx suyanhanx deleted the sync-more-api branch March 30, 2023 04:38
# 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.

2 participants