Progress default value is not validated
Open
Nobody has claimed this yet.
0. Needs triage
bug
- Dominant language
- JavaScript
- Stars
- 214
- Forks
- 52
- Avg merge
- 1d 10h
- Merged PRs (30d)
- 82
Description
Steps to reproduce
- Add new column with type progress
- Set default value to
120 - Default value is accepted
Expected behavior
It should be limited to 100. A fix for this issue must contain a migration step to lower existing default values to 100.
Actual behavior
Unspecified behavior, the progress bar looks "full", but editing the value is not accepted - for example try entering 110 as avlue.
Tables app version
0.8.0
Browser
No response
Client operating system
No response
Operating system
No response
Web server
None
PHP engine version
None
Database
None
Additional info
No response
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No files, tests, or entry points are named. Reproduce the issue by adding a progress column with a default of 120, then trace progress default validation and the migration path for existing defaults. Done means defaults cannot exceed 100 and existing values above 100 are lowered through a migration.
Written by the indexing model from the issue text.
Assessment
- Domain
- database
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100