Problem/Motivation

It's time to move drupal.be under the la_eu umbrella by creating drupal_be config split

Steps to reproduce

At the moment drupal.be runs a "forked" repository version. In order to make it part of the ecosystem, drupal_be config split must be added

Proposed resolution

  • Create new drupal_be config split
  • Make sure the split has the minimum changes

Remaining tasks

This MR requires two other MRs to go in first, so the split is simple:

  1. #3565065: Configure content translation
  2. #3565238: Make views language aware
  3. #3566619: Disable Advanced CSS/JS Aggregation module

    4. (it's disabled on prod; and causes issues for drupal.nl at least)

User interface changes

None

API changes

None

Data model changes

None

Issue fork la_eu-3566627

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

taran2l created an issue. See original summary.

taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Parent issue: #3565254: Create config split for drupal.de »
Related issues: +#3565254: Create config split for drupal.de

taran2l opened merge request !59

taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Title: [PP-2] Create config split for drupal.be » [PP-3] Create config split for drupal.be
Issue summary: View changes
taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented

The split does not contain any config translations yet, afaik - la_be does not have them too, see https://git.drupalcode.org/project/la_be/-/tree/live/config?ref_type=heads

taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Issue summary: View changes
taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Title: [PP-3] Create config split for drupal.be » [PP-2] Create config split for drupal.be
evilargest’s picture

evilargest commented
Status: Postponed » Active
evilargest’s picture

evilargest commented
Status: Active » Reviewed & tested by the community

The split looks fine and works as expected. Moving the issue to RTBC

svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented
Issue summary: View changes
svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented
Issue summary: View changes
svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented

I noticed the dates are also still formatted in a US style, e.g. medium date format is D, m/d/Y - H:i
Do we need to override this for BE specifically, or do we change this for la_eu as a whole? (latter seems best, since most people in EU use a date format like d/m/Y?)

taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Title: [PP-2] Create config split for drupal.be » Create config split for drupal.be

  • taran2l committed 616d21f9 on staging 
    feat: #3566627 Create config split for drupal.be

By: taran2l
By:...
taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • taran2l authored d80212ca on staging 
    feat: #3566627 Create config split for drupal.be

By: taran2l
By:...

Status: Fixed » Closed (fixed)

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

svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented
Status: Closed (fixed) » Needs work

When testing on the test environment provided by Lemberg, I noticed I couldn't reach translated content after doing the translation of a node.
After I went over to admin/config/regional/language/detection/url and added nl / fr / de path prefixes, this was resolved.
Could those also be added to the split?

evilargest opened merge request !72

evilargest’s picture

evilargest commented
Status: Needs work » Needs review

Good catch @svendecabooter! I faced the same when I tried to figure out how the search gonna work on the BE database, and when I found this issue to see why the language negotiation config was not added u were an hour and a half ahead of me (:

Added a config and the translations started showing as expected

svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented
Status: Needs review » Reviewed & tested by the community

Looks good to me!

evilargest’s picture

evilargest commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • taran2l authored d80212ca on 1.0.x 
    feat: #3566627 Create config split for drupal.be

By: taran2l
By:...

  • taran2l authored d80212ca on downstream/la_nl 
    feat: #3566627 Create config split for drupal.be

By: taran2l
By:...

Status: Fixed » Closed (fixed)

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