Update drupal/admin_toolbar from 3.4.2 to 3.6.2#362
Open
violinist-bot wants to merge 1 commit intoshaal:mainfrom
Open
Update drupal/admin_toolbar from 3.4.2 to 3.6.2#362violinist-bot wants to merge 1 commit intoshaal:mainfrom
violinist-bot wants to merge 1 commit intoshaal:mainfrom
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
If you have a high test coverage index, and your tests for this pull request are passing, it should be both safe and recommended to merge this update.
Updated packages
Some times an update also needs new or updated dependencies to be installed. Even if this branch is for updating one dependency, it might contain other installs or updates. All of the updates in this branch can be found here:
Release notes
Here are the release notes for all versions released between your current running version, and the version this PR updates the package to.
List of release notes
Changed files
Here is a list of changed files between the version you use, and the version this pull request updates to:
List of changed files
Changelog
Here is a list of changes between the version you use, and the version this pull request updates to:
Issue #3537721 by dydave: Admin Toolbar Search: Removed dependency on Admin Toolbar Tools.Issue #3532010 by phily, dydave, benjifisher: Dropped support for Core Update module routes 'update.module_update' and 'update.theme_update' for Core versions above 11.2.Issue #3532249 by anybody, dydave: Fixed JS error 'Uncaught TypeError: toolbarElement.querySelector(...) is null' when selected HTML IDs are not found in the page.Issue #3527344 by justcaldwell, erutan, dydave: Fixed support for new keyboard shortcuts for MacOS.Issue #3527462 by ressa, dydave: Fixed import of module's translated strings from localize failing due to malformed HTML tag '<br/>'.Issue #3528007 by megakeegman, dydave: Reverted change from DO-3518123, since 'DeprecationHelper' class is not available before Drupal 10.2, thus breaking compatibility with older core versions.Issue #3527315 by pwolanin: Use defensive coding to prevent warning 'Trying to access array offset on value of type null' when configuration is not updated correctly.Initial implementation of the 'toggle_shortcut' feature, based on MR 70.Issue #3487246 by dydave, ericmulder1980: Warn site owners about the removal...Issue #3524284 by dydave: Removed file 'CHANGELOG.txt'.Issue #3523066 by chhavi.sharma, dydave, ressa: Updated Admin Toolbar Search settings page path to remove trailing '-settings'.Min & Max valuesIssue #3506824 by dydave: Fixed compatibility with Project Browser following...Issue #3522548 by ressa, dydave: Increased color contrast of search result links and updated search field placeholder label to include shortcut.Issue #3520680 by ressa, dydave: Improved Admin Toolbar settings form page by adding basic help text and moving around fields or options.Issue #3522518 by dydave: Removed file 'LICENSE.TXT' from module's code base.Issue #3519815 by dydave, tirupati_singh, ressa: Added menu tabs (local tasks) for Admin Toolbar Settings forms.Issue #3494646 by dydave, ressa: Added keyboard shortcut 'Alt + a' to focus Admin Toolbar Search field.Issue #3519967 by dydave: Added initial Functional Tests for the 'AdminToolbarSearchSettingsForm'.Issue #3440852 by dydave, ressa, aaron.ferris: Added configurable hoverIntent 'timeout' setting.Issue #3509584 by dydave, ressa, darkdim: Added toolbar sticky behavior setting to 'Hide on scroll-down, show on scroll-up'.Issue #3452561 by bwaindwain, dydave: Reverted changes from 'Hide the navigation menu on mobile' causing conflicts with default toolbar behavior.Issue #3518123 by dydave: Refactored method 'flushJsCss' with 'backwardsCompatibleCall' to support deprecated '_drupal_flush_css_js' for versions below core:10.2.Issue #3516995 by dydave, ressa: Moved the 'hoverintent_functionality' setting out of 'admin_toolbar_tools' to 'admin_toolbar'.Issue #3517506 by dydave: PHPSTAN: Raised validation level from 0 (min) to 6.Issue #3514075 by benstallings, dydave: Fixed PHPSTAN validation error 'Unsafe usage of new static()'.Issue #3514574 by ankondrat4, dydave: Improved Admin Toolbar Tools 'Flush CSS and Javascript' operation (method 'flushJsCss()') to fully invalidate JS/CSS caches and regenerate aggregated files.Issue #3514774 by dydave: Added basic PHPUNIT Tests coverage for 'admin_toolbar_tools' menu links and 'ToolbarController' class.Issue #3436637 by giannis makantasis, dydave, connbi: Fixed CSS style for active toolbar menu links arrow disappearing.Issue #3513588 by andresalvarez, dydave: Updated the 'jquery-hoverIntent' JS library from 'v1.8.1' to 'v1.10.2'.Issue #3512602 by andresalvarez, dydave: Fixed PHPSTAN (next minor) validation errors.Issue #3255672 by dydave, ressa, silvi.addweb, japerry: Added configuration setting to enable/disable sticky/fixed toolbar behavior.Issue #3507579 by tonytheferg, dydave, berdir: Fixed Fatal error 'Controller for URI /en/admin/flush is not callable' by reverting change introduced in #3486841 with commit f6acda66.Issue #3508173 by dydave: Fixed GitLab CI configuration causing jobs to fail and adjusted 'AdminToolbarSettingsFormTest' to be compatible with supported versions.Issue #3493798 by ressa, shivam_tiwari, paul_serval, estoclet: Fixed compatibility issue: Route 'update.theme_install' does not exist in Drupal 11.0.x and 10.4.x.Issue #3466743 by DYdave: Added initial automated tests for the 'AdminToolbarToolsSettingsForm'.Issue #3494654 by ressa, dydave: Fixed broken tests by fixing indentation and PHPCS errors.3486841: Add menu item for re-index siteIssue #3474908 by japerry, hswong3i: Remove the now defunct menu items for Drupal 11 sitesIssue #3471798 by chr.fritsch: Admin Toolbar Links Access Filter was not marked D11 compatibleIssue #3465604 by dydave: TypeError: method_exists(): Argument #1 ($object_or_class) must be of type object|string, null givenIssue #3466125 by dydave, japerry, teknocat: [AdminToolbarSettingsForm] Fix Fatal Error: Call to undefined method Drupal\Core\Menu\MenuLinkManager::invalidateAll()Issue #3465705 by vishal.kadam: Toolbar image missingIssue #3411005: GitlabCI: Fix PHPCS and PHPStan validation errorsIssue #3420788: Search links always open new window / tabIssue #3465313: Max Bundles not being honoredIssue #3463378: GitlabCI: Fix CSPELL validation errorsIssue #3391853 by YazanMajadba: The arrow is rotated in [RTL] when I focused on itIssue #3336787: TypeError in commerce_product if enable admin_toolbar_extraIssue #3427112: Do not checkNamedRoute if route_name is NULLIssue #3457902: Forward Compatibility with Project BrowserIssue #3317002: Access search field via a keyboard shortcut with accesskeyIssue #3416954: [admin_toolbar_tools] Don't add disabled views to the admin menuIssue #3397570: Add check for Undefined array key "#active".Issue #3446836: Incorrect logger name in admin_toolbar_links_access_filter module.Issue #3462559 by DYdave: Link to "Block" content overview page not displayedIssue #3463291 by japerry: Deprecate the admin_toolbar_links_access_filter moduleIssue #3462195: GitlabCI: Fix Stylelint validation errorsIssue #3363604: Add Drupal 11 supportFix TestsAdd Gitlab ci testingWorking with this branch
If you find you need to update the codebase to be able to merge this branch (for example update some tests or rebuild some assets), please note that violinist will force push to this branch to keep it up to date. This means you should not work on this branch directly, since you might lose your work. Read more about branches created by violinist.io here.
This is an automated pull request from Violinist: Continuously and automatically monitor and update your composer dependencies. Have ideas on how to improve this message? All violinist messages are open-source, and can be improved here.