- Issue created by @wells
- Status changed to Closed: works as designed
over 1 year ago 4:39pm 1 September 2023 - Status changed to Active
over 1 year ago 5:01pm 1 September 2023 - 🇺🇸United States wells Seattle, WA
Ah but this is still an issue on the 8.x-1.x branch...
- @wells opened merge request.
- 🇺🇸United States wells Seattle, WA
And here's a plain diff of the MR since tests are run automatically.
- last update
over 1 year ago 25 pass - Status changed to Needs review
over 1 year ago 5:16pm 1 September 2023 - Status changed to RTBC
over 1 year ago 1:35pm 6 September 2023 - 🇧🇷Brazil elber Brazil
Hi I reviewed it using drupal 9.5 version.
Testing /app/web/modules/contrib/dfp ......................... 25 / 25 (100%) Time: 00:44.676, Memory: 16.00 MB OK (25 tests, 154 assertions)Tests are passing now, moving to RTBC
- First commit to issue fork.
-
VladimirAus →
committed 585f498d on 8.x-1.x authored by
wells →
Issue #3384851 by wells: $defaultTheme missing for tests
-
VladimirAus →
committed 585f498d on 8.x-1.x authored by
wells →
- Status changed to Fixed
over 1 year ago 1:52am 15 September 2023 Automatically closed - issue fixed for 2 weeks with no activity.