Problem/Motivation

There are users that are uploading paragraphs on their own, instead of embedding on the parent entities.
This is bad UX for translators, and even if this is a desirable solution in some cases, most of them should be being embedded in their parent entities (e.g. nodes).

Proposed resolution

We should hide this tab by default, and have the option of showing it, but only if this flag is activated, so we prevent people of using it unless they really know the pros and cons.

Remaining tasks

Patch that disables the paragraphs tabs and menu links if the option is disabled, which will be the default.

User interface changes

Paragraphs tab is not shown.
Added a contrib section in the settings page for extending with integration settings for contrib modules.

API changes

None.

Data model changes

Added preference.contrib.paragraphs.enable_bulk_management in lingotek.settings configuration object and proper config schema.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

penyaskito created an issue. See original summary.

penyaskito’s picture

Status: Active » Needs review

Attached patch that implements the form settings for paragraphs module and disables the route and local task if the setting is not active.

penyaskito’s picture

Forgot the patch actually.

Status: Needs review » Needs work

The last submitted patch, 3: 2895307-paragraphs-bulk-management-2.patch, failed testing. View results

penyaskito’s picture

Status: Needs work » Needs review
FileSize
16.52 KB

Forgot to include the form itself. Some phpcs fixes too.

Status: Needs review » Needs work

The last submitted patch, 5: 2895307-paragraphs-bulk-management-5.patch, failed testing. View results

penyaskito’s picture

Fixed tests. We need to active the tab in that test too.

penyaskito’s picture

penyaskito’s picture

Status: Needs review » Reviewed & tested by the community

Tests passed as expected now.

  • penyaskito committed 54b878d on 8.x-2.x
    Issue #2895307 by penyaskito: Remove paragraphs tab for bulk management
    
penyaskito’s picture

Status: Reviewed & tested by the community » Fixed

Committed 54b878d and pushed to 8.x-2.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.