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.

Sign up for GitHub

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

Jump to bottom

Add new builder method scopes for soft deleting #276

Closed
josephmancuso opened this issue Dec 21, 2020 · 3 comments
Closed

Add new builder method scopes for soft deleting #276

josephmancuso opened this issue Dec 21, 2020 · 3 comments
Labels
easy These issues are geared for people who have not yet contributed to this project yet Query Builder Related to the query builder

Comments

@josephmancuso
Copy link
Member

Need to add the following scopes or methods to the SoftDeletesMixin class:

  • only_trashed()
  • restore()
  • force_delete
@josephmancuso josephmancuso added Query Builder Related to the query builder easy These issues are geared for people who have not yet contributed to this project yet labels Dec 21, 2020
@girardinsamuel
Copy link
Contributor

girardinsamuel commented Jan 3, 2021 •
edited

Yeah that's exactly what I was looking for the pingcrm inertia demo !

I am struggling to get only trashed users as we should check that users have a deleted_at field not null or not empty.. => where_not_null("deleted_at") does not seem to work, I guess that the default value for deleted_at is not the SQL NULL but maybe "" ?

@girardinsamuel
Copy link
Contributor

I will open a PR

@girardinsamuel girardinsamuel mentioned this issue Jan 3, 2021
Merged
3 tasks
@josephmancuso
Copy link
Member Author

This is done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
easy These issues are geared for people who have not yet contributed to this project yet Query Builder Related to the query builder
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@girardinsamuel @josephmancuso