add like feature for comments

[ViditChitkara]

closes #2498

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

• tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with rake test
• code is in uniquely-named feature branch and has no merge conflicts
• PR is descriptively titled
• PR body includes fixes #0000-style reference to original issue #
• ask @publiclab/reviewers for help, in a comment below

We're happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged if your tests fail at first!

If tests do fail, click on the red X to learn why by reading the logs.

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software

We have a loose schedule of reviewing and pulling in changes every Tuesday and Friday, and publishing changes on Fridays.

Thanks!

[jywarren]

Ooh, this is really nice. Almost ready to go, no? I see a test getting written -- i like this a lot! And i see you left the likeable_type over so we could specify emojis too 😄 🎉

[ViditChitkara]

Hi, sorry for the late reply--had an exam today. There are some changes to be made in tests, will ping once completed

[ViditChitkara]

@jywarren made certain test related changes. I guess, it is good to be merged now.

[ViditChitkara]

Don't know why the tests are failing. If you could have a look?

[jywarren]

Try rebasing over latest master; we just merged some fixes!

…

On Fri, May 25, 2018, 3:54 PM Vidit ***@***.***> wrote: Don't know why the tests are failing. If you could have a look? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2591 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABfJ8UG1VntHKZQ05yZhmQJ4co6WOzpks5t2GFogaJpZM4TK8ol> .

[PublicLabBot]

	1 Warning
⚠️	New migrations added. Please update schema.rb.example by overwriting it with a copy of the up-to-date db/schema.rb. Also, be aware to preserve the MySQL-specific conditions for full-text indices.

	1 Message
📖	@ViditChitkara Thank you for your pull request! I’m here to help with some tips and recommendations. Please take a look at the list provided and help us review and accept your contribution! And don’t be discouraged if you see errors – we’re here to help.

Generated by 🚫 Danger

[ViditChitkara]

Try rebasing over latest master; we just merged some fixes!

Thanks a ton!! So can we merge this if this looks good to you?

[jywarren]

Fantastic! Do you think we should think about refactoring the current Node like system? Don't have to think about this just yet.

Maybe more pressing would be tests for the comment like system. Unit tests -- do you think you could add a few? Thanks!!