- Issue created by @nambisolo
- First commit to issue fork.
- Status changed to Needs review
8 months ago 11:00am 6 June 2024 - Status changed to RTBC
8 months ago 1:31pm 11 June 2024 - 🇸🇮Slovenia primsi
Thanks for the patch. I tested it and it works.
That being said, I am not sure if there is a better place to fix that. I was checking \Drupal\content_workflow_bynder\MigrationDefinitionCreator::buildMigrationDefinition but that's probably not enough.
Still setting to RTBC, then let's see what @Berdir says.
- Status changed to Needs work
7 months ago 9:00am 1 July 2024 - 🇨🇭Switzerland berdir Switzerland
Agreed, I think this should be adjusted in \Drupal\content_workflow_bynder\MigrationDefinitionCreator::buildMigrationDefinition instead(), and convert the language identifier there to something that's a valid id.
- 🇨🇭Switzerland berdir Switzerland
We can extend/adjust \Drupal\Tests\content_workflow_bynder\Kernel\ConfigCreatorTest with a langcode that includes a -
- Status changed to Needs review
6 months ago 9:43am 31 July 2024 - 🇸🇮Slovenia primsi
Moved the logic and .... after some struggle ... extended the test.
- Status changed to Fixed
3 months ago 3:24pm 24 October 2024 -
berdir →
committed 960fd172 on 1.x authored by
immaculatexavier →
Issue #3452928 by immaculatexavier, berdir, primsi: Imports fail when...
-
berdir →
committed 960fd172 on 1.x authored by
immaculatexavier →
Automatically closed - issue fixed for 2 weeks with no activity.