- ๐ณ๐ฟNew Zealand quietone
@dsdeiz, Sorry, I do not recall what I did to force the error.
I took another look here and then went ahead and added the change I proposed in #7. I also updated \Drupal\Tests\migrate\Unit\process\MigrationLookupTest::testTransformWithStubbing to use a data provider with two cases for this change.
- Merge request !6764Resolve #3197915 "skip row messages caused by stub creation need more detail" โ (Closed) created by quietone
- Status changed to Needs work
10 months ago 7:15am 26 February 2024 - ๐บ๐ธUnited States dww
This came up as a random #bugsmash triage target, which is where the new life is coming from. Mostly looks fine. Added a few MR suggestions for some nits.
- ๐ฎ๐ณIndia TanujJain-TJ
Tanuj. โ made their first commit to this issueโs fork.
- Status changed to Needs review
10 months ago 11:40pm 28 February 2024 - Status changed to RTBC
10 months ago 3:49pm 29 February 2024 - ๐บ๐ธUnited States smustgrave
Had to run locally as I couldn't run the test-only feature.
Appears this has already gone through a few rounds of reviews and appears the feedback has been addressed.
- Status changed to Needs work
10 months ago 11:05pm 2 March 2024 - ๐ฌ๐งUnited Kingdom longwave UK
Appears to be one out of scope change in the MR.
- Status changed to RTBC
10 months ago 11:11pm 2 March 2024 - ๐ฌ๐งUnited Kingdom longwave UK
Applied my own suggestion to save time here.
-
longwave โ
committed 084176b4 on 10.3.x
Issue #3197915 by quietone, joachim, Tanuj., karishmaamin, dww: Add...
-
longwave โ
committed 084176b4 on 10.3.x
-
longwave โ
committed f21bb25c on 11.x
Issue #3197915 by quietone, joachim, Tanuj., karishmaamin, dww: Add...
-
longwave โ
committed f21bb25c on 11.x
- Status changed to Fixed
10 months ago 10:22am 3 March 2024 - ๐ฌ๐งUnited Kingdom longwave UK
Committed and pushed to 11.x and 10.3.x. Did not backport cleanly to 10.2.x, this is only a minor improvement so not sure it is worth backporting there.
Automatically closed - issue fixed for 2 weeks with no activity.