Skip to content

fix: coerce numeric values, and handle nullable correctly #2201

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

HarshMN2345
Copy link
Member

@HarshMN2345 HarshMN2345 commented Aug 8, 2025

What does this PR do?

  • Coerce input[type=number] string → number on input/change
  • Treat '' as null when nullable; otherwise keep undefined for required to trigger
  • Remove custom integer/min/max/error checks; use native validity + invalid handler
  • Prevents sending string/empty values for integer attrs

Test Plan

before:
image

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

yes

Copy link

appwrite bot commented Aug 8, 2025

Console

Project ID: 688b7bf400350cbd60e9

Sites (2)
Site Status Logs Preview QR
 console-qa
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code
 console-cloud
688b7c18002b9b871a8f
Ready Ready View Logs Preview URL QR Code

Note

Appwrite has a Discord community with over 16 000 members.

@HarshMN2345 HarshMN2345 requested a review from ItzNotABug August 8, 2025 18:51
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.

1 participant