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

🐛 Bug Report: Updating attributes doesn't allow null defaults, min, max etc #38

Open
2 tasks done
pingu2k4 opened this issue May 10, 2024 · 5 comments
Open
2 tasks done
Labels
bug Something isn't working

Comments

@pingu2k4
Copy link

pingu2k4 commented May 10, 2024

👟 Reproduction steps

Databases.UpdateStringAttribute and others such as email, ip, url etc - the xdefault parameter could be null but the parameter it expects is non nullable. Also, for things like updating an "integer" attribute, the min, max and default are all non nullable, and its not possible to pass null into these at all.

👍 Expected behavior

Should accept nullable values for default

👎 Actual Behavior

Deson't accept nullable values

🎲 Appwrite version

Version 0.10.x

💻 Operating system

Windows

🧱 Your Environment

No response

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

@pingu2k4 pingu2k4 added the bug Something isn't working label May 10, 2024
@pingu2k4 pingu2k4 changed the title 🐛 Bug Report: Updating string based attributes doesn't allow null defaults 🐛 Bug Report: Updating attributes doesn't allow null defaults, min, max etc May 10, 2024
@adityaoberai
Copy link
Member

Hi @pingu2k4
Just want to double check that the Appwrite version is correct?
Because the current version of the SDK is not meant for v0.10

@pingu2k4
Copy link
Author

Hey, this was on V 1.5.x of Appwrite. Not sure what you mean with V 0.10 sorry?

@adityaoberai
Copy link
Member

Hey, this was on V 1.5.x of Appwrite. Not sure what you mean with V 0.10 sorry?

Sorry, I noticed this here and it had me confused 😅
image

@pingu2k4
Copy link
Author

Ohh, thats my bad... I have no idea why I put that... Complete accident sorry!

@adityaoberai
Copy link
Member

Haha, no worries

I just noticed the error you have mentioned. Sorry about that.
image

I'll definitely discuss this with the team ASAP. Just as a general note, the best place to raise SDK-related issues is the SDK-generator repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants