-
Notifications
You must be signed in to change notification settings - Fork 33
feat: Added bulk delete to version change view #338
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.
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
feat: Added bulk delete to version change view #338
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #338 +/- ##
==========================================
+ Coverage 90.88% 91.21% +0.32%
==========================================
Files 72 72
Lines 2546 2663 +117
Branches 361 307 -54
==========================================
+ Hits 2314 2429 +115
+ Misses 168 163 -5
- Partials 64 71 +7 ☔ View full report in Codecov by Sentry. |
601944c
to
1deb2b7
Compare
…s-djangocms-versioning
…s-djangocms-versioning
Do not delete anything if a published or draft version is amongst the selected objects
Update the queryset to contain content elements
@polyccon I updated your PR. People keep asking for this feature! I guess you were 80% there. I hope you do not mind, that I added some stuff to finish this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@polyccon Thanks so much for getting this started. It's been a feature that people have kept requesting. Great that we can finish this now!
Nice work @fsbraun thank you for your hard work to bring it over the finish line!!!!!! |
Description
*- Initial draft on adding functionality to bulk delete to versions djangocms versioning
*- More information required on the ticket that needs to be a user story - see link below
Related resources
Checklist
master
Slack to find a “pr review buddy” who is going to review my pull request.