- Issue created by @dydave
- Merge request !37Issue #3527537 by dydave, dgwolf: Fixed compatibility with Admin Toolbar 3.6.x... → (Merged) created by dydave
- 🇫🇷France dydave
Quick follow-up on this issue:
Created the initial merge request MR !37 above at #2 which :
Converts the
__construct
method tocreate
and all the existing code or logic was kept pretty much "as-is".After testing the changes locally, the error disappeared and the expected menu items seemed to display properly under Content, for example: this patch should fix the fatal error.👌
Moving issue to Needs review for now, as an attempt to attract more attention, reviews, testing and reporting feedback.
Feel free to let us know if you have any questions on this issue or the changes suggested in the merge request, we would certainly be glad to help.
Thanks in advance! Thanks a lot, @dydave! Your patch for admin_toolbar_content works very well here!
- First commit to issue fork.
-
kriboogh →
committed 5cc55118 on 2.0.x authored by
dydave →
Issue #3527537: Compatibility with Admin Toolbar 3.6.0
-
kriboogh →
committed 5cc55118 on 2.0.x authored by
dydave →
- 🇫🇷France dydave
Wow Kris (@kriboogh), thanks a lot for the very quick feedback and follow-up on this! 🥳
Since the module is marked Minimally maintained, I was certainly not expecting this fix to be merged so quickly.
Thank you very much for the prompt, positive feedback and for creating a release right away.
Thanks also for granting us (@dgwolf and I) the credits on the commit and this issue.
Cheers!