Part of meta #1853522: [META] (Re)introduce Views data integration for core modules. From the 8.x-3.x branch of Views.

CommentFileSizeAuthor
translation.patch15.26 KBxjm

Comments

lars toomre’s picture

lars toomre commented

@xjm: Throughout this patch, shouldn't 'Definition of' be 'Contains'? Also ' id ' strings should be ' ID '.

xjm’s picture

xjm
she/her
Primary language English
commented

@Lars Toomre, this patch is not new code. It is taken verbatim from the 8.x-3.x version of Views, as stated in the summary, and has been uploaded as merely a starting point to reintroduce the integration.

Status: Needs review » Needs work
Issue tags: -VDC

The last submitted patch, translation.patch, failed testing.

andyceo’s picture

andyceo commented
Status: Needs work » Needs review
Issue tags: +VDC

translation.patch queued for re-testing.

dawehner’s picture

dawehner
Primary language German
commented

Does this actually still make sense if we have entity_translation?

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Postponed (maintainer needs more info)
tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented

No, this would be a total waste of effort. #1952062: Remove legacy translation module in favor of content translation is the way to go. I don't know if we want to mark this postponed or directly won't fix or duplicate or whatever.

dawehner’s picture

dawehner
Primary language German
commented
Status: Postponed (maintainer needs more info) » Closed (won't fix)

Thanks! Let's go with the simplest one.