Problem/Motivation

The default_nodes_main variable does not have a migration to D8 (node.settings).

Proposed resolution

Create the migration.

Remaining tasks

Create the migration, write the test.

User interface changes

None.

API changes

None.

CommentFileSizeAuthor
#4 2353699-4.patch2 KBjarsenx
#3 2353699-3.patch2 KBjarsenx
#2 2353699-2.patch2 KBjarsenx

Comments

jarsenx’s picture

jarsenx commented
Assigned: Unassigned » jarsenx
jarsenx’s picture

jarsenx commented
Status: Active » Needs review
StatusFileSize
new 2353699-2.patch2 KB

Fixed migration issue for default_nodes_main variable.

jarsenx’s picture

jarsenx commented
StatusFileSize
new 2353699-3.patch2 KB

Updated patch based on results of code review.

jarsenx’s picture

jarsenx commented
StatusFileSize
new 2353699-4.patch2 KB

Cleaned up some extra spaces at the end of a line.

benjy’s picture

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

Thanks this looks good. Nice spot with the un-needed dependency on d6_node_type

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed 864a9b9 and pushed to 8.0.x. Thanks!

  • alexpott committed 864a9b9 on 8.0.x 
    Issue #2353699 by jarsenx | ultimike: D6->D8 Migration missing variable...

Status: Fixed » Closed (fixed)

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