-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
🐛 Bug Report: Using falsy values for optional parameters #5416
Labels
Comments
stnguyen90
added
the
product / databases
Fixes and upgrades for the Appwrite Database.
label
Apr 21, 2023
@Suven-p, thanks for creating this issue! 🙏🏼 It makes perfect sense to allow an empty string as the default. I'll bring this up to the team to see how we can proceed. |
This was referenced Jul 25, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
👟 Reproduction steps
When I make a request to
path := r.Replace("/databases/{databaseId}/collections/{collectionId}/attributes/string")
with parameters
and value of default is an empty string.
👍 Expected behavior
Handle empty strings as parameter not being sent.
https://discord.com/channels/564160730845151244/1099033836731519006
👎 Actual Behavior
It throws an error in appwrite 1.3.1 but not in older versions of appwrite.
🎲 Appwrite version
Version 1.3.x
💻 Operating system
Linux
🧱 Your Environment
No response
👀 Have you spent some time to check if this issue has been raised before?
🏢 Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: