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.

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

Added wipeAppwriteBucket() Function using PHP #53

Merged
merged 4 commits into from
Jan 23, 2023
Merged

Added wipeAppwriteBucket() Function using PHP #53

merged 4 commits into from
Jan 23, 2023

Conversation

pikami
Copy link
Contributor

@pikami pikami commented Oct 5, 2022

Summary

This PR adds an example for PHP to wipe all files in an Appwrite bucket.

Screens

Bucket with existing files
screen-sel-221005-2228-06
Executing function with payload
screen-sel-221005-2228-42
Function response
screen-sel-221005-2228-51
Bucket after executing the function
screen-sel-221005-2228-59
Function response when passing a payload without bucket id
screen-sel-221005-2241-14

Ticket Link

Fixes appwrite/appwrite#4117

@stnguyen90 stnguyen90 self-requested a review October 24, 2022 22:41
Copy link
Contributor

@stnguyen90 stnguyen90 left a comment

Choose a reason for hiding this comment

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

@pikami Great PR! 🤯 We left some comments during the review, please check them out.

php/wipe_appwrite_bucket/index.php Outdated Show resolved Hide resolved
@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! 🎃

@pikami pikami requested review from stnguyen90 and removed request for stnguyen90 November 19, 2022 16:03
php/wipe_appwrite_bucket/index.php Outdated Show resolved Hide resolved
@christyjacob4
Copy link
Contributor

@pikami could you please address the review comment :)

Copy link
Contributor

@stnguyen90 stnguyen90 left a comment

Choose a reason for hiding this comment

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

Hey 👋 awesome work on your PR! We've approved your work and it'll be merged soon!

@Meldiron
Copy link
Contributor

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

@Meldiron Meldiron merged commit 3079a61 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

⚡ Write a wipeAppwriteBucket() Function using PHP
4 participants