Antwerp, BE
Account created on 16 October 2013, almost 12 years ago
  • Drupal Developer at iO 
#

Merge Requests

More

Recent comments

🇧🇪Belgium baikho Antwerp, BE

Yes, at the moment you’d either need to fork the GitHub repository and build the JS bundle with translatable strings, or implement a proxy endpoint in Drupal that serves translated data. I’m happy to review merge requests if someone would like to contribute an alternative implementation.

🇧🇪Belgium baikho Antwerp, BE

Follow up issue here 📌 Fix PHPStan violations Active for phpstan corrections after the merge.

🇧🇪Belgium baikho Antwerp, BE

Also tested this and it works fine. If someone else could also review and move to RTBC please.

🇧🇪Belgium baikho Antwerp, BE

baikho made their first commit to this issue’s fork.

🇧🇪Belgium baikho Antwerp, BE

baikho created an issue.

🇧🇪Belgium baikho Antwerp, BE

baikho created an issue.

🇧🇪Belgium baikho Antwerp, BE

Not gaining as many code contributions from a region can’t be used to dismiss or downplay genocide. That’s a deeply flawed and unjust metric. It takes no effort to be human and acknowledge suffering, to show solidarity, and to affirm that Palestinian lives matter just as much as any others.

🇧🇪Belgium baikho Antwerp, BE

There shouldn’t even be a debate about whether or not to show solidarity with Palestine... We are witnessing one of the biggest war crimes in modern history, the fact that this even needs justification is beyond understanding.

🇧🇪Belgium baikho Antwerp, BE

baikho changed the visibility of the branch 3536544-expose-migration-in to hidden.

🇧🇪Belgium baikho Antwerp, BE

Changes in the MR work fine. Tested on:
Drupal core 11.2.2
Pathauto 8.x-1.13

🇧🇪Belgium baikho Antwerp, BE

#13

Maybe the SkipRow exception should be re-thrown as a SkipProcess exception. Or maybe we can stop using exceptions after 📌 Allow process plugins to flag a row to be skipped Needs work : Allow process plugins to flag a row to be skipped is fixed.

Yes, or something along these lines seems to work too:

+ $migrate_executable->saveMessage($new_message, MigrationInterface::MESSAGE_INFORMATIONAL);
+ $this->stopPipeline();
+ return NULL;
- throw new MigrateSkipRowException($new_message, 0);
🇧🇪Belgium baikho Antwerp, BE

Addressed a few remarks from #4 and #5.

Tests still need updating.

🇧🇪Belgium baikho Antwerp, BE

baikho made their first commit to this issue’s fork.

🇧🇪Belgium baikho Antwerp, BE

#9 Thanks @benjifisher, I have updated the IS accordingly.

🇧🇪Belgium baikho Antwerp, BE

Please see the Project page for details:

See CSRF Anonymous Token for the Drupal 8 variant of this module's 7.x-1.x branch.

🇧🇪Belgium baikho Antwerp, BE

#35 works fine, but made a slight adjustment for multiple migrate sources 🐛 ParagraphsSubProcess plugin does not merge paragraph items when using += operator Active

🇧🇪Belgium baikho Antwerp, BE

Linking this up with Show migrate plugins on the Migrate UI lists Active and UI will not display migrations that are not in configuration. Active as I have not found any UI solution that displays migration Plugins.

🇧🇪Belgium baikho Antwerp, BE

baikho changed the visibility of the branch 3037234-give-authentication-plugins to hidden.

🇧🇪Belgium baikho Antwerp, BE

Using patch from #105, works as expected on:

Drupal Core 11.1.7
PHP v8.3

🇧🇪Belgium baikho Antwerp, BE

#4

It's a new installation:

$ composer require drupal/entity_usage
...
www-data@2a078b834d81:/docroot$ drush en paragraphs_library
The following module(s) will be enabled: paragraphs_library, entity_usage
...
🇧🇪Belgium baikho Antwerp, BE

The change in the MR work fine, but I had to re-install the module before the entity route changes applied.

🇧🇪Belgium baikho Antwerp, BE

I think this should be considered as a bug

🇧🇪Belgium baikho Antwerp, BE

#3 I noticed the flag, but didn't read the description. However when trying to remove the flag and re-running, doesn't appear to work. For now I am sticking to the patch, will do some more investigation later. Thanks!

🇧🇪Belgium baikho Antwerp, BE

baikho created an issue.

🇧🇪Belgium baikho Antwerp, BE

@cmlara could we please have a new tagged release? Thanks!

🇧🇪Belgium baikho Antwerp, BE

baikho changed the visibility of the branch 3517986-entity-finder-plugin to hidden.

🇧🇪Belgium baikho Antwerp, BE

I tested patch from #18 on v3.0.0, but that is still reproducing the behaviour described as in #6

But the first imported german node is gone, and was replacing through the english version.

Or what I think is happening is that the original source node's language get swapped with the overridden language instead.

🇧🇪Belgium baikho Antwerp, BE

Just adding my two cents — I think the solution in Support Batch Processing Active could serve as an example for implementing something similar for JSON:API.

And then there is also https://www.drupal.org/project/feeds_paged_drupal_jsonapi_fetcher , although it looks quite outdated.

🇧🇪Belgium baikho Antwerp, BE

baikho made their first commit to this issue’s fork.

Production build 0.71.5 2024