Skip to content
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 ability to interact with backups. #605

Merged
merged 1 commit into from
May 27, 2015

Conversation

evanlucas
Copy link
Contributor

No description provided.

@evanlucas
Copy link
Contributor Author

Still could probably use some more tests

@evanlucas
Copy link
Contributor Author

Related: #337

public $datastore;
public $updated;
public $instance;
public $instance_id;
Copy link
Contributor

Choose a reason for hiding this comment

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

This needs to be camel case (instanceId). You can then alias instance_id like so.

@jamiehannaford
Copy link
Contributor

@evanlucas 👍 This is really good, thank you so much for submitting! I've annotated a lot of stuff with some thoughts - if anything's unclear feel to comment on it and I'll elaborate on what I mean

@evanlucas evanlucas force-pushed the backups branch 2 times, most recently from 4adc887 to 90f4561 Compare May 26, 2015 12:00
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 88.02% when pulling 90f4561 on evanlucas:backups into 7b54706 on rackspace:working.

@evanlucas
Copy link
Contributor Author

Strange, apparently, the API does not accept the description if it is null. It is listed as optional in the API docs, but if the key exists, but is null, the call fails

@evanlucas
Copy link
Contributor Author

Ok, just added a fix to account for the description issue.

All other comments should be addressed. :]

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 88.04% when pulling 6742812 on evanlucas:backups into 7b54706 on rackspace:working.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 88.04% when pulling 6742812 on evanlucas:backups into 7b54706 on rackspace:working.

jamiehannaford pushed a commit that referenced this pull request May 27, 2015
Add ability to interact with backups.
@jamiehannaford jamiehannaford merged commit 29e1f27 into rackspace:working May 27, 2015
@jamiehannaford
Copy link
Contributor

@evanlucas This is perfect, thanks for all your hard work! 🚀

@evanlucas
Copy link
Contributor Author

Glad to help out :]

@evanlucas evanlucas deleted the backups branch May 27, 2015 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants