-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature: Sync with meilisearch in bun runtime #223
Feature: Sync with meilisearch in bun runtime #223
Conversation
Great PR, some small changes required! |
Made the changes, sorry for so many PR review requests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One final query :D 🚀
"path": "^0.12.7", | ||
"url": "^0.11.3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need these?
We want to do things the bun way
Hey there! There were a lot of big PRs during this Hacktoberfest, and we wanted to give everyone ample time to collaborate with our engineering team. If you were able to merge your PRs during October, amazing. If it’s still not merged, don’t worry about it either. Either way, we’ve got your Hacktoberfest swag minted and ready to ship. Please comment with your Discord username here so we can contact you about your shipping information to deliver your Hacktoberfest swag. |
Hey, my discord id is fresauce |
What does this PR do?
Adds a template for Sync With Meilisearch function in bun runtime.
Test Plan
The template demo is deployed here: https://651ef5e34237d8219e7e.appwrite.global/
Test video: https://drive.google.com/file/d/1Zs4uDsRY-XEvX84smaHI3MjGdfYBTEwF/preview
Related PRs and Issues
This is related to Issue #186
Have you read the Contributing Guidelines on issues?
Yes