Problem/Motivation

Add migration support for upgrading from the module version 7.x-1.x.

CommentFileSizeAuthor
#2 add_migration_support-2802407-2.patch9.77 KBhussainweb

Comments

hussainweb created an issue. See original summary.

hussainweb’s picture

hussainweb
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new add_migration_support-2802407-2.patch9.77 KB

Status: Needs review » Needs work

The last submitted patch, 2: add_migration_support-2802407-2.patch, failed testing.

hussainweb’s picture

hussainweb
Primary language English
commented
Status: Needs work » Needs review

The failures don't look related to this patch. It worked great in manual testing.

jonathan1055’s picture

jonathan1055 commented

Hi,
I had the same problem. In local testing on 8.1 my view definition was fine, and testing on drupal.org up to 25th Aug at 8.3 was also OK, but then some time between 25th Aug and 12th Sept, there must have been a core change which altered the schema checking. More details are on the Scheduler issue #2799095: Fix schema error in scheduled content view. Status value should be string not boolean

The problem I have now is that it is fixed for 8.2 / 8.3 on Drupal.org but I cannot run the tests locally at 8.1. For the announcments of drupal-8-2-0-rc1 it states

Drupal 8.2.x is backwards-compatible with 8.1.x

but this problem shows that this is not the case.

Maybe we should create an issue on core 8.2 to see if there is some way that the schema checking can be consistent at 8.1 and 8.2/8.3