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.

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

Select - Row Limit picker hides 0 in Console #4559

Closed
brenjay opened this issue Apr 25, 2020 · 1 comment
Closed

Select - Row Limit picker hides 0 in Console #4559

brenjay opened this issue Apr 25, 2020 · 1 comment
Assignees
Labels
c/console Related to console e/quickfix can be wrapped up in few hours k/bug Something isn't working

Comments

@brenjay
Copy link

brenjay commented Apr 25, 2020

First off - wonderful project! Ran into a maddening issue in the Console UI which caused me trouble for hours until I could figure out what it was; currently using v1.2.0-beta.5.

When editing permissions schema for 'select' and specifying a row limit, it is possible to iterate the number picker down to 0. However, the picker doesn't actually display '0'. Thus, you can save the schema and assume you actually have no row limit; when in fact its actually set at 0, but the number picker is blank so you assume everything is good and its set to null. All my queries for a particular role kept returning nothing; and I couldn't figure out why; as I kept pouring over the permissions schema and everything had "green checkmarks".

The row limit picker's input for 'select' is hidden when set to 0 AND null in /tables/{name}/schema page, it just appears blank. However, in /schema/public/permissions, it actually does show the row limit integer, even when 0, and when null it displays a X, so that's how I figured out what the problem was.

If the row limit gets set accidently to 0; you have to go back into the /tables/{name}/schema/ page, set the row limit picker to any positive integer, save the schema, then refresh the page, and remove the integer altogether so the field is blank; then save, and finally the row limit will be set to null instead of 0. Also I found that if you type '0' in the number picker, even though the picker shows nothing, if you save it, it will be saved as a 0 and you'll have to do the process described to remove it again.

Finally, it would be helpful for the UI to display the 'filter' icon when the Row Limit is set to anything but null. The 'select' column displays a green check for the field regardless of row limit value; which can be deceptive when you're looking at permissions and think an action has no modifications when in fact it does.

@GavinRay97
Copy link
Member

Good catch, thank you!

@GavinRay97 GavinRay97 added c/console Related to console e/quickfix can be wrapped up in few hours k/bug Something isn't working labels Apr 25, 2020
beerose pushed a commit to beerose/graphql-engine that referenced this issue May 12, 2020
stevefan1999-personal pushed a commit to stevefan1999-personal/graphql-engine that referenced this issue Sep 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/console Related to console e/quickfix can be wrapped up in few hours k/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants