AAfghahi opened a new pull request #14381:
URL: https://github.com/apache/superset/pull/14381


   ### SUMMARY
   We wanted more robust messaging in Sql Lab that tells user what, if 
anything, is limiting the results that they get when they run a SQL query. 
Currently this file will also include a database migration, we are going to 
wait for this migration to resolve before we continue with this PR.
   
   This has custom messages that are based on: 
   -  DISPLAY_MAX_ROW causing the results to be limited (admin and non-admin 
messaging)
   -  The query itself limiting the options
   - the dropdown limiting results
   - the default dropdown of 1000 being the cause for the limit
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   Before:
   
![image](https://user-images.githubusercontent.com/48933336/116311763-9c4b6400-a779-11eb-886c-c36461d6e547.png)
   
   
   
   After:
   
   display max row for admin
   ![Screen Shot 2021-04-27 at 3 58 14 
PM](https://user-images.githubusercontent.com/48933336/116311258-0c0d1f00-a779-11eb-9680-81b8874469c7.png)
   non-admin
   ![Screen Shot 2021-04-27 at 3 14 27 
PM](https://user-images.githubusercontent.com/48933336/116311380-31019200-a779-11eb-9ce8-8b3b6c4561cf.png)
   Query Limit
   ![Screen Shot 2021-04-27 at 4 06 05 
PM](https://user-images.githubusercontent.com/48933336/116311444-4080db00-a779-11eb-8479-503989e3aae2.png)
   Dropdown limit:
   ![Screen Shot 2021-04-27 at 4 56 20 
PM](https://user-images.githubusercontent.com/48933336/116311675-850c7680-a779-11eb-9505-8c1aa4865451.png)
   Default Dropdown limit of 1000
   ![Screen Shot 2021-04-27 at 4 46 42 
PM](https://user-images.githubusercontent.com/48933336/116311729-935a9280-a779-11eb-9d06-b5a2f65154a5.png)
   
   
   ### TEST PLAN
   Visual Testing
   
   ### ADDITIONAL INFORMATION
   <!--- Check any relevant boxes with "x" -->
   <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->
   - [ ] Has associated issue:
   - [ x] Changes UI
   - [ ] Includes DB Migration (follow approval process in 
[SIP-59](https://github.com/apache/superset/issues/13351))
     - [ ] Migration is atomic, supports rollback & is backwards-compatible
     - [ ] Confirm DB migration upgrade and downgrade tested
     - [ ] Runtime estimates and downtime expectations provided
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API
   


-- 
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.

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