Skip to content
This repository has been archived by the owner on Jul 16, 2022. It is now read-only.

Add function to retrieve files from Appwrite storage and upload them … #301

Closed

Conversation

geryjuhasz
Copy link

…to Dropbox

@geryjuhasz
Copy link
Author

out

logs

db

Copy link
Member

@christyjacob4 christyjacob4 left a comment

Choose a reason for hiding this comment

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

Please address the comments

Make sure to set the following environment variables.

* **APPWRITE_ENDPOINT** - AppWrite Domain
* **APPWRITE_FUNCTION_PROJECT_ID** - AppWrite Project ID
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* **APPWRITE_FUNCTION_PROJECT_ID** - AppWrite Project ID

you can remove this var. it's passed by default to the cloud function

## 📝 Environment Variables
Make sure to set the following environment variables.

* **APPWRITE_ENDPOINT** - AppWrite Domain
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* **APPWRITE_ENDPOINT** - AppWrite Domain
* **APPWRITE_ENDPOINT** - Appwrite Domain

* **APPWRITE_ENDPOINT** - AppWrite Domain
* **APPWRITE_FUNCTION_PROJECT_ID** - AppWrite Project ID
* **APPWRITE_API_KEY** - AppWrite API key
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* **APPWRITE_API_KEY** - AppWrite API key
* **APPWRITE_API_KEY** - Appwrite API key
└── composer.lock
└── README.md
```

Copy link
Member

Choose a reason for hiding this comment

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

You're missing the steps required to package this as a tarfile. Please take a look at the other examples to find how.

## 🎯 Trigger

```bash
php index.php fileId1 fileId2 ...
Copy link
Member

Choose a reason for hiding this comment

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

The trigger section does not look right. Please refer to my comment from earlier #244 (comment)

@christyjacob4
Copy link
Member

@geryjuhasz closing this PR for now . Feel free to reopen it / create a new one when you're ready to start working on it once again

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
2 participants