django-cms / djangocms-moderation

Other
13 stars 21 forks source link

Ask: Django 4.2 support and aldryn-forms dependency #257

Closed joshyu closed 4 weeks ago

joshyu commented 6 months ago

Hi team,

I found that djangocms-moderation depends on aldryn-forms which hasn't been supported by divio any more. If we want to upgrade this addon to django 4.2 compatible, we also need to make aldryn-forms django 4.2 compatible.

How do you plan to achieve this?

fsbraun commented 6 months ago

@Aiky30 started some work recently in #256. A likely scenario is that the aldryn-forms dependency will be dropped.

joshyu commented 6 months ago

Hi @fsbraun ,

Right now we need to add django4.2 support and work with django-cms 4.0.1.x. Can you please help to create a branch support/django-cms-4.0.x based on version 2.1.6 ? Thanks.

fsbraun commented 6 months ago

@joshyu What exactly are you referring to with "version 2.1.6"?

joshyu commented 6 months ago

@fsbraun,

the latest released version which is compatible with django-cms 4.0.1.x branch is 2.1.6, so please create a branch support/django-cms-4.0.x based on tag 2.1.6(https://github.com/django-cms/djangocms-moderation/tree/2.1.6)

then we can continue supporting django-cms 4.0.1.x and django 4.2 .

fsbraun commented 6 months ago

Hi @joshyu,

ok, now I got it. That makes sense, once there are changes introduced at any point that will not allow running moderation on django CMS 4.0. For now, I do not see the need and would advise against it to avoid the code bases drifting apart for no reason. If you think differently, you can create that branch yourself.

Generally speaking: is it not much easier to move on to django CMS 4.1 and avoid reinventing everything again? Wouldn't that save quite some resources?

joshyu commented 6 months ago

Hi @fsbraun ,

As currently we have no much effort to do CMS upgrade, and there are some other changes due to versioning upgrade. The leaders want to do it later.

Can you help me create this branch in current repo? I don't have the permission to do that. Thanks.

fsbraun commented 6 months ago

ok, but let's give it a meaningful name, like support/moderation-forms which is the bit that depends on aldryn-forms and probably will go away in the main branch when we add django 4.2 support. Are you sure you are using the module djangocms_moderation.contrib.moderation_forms?

joshyu commented 6 months ago

Hi @fsbraun ,

I think your suggestion is good. We can use that folder name.

fsbraun commented 6 months ago

Superb! Let us keep the django CMS 4.0 compatibility here. I have already created the branch (based on 2.1.6).