search

mockdeep / better

Open. Democratic. Project management platform.
bettermeans.com
51 stars 19 forks source link

Comment spec #29

Closed dklounge closed 10 years ago

dklounge commented 10 years ago

Completed spec - I think the third parameter mention_text in mention method may be a bug, because in the Notification that is created is:

:mention_text => self.comments (line 26)

which means that it is assigned whatever self.comments are independent of the argument that is passed in. That seemed to be the case when I tested it out in console.

dklounge commented 10 years ago

made updates per review notes and submitted as amended commit