Skip to content
Snippets Groups Projects

V1.14.0 Remove custom logger to support Drupal>7

Closed Rich requested to merge v1.14.0 into main
All threads resolved!

Re:

Hello Drupal 8+ users, would any of you mind testing this branch? @pradeep @seamuslee @sluc23 @ufundo

Changes:

  • only require_once the vendor libs if they exist in the extension's dir. So you can have the libs in a central place somewhere else.
  • Add status check for class_exists(\GoCardlessPro\...) so it's clear when an install is broken.
  • Remove custom logger; fixing signature incompatibilities.
  • phpunit test environment only: dependency on taggedlog extension. I have addressed the signature problems therein (see how).
Edited by Rich

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
  • Rich resolved all threads

    resolved all threads

  • Thanks @ufundo for testing this PR :heart_eyes:

    I'll probably release this in a week or so, to give anyone else chance to test it.

  • Closing in favour of !130 (closed) (which includes the changes here too).

  • closed

  • Please register or sign in to reply
    Loading