Problem/Motivation

i18n user settings are not migrated.

Proposed resolution

Migrate i18n variable, user settings to config.

References:

Remaining tasks

Write migration.
Write tests, includes updating the drupal6 test fixture. See Generating database fixtures for D8 Migrate tests.

User interface changes

N/A

API changes

N/A

Data model changes

N/A

CommentFileSizeAuthor
#4 2655346-4.patch10.92 KBquietone

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone commented
Issue tags: +i18n-migrate

Added tag.

quietone’s picture

quietone commented
Issue tags: +migrate-d6-d8
quietone’s picture

quietone commented
Version: 8.0.x-dev » 8.1.x-dev
Status: Active » Needs review
Related issues: +#2655348: i18n Variable to config: user mail [d6]
StatusFileSize
new 2655346-4.patch10.92 KB

On closer look at MigrateUserConfigsTest, it tests both the user settings and user mail settings. So, moving the work already done in #2655348: i18n Variable to config: user mail [d6] to this issue, so the resulting test is consistent with the corresponding d6 one. And there is no need for two issues for these straight forward migration tests.

benjy’s picture

benjy commented
Status: Needs review » Reviewed & tested by the community

I think we may as well get these in so people can start testing them.

+++ b/core/modules/config_translation/tests/src/Kernel/Migrate/d6/MigrateI18nUserConfigsTest.php
@@ -0,0 +1,79 @@
+/**
+ * @file
+ * Contains \Drupal\Tests\config_translation\Kernel\Migrate\d6\MigrateI18nUserConfigsTest.
+ */

Un-needed comment

  • catch committed 87f6b35 on 8.2.x 
    Issue #2655346 by quietone, benjy: i18n Variable to config: user...

  • catch committed 032ebbb on 8.1.x 
    Issue #2655346 by quietone, benjy: i18n Variable to config: user...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.2.x and cherry-picked to 8.1.x. Thanks!

Status: Fixed » Closed (fixed)

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