- Issue created by @tohesi
- Assigned to akshaydalvi212
- Merge request !22#3467777: Declaring missing properties to avoid error messages about creation... → (Open) created by tohesi
- 🇫🇮Finland tohesi
Sorry @akshaydalvi212 , didn't notice you picked this up so fast (and forgot to assign this to myself). I already created a quick PR to fix the deprecation error messages.
- Merge request !23Issue #3360742 by gilles_webstanz: Version 4 of the module is compatible with... → (Open) created by akshaydalvi212
- 🇮🇳India akshaydalvi212
Hey @tohesi,
Sorry, I also tried removing the deprecation errors I encountered on my local with the new branch PR.
Adding the image as an attachment when I encountered the error on the configuration setting page of Fastly.
- Issue was unassigned.
- Status changed to Needs review
7 months ago 1:01pm 13 August 2024 - 🇪🇸Spain niteman
I think merge request !22 (@tohesi's) is the way to go since it addresses the appropriate branch, can someone test it? (The change is so minimal I wonder why it didn't get merged)
- 🇳🇱Netherlands Johan den Hollander
Tested MR #22 which surely is the cleanest solution to make the deprecation warnings go away.
- 🇳🇱Netherlands Johan den Hollander
After going through the settings again today I still get this warning:
Deprecated: Creation of dynamic property Drupal\fastly\VclHandler::$snippetData is deprecated in /var/www/html/web/modules/contrib/fastly/src/VclHandler.php on line 1259