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

fix: miscellaneous backend fixes #1568

Merged
merged 8 commits into from
Aug 7, 2018

Conversation

tech4him1
Copy link
Contributor

- Summary

- Test plan

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@tech4him1 tech4him1 requested a review from Benaiah August 1, 2018 17:49
@verythorough
Copy link
Contributor

verythorough commented Aug 1, 2018

Deploy preview for netlify-cms-www ready!

Built with commit f241bc6

https://deploy-preview-1568--netlify-cms-www.netlify.com

@verythorough
Copy link
Contributor

verythorough commented Aug 1, 2018

Deploy preview for cms-demo ready!

Built with commit f241bc6

https://deploy-preview-1568--cms-demo.netlify.com

@tech4him1 tech4him1 mentioned this pull request Aug 4, 2018
7 tasks
Copy link
Contributor

@erquhart erquhart left a comment

Choose a reason for hiding this comment

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

Just one change request, otherwise LGTM 👍

@@ -118,10 +118,15 @@ export default class API {
return this.processFiles(entries);
};

uploadBlob = async item => {
uploadBlob = async (item, { commitMessage, branch = this.branch, author = this.commitAuthor } = {}) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Doesn't look like we're using author, maybe we should remove until there's a use.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, it should be added to the commit.

@tech4him1 tech4him1 force-pushed the misc-backend-fixes branch from f5aece1 to e369cb8 Compare August 6, 2018 20:54
@tech4him1 tech4him1 changed the title miscellaneous backend fixes WIP: miscellaneous backend fixes Aug 6, 2018
@tech4him1 tech4him1 force-pushed the misc-backend-fixes branch from 4b40ce0 to 31930ba Compare August 6, 2018 22:45
@tech4him1
Copy link
Contributor Author

@erquhart Updated!

@tech4him1 tech4him1 changed the title WIP: miscellaneous backend fixes fix: miscellaneous backend fixes Aug 7, 2018
@erquhart erquhart merged commit f7e7120 into decaporg:master Aug 7, 2018
@tech4him1 tech4him1 deleted the misc-backend-fixes branch August 7, 2018 16:41
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