-
Notifications
You must be signed in to change notification settings - Fork 69
new: Add POST /images #45
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
new: Add POST /images #45
Conversation
src/data/endpoints/images.yaml
Outdated
@@ -20,6 +19,35 @@ endpoints: | |||
python: | | |||
import linode | |||
TODO | |||
POST: | |||
money: true |
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.
Pretty sure we don't charge for images
src/data/endpoints/images.yaml
Outdated
@@ -20,6 +19,35 @@ endpoints: | |||
python: | | |||
import linode | |||
TODO | |||
POST: | |||
money: true | |||
oauth: images:create |
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.
also linodes:view ?
ea48426
to
d54f3f1
Compare
Approved, but someone needs to kick tests again |
src/data/endpoints/images.yaml
Outdated
limit: "1-50 ASCII characters limited to letters, numbers, and underscores" | ||
value: this_is_a_label | ||
description: | ||
optioanl: true |
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.
optional
d54f3f1
to
7e4086a
Compare
src/data/endpoints/images.yaml
Outdated
description: A longer summary that describes the image | ||
type: String | ||
limit: "1-65000 ASCII characters" | ||
value: A longer description of the image |
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.
now you've got extra indentation here
src/data/endpoints/images.yaml
Outdated
optional: false | ||
description: The id of the Linode Disk that should be imagized | ||
type: Integer | ||
value: 123 |
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.
here too - I think these files are 2 spaces
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.
Indentation is weird and inconsistent, but it's fine otherwise
7e4086a
to
5959291
Compare
@Dorthu all fixed... bad editor, I blame it on |
5959291
to
90dcc2d
Compare
bug: Removed root_device_ro from LinodeConfig
More capitalization
[New] Add way to build the api docs site from a local spec file
No description provided.