Problem/Motivation

Transliteration is now in core and always available.

Proposed resolution

Update default config so that transliteration is enabled by default. Remove the part about transliteration module from the description.

Remaining tasks

User interface changes

API changes

Data model changes

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir created an issue. See original summary.

sdstyles’s picture

Remove the part about transliteration module

Do you mean to remove description about transliteration module from pathauto page?

Berdir’s picture

Status: Needs review » Needs work

Yes exactly. That's not true, we don't need that module anymore.

sdstyles’s picture

Status: Needs work » Needs review
FileSize
1.28 KB

I've removed description too. Please take a look.

Berdir’s picture

Status: Needs review » Fixed

Thanks, committed.

  • Berdir committed 26ce13b on 8.x-1.x authored by sdstyles
    Issue #2655776 by sdstyles: Enable transliteration by default
    
Dave Reid’s picture

@Berdir: Can we keep patches like this open for review for 24 hours or so? I have a hard time reviewing anything in such short windows.

Berdir’s picture

Sure. I think this was trivial and already discussed elsewhere. I also didn't wait with the upgrade path fixes as I thought it's better to have them in asap. But I definitely planned on waiting for anything non-trivial, especially if I write it myself.

But I'll wait with anything at least 24h.

Status: Fixed » Closed (fixed)

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