The request notification contains the wrong user name; it should display the name of the administrator instead of the name of the person who requested it.
I found two cases where the notification contains incorrect data; see the merge request (MR).
Issue: If I approve a rejected request, the system will not send a notification. i expect that system sends "Approve notification"
I was able to use "private://..." as path for keys, tested on the latest v6.
## Performance impact
**Before fix:** Route lookups (`pathValidator->getUrlIfValidWithoutAccessCheck()` + `routeProvider->getRouteByName()`) occur on EVERY request, regardless of whether OAuth is being used.
**After fix:** Route lookups only occur when a Bearer token is present (typically <1% of requests on most sites). For non-OAuth requests, the method returns FALSE immediately after a simple header check.
For some reason, I cannot set credits: you need to be a maintainer of the module to make changes. Leaving it as “Reviewed and tested by the community.”
I performed manual tests—import and export—and did not notice any issues.
alorenc → created an issue. See original summary → .
The module is not yet stable, so I assume that we do not need an update script.
alorenc → changed the visibility of the branch 3547582-link-widget-placeholderurl to hidden.
babel_update_8001 removes duplication
Duplicated are not longer created
I performed manual tests: hook removes OG menus. I uninstalled and reinstalled the module a few times in order to test the batch process. I also updated a few menu items. It looks fine, thanks.
I can see warning messages during the import if any occur.
Is there any reason why the README.md file was not updated as well?
I can still see many references to the "4.32.2" library version.