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

EX-1687 graphql commands #2240

Merged
merged 7 commits into from
Sep 16, 2020
Merged

EX-1687 graphql commands #2240

merged 7 commits into from
Sep 16, 2020

Conversation

nlinker
Copy link
Contributor

@nlinker nlinker commented Sep 14, 2020

Implements GraphQL for commands.
Also fixes some clippy warnings.

Screenshot 2020-09-14 at 18 41 25
image


This change is Reviewable

@nlinker nlinker requested a review from a team September 14, 2020 13:11
@nlinker nlinker self-assigned this Sep 14, 2020
@ip1981
Copy link
Member

ip1981 commented Sep 14, 2020

We usually need a command with specific id, no?

@nlinker
Copy link
Contributor Author

nlinker commented Sep 14, 2020

We usually need a command with specific id, no?

image
The original task however haven't mentioned this.
We have /api/command/123/ for the ability to grab commands by id.

iml-api/src/graphql.rs Outdated Show resolved Hide resolved
mkpankov
mkpankov previously approved these changes Sep 15, 2020
Copy link
Contributor

@mkpankov mkpankov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 5 of 5 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @ip1981)

Signed-off-by: Nick Linker <nlinker@gmail.com>
Signed-off-by: Nick Linker <nlinker@gmail.com>
Signed-off-by: Nick Linker <nlinker@gmail.com>
Signed-off-by: Nick Linker <nlinker@gmail.com>
Signed-off-by: Nick Linker <nlinker@gmail.com>
Signed-off-by: Nick Linker <nlinker@gmail.com>
mkpankov
mkpankov previously approved these changes Sep 15, 2020
Copy link
Contributor

@mkpankov mkpankov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 2 of 2 files at r2.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @ip1981)

ip1981
ip1981 previously approved these changes Sep 15, 2020
@jgrund
Copy link
Member

jgrund commented Sep 15, 2020

Looks like sqlx schema check is failing, otherwise LGTM.

Signed-off-by: Nick Linker <nlinker@gmail.com>
@nlinker nlinker dismissed stale reviews from ip1981 and mkpankov via e2f3888 September 15, 2020 14:02
@nlinker nlinker requested a review from a team September 15, 2020 15:57
@jgrund jgrund merged commit a5ffcdd into master Sep 16, 2020
@jgrund jgrund deleted the ex-1687-graphql-commands branch September 16, 2020 01:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants