wagtail / wagtailtrans

A Wagtail add-on for supporting multilingual sites
http://wagtailtrans.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
104 stars 60 forks source link

TranslatableMixin? #152

Open kaedroho opened 5 years ago

kaedroho commented 5 years ago

I wonder if we could move the implementation of TranslatedPage into a mixin class instead?

This would have a couple of advantages:

The only disadvantage I can think of is that a translated version of a page must be the same type. But, conversely, adding this constraint will make queries for specific translated versions of a page quicker.

For backwards compatibility, we could define TranslatedPage as:

class TranslatedPage(TranslatableMixin, Page):
    pass
irugoy commented 3 years ago

Hi there, is this TranslatableMixin still around? At this point there another option/method for translating Snippets or just django models?

Thanks