-
Notifications
You must be signed in to change notification settings - Fork 15
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
Fix missing rows test management #622
Conversation
@@ -15,6 +15,15 @@ export const TEST_MANAGEMENT_PAGE_QUERY = gql` | |||
id | |||
name | |||
} | |||
testPlans { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
REVIEWERS: even if we do not go with a separate Test Plan table, this is the query I would like to use.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, code looks good and I verified that the issue identified by Matt is fixed.
d11437a
to
568279f
Compare
This PR addresses #580. To see, run
yarn dev
, and then navigate to the Management page. Notice that all the test plan are present, even though not all of them have been tested.