Move field type into Reference group in new Drupal 10.2 field UI

Created on 26 October 2023, 8 months ago
Updated 26 January 2024, 5 months ago

Problem/Motivation

Paragraphs promoted itself to a new top level option, in turn, Entity Reference Revisions should become less visible and be just another option in the Reference group, for the fairly rare cases of creating non-paragraph ERR fields.

Also, fixing 10.2 test fails.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Fixed

Version

1.0

Component

Code

Created by

🇨🇭Switzerland Berdir Switzerland

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @Berdir
  • Merge request !29Adjustments for Drupal 10.2 field UI → (Merged) created by Berdir
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7
    last update 8 months ago
    39 pass
  • Status changed to Needs work 8 months ago
  • 🇨🇭Switzerland Berdir Switzerland

    Started a merge request, but this is a bit of a chicken/egg situation together with paragraphs.

    To be able to promote paragraphs to a top-level option, we need it as a preconfigured option that we then move out.

    But by no longer promoting ERR to a top-level option/separate category, we don't really want to do preconfigured options anymore.

  • 🇷🇺Russia zniki.ru

    Should we adapt gitlab ci to run tests on D9 and next minor?

    variables:
      OPT_IN_TEST_PREVIOUS_MAJOR: 1
      OPT_IN_TEST_NEXT_MINOR: 1
    
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7
    last update 6 months ago
    39 pass
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7
    last update 6 months ago
    39 pass
  • Status changed to Needs review 6 months ago
  • 🇨🇭Switzerland Berdir Switzerland

    So, here's what this does now:

    * Either old string category or reference, reverted the logic from before, so we can remove the alter hook in the future, and the default is the new one.
    * Extend the preconfigured options to include a (revisions) suffix to make that clearer. We kind of need this still so paragraphs can move it out and only very few entity types use the common revisions refernce flag, so most people won't see that
    * Change the default option to Other (revisions).

    Will see what tests think about it, some explicit test coverage of this would be neat too, make sure the new labels show up as expected.

  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7
    last update 6 months ago
    39 pass
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7
    last update 6 months ago
    39 pass
    • Berdir committed d1ee7921 on 8.x-1.x
      Issue #3397044 by Berdir: Move field type into Reference group in new...
  • Status changed to Fixed 6 months ago
  • 🇨🇭Switzerland Berdir Switzerland

    Fixed that test fail, added a check on the label and merged.

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

Production build 0.69.0 2024