@duaelfr can you update the merge request target to branch 3.0.x? This option is only available for the creator (or maintainers)
@dewalt can you tag a new release, so this module can be installed with Drupal 11?
Rerolled against the latest 6.0.x
The merge request fixes the PHP warnings when a view has multiple Facet summaries
The merge request changes the button text to use the default when adding a content translation for a group.
The dependency is implicit since the config file name starts with rabbit_hole
idebr → created an issue.
- simple_oauth_cron() now checks for falsy values for token_cron_batch_size and defaults to 0
- token_cron_batch_size is now a required field on the config form, since it is not nullable
- Added the 0 limit to the description of the form item on the config form and schema
The table class was updated from sticky-enabled to sticky-header in Drupal 10.3, see
https://www.drupal.org/node/3440477 →
The merge requests implements the following changes:
- The 'Issued by' (iss) claim is now generated without a language prefix
- The OpenIdConnectTest now checks for the correct URL, see https://git.drupalcode.org/issue/simple_oauth-3502576/-/jobs/4153025
The merge request updates the dataProvider to be a static method.
idebr → created an issue.
idebr → created an issue.
The z-indexes in Drupal are documented → and considered stable.
I'm unable to reproduce this issue on 11.x, .ui-widget-overlay gets a z-index of 1259 and is displayed over the sticky header
@eojthebrave Is it possible to tag a new release, so downstream modules that require a stable release for Consumers have a green pipeline again? See for example https://git.drupalcode.org/project/simple_oauth/-/jobs/3848264
The test checks for the aria-describedby attribute. This attribute is only available when the field has a description.
The remaining deprecations are now fixed in the merge request
The merge request updates configuration forms to use #config_target
The merge request raises the minimum supported Drupal version to 10.3: https://endoflife.date/drupal
The logger can now be autoconfigured, see https://www.drupal.org/node/3395436 →
I have created a merge request to either fix or disable cspell. Only one should be merged.
PHPUnit failures are caused upstream, see 📌 PHP 8.4: Implicitly nullable parameter declarations deprecated Active
Considering Drupal 11 has been released a while ago, this issue can now be marked as fixed?
The merge request was not correctly linked to this issue. It is available at https://git.drupalcode.org/project/consumers/-/merge_requests/20
This issue also happens to be caught by phpcs, so I fixed these issues in a single go
The merge request removes lingering 'priority' keys from node access records
🌱 Move to thephpleague/oauth2-server 9.0 Active has been committed. However, there is no upstream support yet for introspection
The change is available in 📌 Fix phpunit test Active for 6.0.x
More test failures due to 📌 Define 'original' as property on the entity object Needs work
Test failure due to 🐛 Password and confirm password should be mandatory fields while setting up password using one time link following by email Needs work
The merge request adds the following changes:
- The configuration form now uses #config_target
- The default configuration syntax now matches the config when first pressing 'Submit configuration'
In the merge request ctools is no longer installed when testing pathauto
I converted the patch to a merge request: https://git.drupalcode.org/project/tmgmt/-/merge_requests/86
The change works as expected.
idebr → made their first commit to this issue’s fork.
Sure, that makes sense. Can completing the test coverage be done in a follow-up issue in 2.3.x?
@steffenr The merge request seems pretty complete. Is there anything that needs to be changed before a merge? The official PHP client would be very welcome when working on ✨ Support translation of documents Active
The conversion exposes a bug in the config schema where tfa.settings:mail is defined as a config object:
Symfony\Component\Validator\Exception\LogicException: The LangcodeRequiredIfTranslatableValues constraint is applied to 'tfa.settings::mail'.
This should be fixed in this issue to make the tests pass
greggles → credited idebr → .
gitlab-ci.yml was added in 🐛 Kernel Tests: Errors and Deprecations Fixed
This issue was fixed in 🐛 Update to work with latest Domain dev version Fixed
The merge request adds the following changes:
- All configuration forms now use #config_target
- All configuration forms have a Configure link on the Modules list
- The default configuration syntax now matches the config when first pressing 'Submit configuration'
The reroll in #80 is missing the SimplesitemapMetatagTest from #79
This issue was fixed in ✨ Use core route provider with addExtraCacheKeyPart for route caching? Needs review
The merge request removes mpyw/phpunit-patch-serializable-comparison as a dev dependency
Update the merge request, so drupal/video_embed_field can now be installed with ^2.5 or ^3
Before
Time: 00:28.525, Memory: 12.00 MB
After
Time: 00:09.647, Memory: 12.00 MB
Postponed on 📌 Test with drupal/video_embed_field 3.x for Drupal 11 compatibility Active to conform tests are green on both D10 and D11
The merge request updates the composer package for drupal/video_embed_field to 3.x for Drupal 11 compatibility
I think this according to the RSS specs? See https://www.rssboard.org/rss-profile#element-channel-image
<image>
<link>http://dallas.example.com</link>
<title>Dallas Times-Herald</title>
<url>http://dallas.example.com/masthead.gif</url>
<description>Read the Dallas Times-Herald</description>
<height>32</height>
<width>96</width>
</image>
📌 Implement js- prefix for linkit textfields to separate JavaScript behavior from styling Needs review essentially does the same, but without duplicating Claro CSS
This issue was fixed in Drupal Core, see 📌 Add aria-current attribute to navigation items RTBC
There is an earlier issue for concurrent tests available at 📌 Enable concurrent phpunit Active that needs work.
I'll close this issue as a duplicate, so we can focus our efforts in the related issue.
MR 10182 implements an alternative solution where the code ends up as follows:
$form['parent']['akey']['asubkey'] = [
'#type' => 'text_format',
'#title' => $this->t('Subkey title'),
'#format' => $this->config('mymodule.settings')->get('akey.asubkey.format'),
'#config_target' => "mymodule.settings:akey.asubkey",
];
I'm not sure ConfigTarget was intended to control any element property other than #default_value when reading the change record at https://www.drupal.org/node/3373502 →
Another interesting follow-up where a config value has to be mapping to multiple element properties: 🐛 Two #config_targets error when used on a text_format form element Active
Postponed on ✨ Add a status/published field in a consumer Needs review
The 6.x branch is not currently under development.
phpcs fails in the MR
The merge request updates thephpleague/oauth2-server to 9.0.x
Breaking changes are listed at the release page https://github.com/thephpleague/oauth2-server/releases/tag/9.0.0, but most notably:
- New: Strict typing and return types
- Changed: some exceptions return a different status code and the error message data structure has changed
- Refresh token scopes are now finalized again, so any invalid scopes are removed (this was the last test failure in #4)
Seems fine to include in the module's beta phase, but this is up to the module's maintainer
One test failure remaining
This was fixed in 📌 Switch GitLab CI configuration to the template developed by the DA Fixed
The phpstan pipeline is now fixed, but one test is now failing. Oddly enough it doesn't fail on my local machine.