Account created on 10 April 2013, about 11 years ago
#

Merge Requests

Recent comments

πŸ‡¬πŸ‡§United Kingdom kalpaitch

I have tested this with a migration from 10.1 to 10.2.

This is a non-breaking change, thanks for @Eli-T for adding the note of cation, but 10.2 will not use the state for caching the asset filename see this comment from πŸ“Œ Remove the aggregate stale file threshold and state entry Fixed , good to merge, have upgraded to major because flysystem will not work with 10.2 without it:

The asset filename with the new system is cheaper to recreate, so we don't really need this extra level of i/o caching there at all.

πŸ‡¬πŸ‡§United Kingdom kalpaitch

Yea, the crown logo wasn't updated, so I've added that now.

πŸ‡¬πŸ‡§United Kingdom kalpaitch

kalpaitch β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom kalpaitch

Following on from @codebymikey's point #29 I've also identified this happens to the 'container' element too, (albeit with the caveat that the container element does not support the #description attribute):

$form['element'] = [
      '#type' => 'container',
      '#description' => 'A description added to the container.'
];

This will result in a container wrapper being rendered with an incorrect 'aria-describedby' attribute.

Following on from @andrewmacpherson's comment #22, I agree there needs to be a more extensible way to declare a form element as a composite form element. But there are also needs to be some consideration for other render elements which get the 'aria-describedby' attribute added.

πŸ‡¬πŸ‡§United Kingdom kalpaitch
πŸ‡¬πŸ‡§United Kingdom kalpaitch

kalpaitch β†’ created an issue.

πŸ‡¬πŸ‡§United Kingdom kalpaitch

Added the core_version_requirement update, can confirm this does the job for me.

πŸ‡¬πŸ‡§United Kingdom kalpaitch

kalpaitch β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom kalpaitch

Tested this and can confirm it resolves the fatal error. Seems to be a pretty straight forward fix to get in, especially as it breaks the current release otherwise.

πŸ‡¬πŸ‡§United Kingdom kalpaitch

I can confirm that this is an issue in 9.5 as per #3354857-5: A "tags" entry must be an array for service "search_api_opensearch.synonyms_subscriber" β†’ .

Regarding #3354857-4: A "tags" entry must be an array for service "search_api_opensearch.synonyms_subscriber" β†’ , whether or not this is the case this is not how it is treated in 9.5. All other tagged instances in core use the `- { name:...` syntax.

πŸ‡¬πŸ‡§United Kingdom kalpaitch

kalpaitch β†’ created an issue.

Production build 0.69.0 2024