- Issue created by @majorrobot
- πΊπΈUnited States majorrobot
Sry, somehow assigned this to the wrong person.
- @majorrobot opened merge request.
- πΊπΈUnited States majorrobot
This is ready for review. Here's an easy way to QA:
ddev drush php-eval "print_r(\Drupal::service('ai_migration.ai_migrator')->convert('https://example.com'));"
Replace example.com with any url.
- πΊπΈUnited States majorrobot
Please note that I am aware of the PHPStan issues -- I did not use DI with the logger service in order to quickly get logging in. I can refactor this to DI later.
- πΊπΈUnited States dmundra Eugene, OR
Looking good @majorrobot and thank you for the easy to follow documentation. Left a few comments and here my tests:
Working URL:
dmundra in ~/workspace/community/ai_migration on branch 3537651-confirm-minimal-ai > ddev drush php-eval "print_r(\Drupal::service('ai_migration.ai_migrator')->convert('https://ndhurandhar.art/art/2025-06/artist-impression-rex'));" ```json { "title": "Artist impression of Rex" } ```%
Non-existent URL:
dmundra in ~/workspace/community/ai_migration on branch 3537651-confirm-minimal-ai > ddev drush php-eval "print_r(\Drupal::service('ai_migration.ai_migrator')->convert('https://ndhurandhar.com'));" [error] Failed to fetch content from URL https://ndhurandhar.com: cURL error 6: Could not resolve host: ndhurandhar.com (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for https://ndhurandhar.com
- πΊπΈUnited States majorrobot
Thx @dmundra. Merged and marking as Fixed.
-
majorrobot β
committed a59a71ac on 1.0.x
Issue #3537651: Update for a minimal API call and JSON response
-
majorrobot β
committed a59a71ac on 1.0.x
Automatically closed - issue fixed for 2 weeks with no activity.