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

Update link in deleting the files #850

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from

Conversation

codewithdev
Copy link
Contributor

Closing #838

What thing being changed:

The documentation for "Deleting the files" contains invalid URL. This can be fixed by changing the URL from https://YOUR.PARSE-SERVER.HERE/parse/files/...profile.png to https://YOUR.PARSE-SERVER.HERE/parse/files/profile.png

@mtrezza
Copy link
Member

mtrezza commented Oct 2, 2021

Thanks for this PR! Could you incorporate an explanation, as described in the issue?

The documentation can be improved quickly by adding a clear example to explain how to format the URL.

For example :

Although the file's URL is 'http:////YOUR.PARSE-SERVER.HERE/parse/files/appId/my_picture.png', you have to remove the '/appId' section from the URL to access the API endpoind.

So the URL must be : 'http:////YOUR.PARSE-SERVER.HERE/parse/files/my_picture.png'

Otherwise it will not work.

@codewithdev
Copy link
Contributor Author

Definitely!
The images are located in the directory of parse which is /files and whenever send DELETE request to the given URL( https://YOUR.PARSE-SERVER.HERE/parse/files/...profile.png) which is an invalid format of the URL. Although, "appId" is required to retrieve the images which needs to be deleted.

@mtrezza
Copy link
Member

mtrezza commented Oct 2, 2021

Yes, I think a short explanation that appId is required for retrieving but not deleting would be good in the docs. As I read the original issue, that missing info created the difficulty.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants