-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add create_mission_definition and start_mission_exicution #39
Conversation
be5c9ef
to
9159c58
Compare
} | ||
|
||
try: | ||
response_dict: dict[str, Any] = self.client.query(mutation_string, params) |
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.
is it sufficient to say dict?
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.
dont know what you mean. It's the same as the other functions we have made. Il just let it stand and we can update it later maby
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.
Some suggestions for naming and I think we will need to return the mission execution id
Can you add tests for these functions? |
4b22f06
to
90c1cab
Compare
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.
Suggest renaming
90c1cab
to
18cef40
Compare
18cef40
to
9e2cde6
Compare
9e2cde6
to
1cabb27
Compare
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.
LGTM
No description provided.