- Merge request !9Issue #3264595 by dydave, flyke: Added dependency on the Core Toolbar module... → (Merged) created by dydave
- 🇫🇷France dydave
Thanks a lot Joery (@flyke) for creating this issue and documenting it very clearly, it's greatly appreciated.
The problem and solution suggested in the issue summary completely make sense, indeed.
Based on your suggestion, I created merge request MR!9 above at #3, which adds a dependency to the Core Toolbar module library for the
ToolbarModelclass, see:
https://git.drupalcode.org/project/drupal/-/blob/11.x/core/modules/toolb...I took the opportunity to add a few minor additional changes to the module's Tests class, see the merge request description for more details.
Since all the Tests were passing 🟢, I went ahead and merged the changes above at #4.
Marking issue as Fixed for now.
Feel free to let us know if you have any questions or concerns on any of the recent code changes or the module in general, we would surely be glad to help.
Thanks in advance!