django-cms / djangocms-versioning

General purpose versioning package for Django CMS 4 and above.
Other
33 stars 29 forks source link

feat: Simple publishing beginning and end dates #341

Open fsbraun opened 1 year ago

fsbraun commented 1 year ago

Description

Based on the discussion #340 this PR uses the power of djangocms-versionings' abstraction and introduces an option to limit the visibility of a published content object in time.

UX

Tests

This is a proof of concept for discussion. Tests will need to be added and also some informaion on the changelist for versions.

Tests fail since the version admin's publish_view now also accepts a GET request which renders the form for time limits. Tests have now been adjusted, yet

Issues

Related resources

Checklist

marksweb commented 12 months ago

Yes, that point about it providing an example of how to extend functionality is real key one thay we should be able to demonstrate at launch.

And I don't actually know how it's done yet. I'd have a stab at it if I did.

I've got more thoughts but little time. The spiral of complexity is a big one. And nobody has even brought up timezones yet. 😬

jrief commented 9 months ago

Thanks for adding this important missing piece to djangocms-versioning. I just tested this pull request, but there are some issues:

@Aiky30 I get your point about separating this feature into another package. However, how would you implement it? And since it seems that some work has already been done, is this available?

I probably will implement this PR into our installation anyway, because it's a feature our users expect since it was available in CMS3. And I can't wait until a new package emerges anytime.