-
Notifications
You must be signed in to change notification settings - Fork 62
nested comments #10
Comments
Any news on this issue, mate? |
Hi @AurelDragut. First, thank you for your support & ping on this. I was unable to think this package as you mentioned, I will upgrade this package's version to 1.0.0 in a few days with this update. |
Maybe i didnt explain well enough. Anyway this can get alot better and as soon as i get how to do it i will be happy to help on it. I am also thinking about a notification system: any user that comments on a model object can get notified of any new comment that is added to that model object (either that is a model object or a comment added to that model object). |
Hi @AurelDragut could you please check out this example? https://github.com/actuallymab/laravel-comment/blob/master/tests/CommentTest.php#L147 Also, you can find a basic nested Comment object at https://github.com/actuallymab/laravel-comment/blob/master/tests/Models/Comment.php |
Sorry, mate, i have no experience with tests. i don't know how to do tests yet. |
You might seem that as an example. |
If anything wrong or misunderstand, you can reopen issue. |
Hey, i have made comments also commentable to get nested comments.
i made comments commentable and i got them nested but i don't quite see a proper way to count the children of the comments on the number of the main model's comments...
i would do a
but i get an error about some trait, i guess there is a conflict between commentable and cancomment
The text was updated successfully, but these errors were encountered: