-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Conversation
Deploy preview for netlify-cms-www ready! Built with commit f241bc6 |
Deploy preview for cms-demo ready! Built with commit f241bc6 |
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.
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 } = {}) => { |
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.
Doesn't look like we're using author
, maybe we should remove until there's a use.
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.
Good catch, it should be added to the commit.
f5aece1
to
e369cb8
Compare
4b40ce0
to
31930ba
Compare
@erquhart Updated! |
- Summary
- Test plan
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)