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

changed method invocation amse.toggle_check_node() to toggle_check_no… #102

Merged
merged 3 commits into from
Jun 5, 2016
Merged

Conversation

allanwsilva
Copy link
Contributor

…de(), removing the object name from the method call in toggle-checked-state-on-label-click.js.

That prefix is causing NOT to trigger the method properly.

…de(), removing the object name from the method call in toggle-checked-state-on-label-click.js.

That prefix is causing *NOT* to trigger the method properly.
@@ -1,2 +1,3 @@
demo/
node_modules/
/.project
Copy link
Owner

Choose a reason for hiding this comment

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

can you remove this from the commit?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done !

@alexandernst
Copy link
Owner

Indeed, nice catch! I'll merge once the .gitignore modification is removed from the commit ;)

@alexandernst
Copy link
Owner

alexandernst commented May 31, 2016

Your second commit removed too much things. Please put back the

demo/
node_modules/

as those existed before your patch.

@alexandernst
Copy link
Owner

Didn't see this last commit of yours, sorry for that! Merging

@alexandernst alexandernst merged commit f6d6bf5 into alexandernst:master Jun 5, 2016
@allanwsilva
Copy link
Contributor Author

Thanks !

Em dom, 5 de jun de 2016 às 20:14, Alexander Nestorov <
notifications@github.com> escreveu:

Merged #102
#102.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#102 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AFadSyUjtz6ITCE1ofaYdLS6lKr9IDdTks5qIqGFgaJpZM4IoJp_
.


Allan Wojcik da Silva

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.

None yet

3 participants