Skip to content
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.

Sign up for GitHub

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

Jump to bottom

function to wipe-appwrite-collection #64

Merged
merged 2 commits into from
Jan 23, 2023
Merged

function to wipe-appwrite-collection #64

merged 2 commits into from
Jan 23, 2023

Conversation

kritikash18
Copy link

@kritikash18 kritikash18 commented Oct 8, 2022 •
edited by stnguyen90
Loading

@Meldiron
Copy link
Contributor

@kritikash18 , thanks for the PR! 🤯 Please give us some time to review it.

Meldiron
Meldiron requested changes Oct 10, 2022
View reviewed changes
Copy link
Contributor

@Meldiron Meldiron left a comment

Choose a reason for hiding this comment

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

Took a quick peek, please remove the screenshots from the commits 🙏 There are purely for the PR review process so we can trust the function executed successfully. You should include them in your PR description, but no need to include them in the repository.

@kritikash18
Copy link
Author

@Meldiron removed the screenshots from README

@Meldiron
Copy link
Contributor

Thank you so much for the PR 🤩. We're adding the hacktoberfest-accepted label to ensure this PR counts towards your Hacktoberfest contributions count. With that said, please stay active on this PR to address any comments once you receive a review. Happy Hacktoberfest! 🎃

@Meldiron
Copy link
Contributor

Thanks for the amazing PR 😇 I have added small changes implement pagination and improve documentation.
Merging the PR now 🥳

@Meldiron Meldiron merged commit 186b160 into open-runtimes:main Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Meldiron Meldiron Meldiron approved these changes

Assignees
No one assigned
Labels
hacktoberfest-accepted
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

⚡ Write a wipeAppwriteCollection() Function using Node
2 participants
@kritikash18 @Meldiron