laravelista / comments

Native comments for your Laravel application.
MIT License
744 stars 143 forks source link

Ability to see deleted comments #186

Closed S-DICKSON closed 2 years ago

S-DICKSON commented 2 years ago

Added this functionality.

@comments(['model' => $model, 'withTrashed' => true])

image

Not sure if you want this or not so happy to close PR if not required.

I've not tested this yet with models that don't have soft deletes so there is still work required in this PR.

mabasic commented 2 years ago

What is the use case for showing deleted comments?

S-DICKSON commented 2 years ago

What is the use case for showing deleted comments?

A social media moderator for a group page could look at a post that has comments and check nothing has been posted that is inappropriate even if it's been deleted.

mabasic commented 2 years ago

I will accept this PR once completed. Let me know when it is ready. Thank you for your contribution.

Shubhamlmp commented 2 years ago

@mabasic Info from https://repo.packagist.org: #StandWithUkraine Using version ^4.5 for laravelista/comments ./composer.json has been updated Running composer update laravelista/comments Loading composer repositories with package information Updating dependencies Your requirements could not be resolved to an installable set of packages.

Problem 1

You can also try re-running composer require with an explicit version constraint, e.g. "composer require laravelista/comments:*" to figure out if any version is installable, or "composer require laravelista/comments:^2.1" if you know which you need.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

S-DICKSON commented 2 years ago

I'm sorry, I don't have the time to work on it. If people require it in the future, I'll happy to develop it :)