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.

Sign up for GitHub

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

Jump to bottom

Can't limit min and max values on number #926

Closed
AdminPickering opened this issue Mar 12, 2024 · 0 comments · Fixed by #963
Closed

Can't limit min and max values on number #926

AdminPickering opened this issue Mar 12, 2024 · 0 comments · Fixed by #963
Assignees
@enjeck
Labels
2. developing Work in progress bug Something isn't working

Comments

@AdminPickering
Copy link

Steps to reproduce

  1. Create number field with min -100 and max 0
  2. Store value > 0

Expected behavior

On old version(0.2.1) it used to work as expected. So can't store positive values (cell marked red).

Actual behavior

No mater what is min/max, number can be saved.

Tables app version

0.6.6

Browser

No response

Client operating system

Firefox 121.0 (64-bit)

Operating system

Fedora 38

Web server

Apache

PHP engine version

PHP 7.4

Database

MariaDB

Additional info

No response
@AdminPickering AdminPickering added 0. Needs triage Pending approval or rejection. This issue is pending approval. bug Something isn't working labels Mar 12, 2024
@enjeck enjeck self-assigned this Apr 3, 2024
@enjeck enjeck added 2. developing Work in progress and removed 0. Needs triage Pending approval or rejection. This issue is pending approval. labels Apr 3, 2024
@enjeck enjeck changed the title Can't limit negative values on number Can't limit min and max values on number Apr 3, 2024
@enjeck enjeck mentioned this issue Apr 3, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@enjeck enjeck

Labels
2. developing Work in progress bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: limit min and max numbers nextcloud/tables
2 participants
@enjeck @AdminPickering