Skip to content
Snippets Groups Projects

Unable to delete outdated records from the pipelines

Merged Jibran Ijaz requested to merge issue/data_pipelines-3405873:3405873 into 1.x

Signed-off-by: Jibran Ijaz jibran.ijaz@gmail.com

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Lee Rowlands
  • Lee Rowlands
  • Couple of minor nits but looks good

  • Jibran Ijaz added 5 commits

    added 5 commits

    Compare with previous version

  • Jibran Ijaz resolved all threads

    resolved all threads

  • Author Developer

    We still need a copy of \Drupal\Tests\data_pipelines\Kernel\DestinationQueueTest::testCleanup and \Drupal\Tests\data_pipelines\Kernel\DestinationQueueTest::testOutdatedCleanup in \Drupal\Tests\data_pipelines_elasticsearch\Kernel\ElasticsearchDestinationQueueTest.

    This is the only remaining bit. We are ready to merge after that.

  • Jibran Ijaz added 1 commit

    added 1 commit

    Compare with previous version

  • Jibran Ijaz added 1 commit

    added 1 commit

    Compare with previous version

  • Jibran Ijaz added 1 commit

    added 1 commit

    Compare with previous version

  • Jibran Ijaz added 1 commit

    added 1 commit

    Compare with previous version

  • Jibran Ijaz
  • Jibran Ijaz
  • Lee Rowlands
  • Looks good to go. The new method addition is fine in a minor release given we expect people to use the base plugin

  • Jibran Ijaz added 1 commit

    added 1 commit

    • d75b6b30 - Fixed the names of test traits.

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading