-
Notifications
You must be signed in to change notification settings - Fork 159
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
Fix file extension validation #705
Fix file extension validation #705
Conversation
@sourabpramanik is attempting to deploy a commit to the appwrite Team on Vercel. A member of the Team first needs to authorize it. |
… bug-7362-file-extension-validation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Interesting. I will look into this again and make required changes. Thanks for pointing that out ✌️ |
Hey @stnguyen90 , I see you are checking the storage file upload but this issue was not found in storage. It was found in create manual Function step 2. You can check the gif I added to the PR. |
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.
@sourabpramanik here's the same in the create manual function wizard:

Hmm gotta check that again I was unable to reproduce this last time. Thanks for pointing that out. |
@sourabpramanik, any update on this? |
Yes done |
What does this PR do?
This PR fixes the file input upload extension validation in create function. First found in the create function step 2.

Test Plan
Manual test
Related PRs and Issues
Fixes #7362
Have you read the Contributing Guidelines on issues?
YES