Skip to content
This repository was archived by the owner on Jan 26, 2021. It is now read-only.

Conversation

@hungtuchen
Copy link
Contributor

as title said,
I think changing to this behavior would be better, only a minor fix

@facebook-github-bot
Copy link

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at [email protected]. Thanks!

@hungtuchen
Copy link
Contributor Author

done!

@facebook-github-bot
Copy link

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@andrewimm
Copy link
Contributor

Sounds like a good change. Could you match the coding style of the rest of the repository by including spaces in the if and else lines, so that

if(this.state.editText){

becomes

if (this.state.editText) {

and

}else{

becomes

} else {

@hungtuchen
Copy link
Contributor Author

done!

andrewimm added a commit that referenced this pull request May 8, 2015
todoItem should be deleted if updated text is blank
@andrewimm andrewimm merged commit 3897b90 into parse-community:master May 8, 2015
@andrewimm
Copy link
Contributor

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants