Skip to content
This repository has been archived by the owner on Jul 25, 2022. It is now read-only.

Fix Limit #2274

Merged
merged 1 commit into from
Sep 24, 2020
Merged

Fix Limit #2274

merged 1 commit into from
Sep 24, 2020

Conversation

jgrund
Copy link
Member

@jgrund jgrund commented Sep 24, 2020 •
edited
Loading

The limit column is being passed in as Text in graphql calls, which is
not valid.

Instead, we can just keep it as an option, where NULL is the same as no
limit.

Fixes #2273.

Signed-off-by: Joe Grund jgrund@whamcloud.io

This change is Reviewable

@jgrund
Fix Limit
ed92de5 
The limit column is being passed in as Text in graphql calls, which is
not valid.

Instead, we can just keep it as an option, where NULL is the same as no
limit.

Fixes #2273.

Signed-off-by: Joe Grund <jgrund@whamcloud.io>
@jgrund jgrund added the bug label Sep 24, 2020
@jgrund jgrund self-assigned this Sep 24, 2020
@jgrund jgrund requested a review from a team September 24, 2020 19:55
@jgrund jgrund merged commit 6b1b229 into master Sep 24, 2020
@jgrund jgrund deleted the fix_limit branch September 24, 2020 20:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@johnsonw johnsonw johnsonw approved these changes

Assignees

@jgrund jgrund

Labels
bug
Projects
None yet
Milestone
No milestone
2 participants
@jgrund @johnsonw