innovark37 opened a new issue, #36258:
URL: https://github.com/apache/superset/issues/36258

   ### Bug description
   
   ### Bug description
   Error tooltip doesn't appear above Server Page Length select when its value 
isn't integer.
   
   ### Steps to reproduce
   1. Deploy the Docker version of Superset based on 6.0.0rc2 (same on the 
master-branch build version).
   2. Go to "Charts", create new Table chart with any dataset.
   3. When new page is opened, find column with chart settings, select "Data".
   4. Find "Server pagination" checkbox and check it.
   5. Enter non-integer values into Server Page Length and Row limit and save 
them.
   
   ### Expected behavior
   Both Server Page Length and Row limit selects have the same error tooltip: 
"is expected to be an integer"
   
   ### Actual behavior
   Only Row limit has "is expected to be an integer" error tooltip.
   
   ### Suggested solution
   Add value validator legacyValidateInteger for Server Page Length, just like 
in Row limit.
   
   ### Screenshots/recordings
   
   <img width="286" height="175" alt="Image" 
src="https://github.com/user-attachments/assets/66320723-7d72-4d89-a528-e4808b858866";
 />
   
   <img width="319" height="147" alt="Image" 
src="https://github.com/user-attachments/assets/572d8fdf-a420-40c7-a1f5-252cd30b1ed9";
 />
   
   ### Superset version
   
   master / latest-dev
   
   ### Python version
   
   3.11
   
   ### Node version
   
   18 or greater
   
   ### Browser
   
   Chrome
   
   ### Additional context
   
   _No response_
   
   ### Checklist
   
   - [x] I have searched Superset docs and Slack and didn't find a solution to 
my problem.
   - [x] I have searched the GitHub issue tracker and didn't find a similar bug 
report.
   - [ ] I have checked Superset's logs for errors and if I found a relevant 
Python stacktrace, I included it here as text in the "additional context" 
section.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to